release candidate v3.4.2 #1152
Open
release candidate v3.4.2 #1152
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Minor syntax fix .. is required for fetching files from parent dir ... without .. it throws error * Update README and travis.yml to acknowledge develop * Fix typo in readme * add free missing (#746) Co-authored-by: David D <[email protected]> * Update readme (#744) * Fix typo in readme * Update Readme Debug section: add solution to cmake error "could not find JNI" Co-authored-by: Hassan Sahibzada <[email protected]> * Add instruction to set offline mode (#741) Co-authored-by: Hassan Sahibzada <[email protected]> Co-authored-by: Ketul shah <[email protected]> Co-authored-by: Jeremy Gunawan <[email protected]> Co-authored-by: waikup83 <[email protected]> Co-authored-by: David D <[email protected]> Co-authored-by: Hassan Sahibzada <[email protected]> Co-authored-by: Divya Sampath Kumar <[email protected]>
* untie stream-name and thing-name * fixup spacing Co-authored-by: tom schuring <[email protected]>
* expose addFileLoggerPlatformCallbacksProvider in Java with JNI * updated addFileLoggerPlatformCallbacksProvider to parse file path * added a callback from JNI to a Java function for logging * updated logPrintFunc calback * missed client_handle initialization in constructor * added comments * white space change to trigger travis
* github actions setup; disable travis for develop * fix syntax error: * fix syntax error * fix linux builds * fix linux builds * fix linux builds * add missing packages * add keys to env vars * check env vars * check env vars * fix builds * fix builds on ga * test mac clang on ga * test mac clang on ga * test mac clang on ga * test mac clang on ga * test mac clang on ga * unsetting the token * unsetting the token * test macos clang * test macos clang * stop travis builds on each push * use oidc * fix windows build * fix test instruction * comment failing builds on travis and ga * update gcc and macos versions * update cmake * update cmake * use latest os, update log levels, uncomment travis * update failing build * update failing build * trigger travis * trigger travis * Revert "trigger travis" This reverts commit d511f15. * update msvc path, add windows to ga, remove from travis * trigger travis * trigger travis w/o passing builds * add cpath and ldflags for mac build * Revert "add cpath and ldflags for mac build" This reverts commit fafad7b. * test ubsan build * fix cmake instruction for mac-gcc * remove travis file * remove travis checks
* use oidc right before tests * run ga builds for branch * fix test path * update branches * set creds expiration time
) * setup arm cross compilation * update the package list * fix in * specify host in log4cplus * specify host in log4cplus * specify host in log4cplus * run all builds with new host config in log4cplus * build with gstreamer and dependencies * build with gstreamer and dependencies * build with gstreamer and dependencies and openssl * build with gstreamer and openssl * specify build and host * specify build and host * specify build host and target * specify same build and host * specify build and host in all cmake files * use a different compiler * use a different compiler * echo compiler * set CC and CXX correctly * set CC and CXX correctly * set openssl platform * set openssl platform * set openssl platform * expose host name for log4cplus * change log4cplus host name * fix cmake syntax error * fix cmke log4plus condition * fix cmake log4plus condition * fix cmake log4plus condition * change var name cmake * change var name in ci * update cmake instruction in ci * check conditions for log4cplus in cmake * check conditions for log4cplus in cmake * use set instead of option in cmake * check messages * fix ci cmake instruction * fix ci cmake instruction * add definition * use env var * use env var * use build args * update readme * remove host-name from qemu arm build * update readme for cross-compile instructions * add aarch64; move log config file * remove qemu
* expose streaming flags in java with jni * fix indentation
It is not a straight-forward process to enable KVS and IoT credentials. It is required to read and follow through https://docs.aws.amazon.com/kinesisvideostreams/latest/dg/how-iot.html.
* add store pressure policy * update ci to add update
* Allow using CPP SDK as a dependency * Update README
* Add static build to CI * gstkvssink should have static linkage in a static build * log4cplus should be a static lib in a static build * Don't force full static link * Update libkvscproducer-CMakeLists.txt Update Producer C Commit hash Co-authored-by: Hassan Sahibzada <[email protected]>
* fix failing builds + upgrade gtest to 1.12 * install missing package * change os and gcc versions * exclude windows build * use event triggers
* fix ci for fork * blank space change to trigger CI
* Minor syntax fix .. is required for fetching files from parent dir ... without .. it throws error * Fix typo in readme * add free missing (#746) Co-authored-by: David D <[email protected]> * Update readme (#744) * Fix typo in readme * Update Readme Debug section: add solution to cmake error "could not find JNI" Co-authored-by: Hassan Sahibzada <[email protected]> * Add instruction to set offline mode (#741) Co-authored-by: Hassan Sahibzada <[email protected]> * Update README and travis.yml to acknowledge develop (#739) Co-authored-by: Hassan Sahibzada <[email protected]> * Gstreamer RTSP Docs Update (#712) Hoping this help others avoid hours of frustration. As noted in the helpful but well hidden issue comment #193 (comment), the Gstreamer examples for RTSP do not run at all. I'm no Gstreamer expert but using `h264parse` works perfectly, while `video/x-h264, format=avc,alignment=au` just causes the pipeline to hang. * free missing property (#767) Co-authored-by: David D <[email protected]> * Update gstkvssink.cpp * Added millisecond support to kvs_gstreamer_file_upload_sample + gstkvssink --------- Co-authored-by: Ketul shah <[email protected]> Co-authored-by: waikup83 <[email protected]> Co-authored-by: David D <[email protected]> Co-authored-by: Hassan Sahibzada <[email protected]> Co-authored-by: Divya Sampath Kumar <[email protected]> Co-authored-by: jdelapla <[email protected]> Co-authored-by: Anton Vattay <[email protected]> Co-authored-by: bkneff <[email protected]>
Co-authored-by: Divya Sampath Kumar <[email protected]>
* pass parameter by reference to avoid copy constructor * nit style --------- Co-authored-by: James Delaplane <[email protected]>
* replace pull_request_target with pull_request; update versions for codeql * add develop to codeql
* Cmake changes after release 3.3.1 * Added fragment ack handler, reverting changes related to edge * Added gsignal element for fragment ack * Fixed cmake dependency for canarydemos * Reverted c++ versrion to 11 * Commented out g signal in fragment act handler * Re-Added the g signal * Kvs sink initialized to avoid bad memory access * Intialized custom objet kvssink * Moved part of cloud watch metrics and logs * Resolved constructor issue * Added remaining functions for cloud watch logs * Reverted changes for cloudwatch shift * Reverted Cmake changes * Reverted remaining changes * Added stream metric signal * G signal changes * G signal parameter changes * Removed persisted ack check * Changed return type in g signal * Added custome object for put frame * Metric struct update * Intialized custome object metrics * Added check for key frame * Added client metrics element * Name changed * Added first frame emit * First frame changes * Reverted first frame changes * Left change * Added g signal emit on first successfull frame * Fixed compile error * Startup latency test_1 * Added first frame check * Resolved comments on PR 984 * Resolved comments on PR 984-remaining * Resolved comments in PR 984_03-16 * resolved comment in PR-276_03-21
* Signals in develop * Add signals and static analysis fixes * Signals in develop * Add better logging * Pull in new producer C commit * Update gstkvssink.cpp * use get metric property * Use log and throw and catch --------- Co-authored-by: NikunjSarda <[email protected]>
* Adding timeouts configurability * Add jni for new timeouts and create stream flag * Revert log4cplus version and cxx standard * Revert versions * Revert C++ version to 11
Signed-off-by: Alex Li <[email protected]> Signed-off-by: Alex Li <[email protected]>
* Update producer-c dependency to allow for Constrained device flag
* Remove '$' from ReadMe commands * Reomve "$ linux.md * Remove "$" from macos.md * Remove "$" from raspberry-pi.md
* try fixing running cmake twice * try a different cpath * update cc, cxx, ldflags, cpath * remove CC and CXX * try /usr/bin/clang and /Applications/Xcode_13.2.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ld * -L/Applications/Xcode_13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.1.sdk/usr/lib/ * specify host for log4cplus * don't install logcplus with brew * try using clang 14 * add missing space * don't set cc and cxx * configure CMAKE_OSX_SYSROOT before creating the configure command * try ldflags and paths with cmake fix * remove prefix from configure_command * update ldflags * fix install * rearrange * put back configure * put back host in cmake instruction * PREFIX /build * try using source_dir * remove source_dir * move back prefix * put back flags * only remove prefix from configure_command * missing space * if (APPLE) * install * check compiler * use '' STREQUAL * check clang compiler * osxsysroot * Add back osx_sysroot * remove space from libcurl * project(log4cplus-download NONE) to fix gcc * try LANGUAGES CPP) in all cmake files * try C) * try CXX * fix ubuntu build
* Update ci.yml compiler env var * Add refreshenv to windows build * Add import module for refreshenv call * Testing * Revert windows CI changes * Update ci.yml * Trying different PKG_CONFIG_EXECUTABLE path * Add msvc_ to path * More build testing * more * more * more * more * more * more * more * more * more * more * more * more * more * more * more * more * works, testing which line(s) is needed * more * more * Cleanup ci * cleanup cmake * Update README.md * Add check for C vs D drive * Revert ReadMe changes
* m1 build * fix indentation * make once * cmake update * add tests to m1 build * change producer c git_tag to develop * fix mac clang
Co-authored-by: Tom spot Callaway <[email protected]>
* install some files * cleanup --------- Co-authored-by: Tom spot Callaway <[email protected]>
* make versioned shared libraries * Minor syntax fix .. is required for fetching files from parent dir ... without .. it throws error * Fix typo in readme * add free missing (#746) Co-authored-by: David D <[email protected]> * Update readme (#744) * Fix typo in readme * Update Readme Debug section: add solution to cmake error "could not find JNI" Co-authored-by: Hassan Sahibzada <[email protected]> * Add instruction to set offline mode (#741) Co-authored-by: Hassan Sahibzada <[email protected]> * Update README and travis.yml to acknowledge develop (#739) Co-authored-by: Hassan Sahibzada <[email protected]> * Gstreamer RTSP Docs Update (#712) Hoping this help others avoid hours of frustration. As noted in the helpful but well hidden issue comment #193 (comment), the Gstreamer examples for RTSP do not run at all. I'm no Gstreamer expert but using `h264parse` works perfectly, while `video/x-h264, format=avc,alignment=au` just causes the pipeline to hang. * Resolving merge conflicts * Update version # * Update the version string property with the version * Remove duplicate definitions --------- Co-authored-by: Tom spot Callaway <[email protected]> Co-authored-by: Ketul shah <[email protected]> Co-authored-by: Jeremy Gunawan <[email protected]> Co-authored-by: waikup83 <[email protected]> Co-authored-by: David D <[email protected]> Co-authored-by: Hassan Sahibzada <[email protected]> Co-authored-by: Divya Sampath Kumar <[email protected]> Co-authored-by: Anton Vattay <[email protected]>
* Change parent class stat change position * Add failure check Co-authored-by: Niyati Maheshwari <[email protected]> * Testing on mac os 13 * Revert ci to mac os 12 * Add comments explaining state transition guidelines --------- Co-authored-by: Niyati Maheshwari <[email protected]>
* Default to Release build * Dont update log4cplus version for Windows
* Ensure region for kvssink is set either through env or property * Address comments * Revert log4cplus version * Fix redme for region setting
* Ensure region for kvssink is set either through env or property * Address comments
* Make clone command a code block for copy button * Specify languages for code blocks
* add warn on bus, have put frame return actual status * CI tests account for changed putFrame return value * More CI putFrame checks update * Uncomment if block * Update GST warning type * Update .gitignore * Update gstkvssink.cpp --------- Co-authored-by: Hassan Sahibzada <[email protected]>
* Update ci.yml * Update ci.yml * Update ci.yml * Update ci.yml * Trying with 32bit MSVC * Set longPaths in bat file too * Update ci.yml * Update build_windows.bat * Shorten dependency path to d * Update build_windows.bat * Testing more with dependency, "d" works * Update build_windows.bat * Update build_windows.bat * Update ci.yml * Update ci.yml * Update ci.yml * Update ci.yml * Add a comment * Update ci.yml * Remove extra call
* remove unnecessary things from build * setting link/include directories for samples, make sure build type is respected for log4cplus * specify static name of library so it can be found * link to open source directory before adding KinesisVideoProducer target so dependent libraries can be found to fix cross compile builds * LDFLAGS specifying path with alternative curl * Fix Windows CI (#1149) * Update ci.yml * Update ci.yml * Update ci.yml * Update ci.yml * Trying with 32bit MSVC * Set longPaths in bat file too * Update ci.yml * Update build_windows.bat * Shorten dependency path to d * Update build_windows.bat * Testing more with dependency, "d" works * Update build_windows.bat * Update build_windows.bat * Update ci.yml * Update ci.yml * Update ci.yml * Update ci.yml * Add a comment * Update ci.yml * Remove extra call * update log4cplus version to 2.0.7 where they've update Catch2 issue const expr for MINSIGSTKSZ * explicit link to log4cplus for kvssink since it includes logger headers * update windows log4cplus to 2.0.3 * replace windows link library for log4cplus to what it was previously * log4cplus windows 2.0.3 has an issue where tests hang forever, downgrading to 2.0.2 * on windows need to disable threadpool entirely or else will hang, alternataive is to explicitly do init and de-init of the library but we currently do not call inititialize since we use a static getInstance --------- Co-authored-by: Stefan Kieszkowski <[email protected]>
* Add PR description lint * windows build fix * Set CMAKE_OBJECT_PATH_MAX for windows * Add openssl build
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
13 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue #, if available:
What was changed?
Why was it changed?
How was it changed?
What testing was done for the changes?
producerTestrun locally, all passingBy submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.