Skip to content

Commit

Permalink
Fix for aws-sdk-cpp log interface change - issue 44
Browse files Browse the repository at this point in the history
Updating github actions to fix CI issue

Adding sudo for ldconfig

Only run ldconfig on linux
  • Loading branch information
russnic committed Apr 10, 2024
1 parent 9515e85 commit f9d6f72
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
7 changes: 5 additions & 2 deletions .github/workflows/c-cpp.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,13 @@ jobs:
brew install --overwrite gstreamer
- uses: actions/checkout@v3
- run: pip3 install meson ninja
- run: 'git clone --recurse-submodules --depth 1 https://github.com/aws/aws-sdk-cpp.git -b 1.10.30'
- run: cmake -DBUILD_ONLY="s3;sts" aws-sdk-cpp
- run: 'git clone --recurse-submodules --depth 1 https://github.com/aws/aws-sdk-cpp.git -b 1.11.304'
- run: cmake -DCMAKE_PREFIX_PATH=/usr/local/ -DCMAKE_INSTALL_PREFIX=/usr/local/ -DBUILD_ONLY="s3;sts" aws-sdk-cpp
- run: make -j 4
- run: sudo make install
- name: Run ldconfig on linux
if: runner.os == 'Linux'
run: sudo ldconfig
- run: meson build
- run: ninja -C build
- run: 'export LD_LIBRARY_PATH=/usr/local/lib:$LD_LIBRARY_PATH'
Expand Down
4 changes: 4 additions & 0 deletions src/gsts3multipartuploader.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,10 @@ class Logger : public Aws::Utils::Logging::LogSystemInterface
va_end (varargs);
}

void vaLog(Aws::Utils::Logging::LogLevel logLevel, const char* tag, const char* formatStr, va_list args) override
{
}

void LogStream(Aws::Utils::Logging::LogLevel log_level, const char* tag, const Aws::OStringStream &message_stream) override
{
Log(log_level, tag, "%s", message_stream.str().c_str());
Expand Down

0 comments on commit f9d6f72

Please sign in to comment.