adamdebreceni commented on a change in pull request #1004: URL: https://github.com/apache/nifi-minifi-cpp/pull/1004#discussion_r603838206
########## File path: .github/workflows/ci.yml ########## @@ -132,12 +142,13 @@ jobs: run: | sudo apt-add-repository -y "ppa:ubuntu-toolchain-r/test" sudo apt update - sudo apt install -y gcc-4.8 g++-4.8 bison flex uuid-dev openssl libcurl4-openssl-dev ccache libpython3-dev liblua5.1-0-dev libssh2-1-dev + sudo apt install -y gcc-4.8 g++-4.8 bison flex uuid-dev openssl libcurl4-openssl-dev ccache libpython3-dev liblua5.1-0-dev libssh2-1-dev libsqliteodbc + sudo ln -s /usr/lib/x86_64-linux-gnu/odbc/libsqlite3odbc.so /usr/lib/x86_64-linux-gnu/libsqlite3odbc.so Review comment: the binary will be working fine, the symlink is only (apparently) needed on ubuntu for the ODBCService iff we don't want to employ an odbc.ini file, and without an odbc.ini file the connection string is inherently platform specific anyway -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org