New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
qpid-cpp: fix build #37717
qpid-cpp: fix build #37717
Conversation
unfortunately the binaries don't work well, will start investigation |
Failure on aarch64-linux (full log) Attempted: qpid-cpp Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: qpid-cpp Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: qpid-cpp Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: qpid-cpp Partial log (click to expand)
|
The compilation broke due to the flag `-Werror=int-in-bool-context` which caused several compilation errors with GCC v7. Disabling this warning manually with `-Wno-error` in `NIX_CFLAGS_COMPILE` should be fine. This package experienced several radical changes as the entire python build in `$src/management/python` was broken since the given Python interpreter missed several needed modules (including `pythonPackages.qpid-python`). As the CMake build tried to invoke the affected `setup.py` manually and patched the shebangs with `disutil` and caused non-functional executables, I split the package up into two parts, the actual `qpid-cpp` lib and the Python module that will be composed using `buildEnv`. Furthermore I added myself as maintainer for the package as the diff became quite huge and we should have more folks available to maintain this. See https://hydra.nixos.org/build/71519082/log See tickets NixOS#36453 and NixOS#31747
Failure on aarch64-linux (full log) Attempted: qpid-cpp Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: qpid-cpp Partial log (click to expand)
|
|
@garbas could you backport this to release-18.03, please? This is also broken on 18.03: https://hydra.nixos.org/build/71818713 |
Motivation for this change
The compilation broke due to the flag
-Werror=int-in-bool-context
which caused several compilation errors with GCC v7. Disabling this
warning manually with
-Wno-error
inNIX_CFLAGS_COMPILE
should befine.
This package experienced several radical changes as the entire python
build in
$src/management/python
was broken since the given Pythoninterpreter missed several needed modules (including
pythonPackages.qpid-python
). As the CMake build tried to invoke theaffected
setup.py
manually and patched the shebangs withdisutil
andcaused non-functional executables, I split the package up into two
parts, the actual
qpid-cpp
lib and the Python module that will becomposed using
buildEnv
.Furthermore I added myself as maintainer for the package as the diff
became quite huge and we should have more folks available to maintain
this.
See https://hydra.nixos.org/build/71519082/log
See tickets #36453 and #31747
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)