Skip to content
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

webkitgtk: set -DENABLE_JOURNALD_LOG=OFF if !systemdSupport #179391

Merged
merged 2 commits into from Oct 6, 2022
Merged

webkitgtk: set -DENABLE_JOURNALD_LOG=OFF if !systemdSupport #179391

merged 2 commits into from Oct 6, 2022

Conversation

ghost
Copy link

@ghost ghost commented Jun 27, 2022

Description of changes

When webkitgtk is built with systemdSupport=false the following build failure occurs:

-- Could NOT find Journald (missing: Journald_LIBRARY Journald_INCLUDE_DIR)
CMake Error at Source/cmake/OptionsGTK.cmake:486 (message):
  libsystemd or libelogind are needed for ENABLE_JOURNALD_LOG
Call Stack (most recent call first):
  Source/cmake/WebKitCommon.cmake:220 (include)
  CMakeLists.txt:20 (include)

-- Configuring incomplete, errors occurred!
See also "/build/webkitgtk-2.36.0/build/CMakeFiles/CMakeOutput.log".
See also "/build/webkitgtk-2.36.0/build/CMakeFiles/CMakeError.log".

Let's add -DENABLE_JOURNALD_LOG=OFF if !systemdSupport in order to fix this.

Things done
  • Built on platform(s)
    • powerpc64le-linux
    • x86_64-linux
    • aarch64-linux
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Otherwise we get this build failure:

-- Could NOT find Journald (missing: Journald_LIBRARY Journald_INCLUDE_DIR)
CMake Error at Source/cmake/OptionsGTK.cmake:486 (message):
  libsystemd or libelogind are needed for ENABLE_JOURNALD_LOG
Call Stack (most recent call first):
  Source/cmake/WebKitCommon.cmake:220 (include)
  CMakeLists.txt:20 (include)

-- Configuring incomplete, errors occurred!
See also "/build/webkitgtk-2.36.0/build/CMakeFiles/CMakeOutput.log".
See also "/build/webkitgtk-2.36.0/build/CMakeFiles/CMakeError.log".
@ofborg ofborg bot added 10.rebuild-darwin: 11-100 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux labels Jun 27, 2022
@ghost
Copy link
Author

ghost commented Jul 27, 2022

Ping...

1 similar comment
@ghost
Copy link
Author

ghost commented Oct 3, 2022

Ping...

@bobby285271
Copy link
Member

bobby285271 commented Oct 5, 2022

Did not test, can we also remove -DUSE_SYSTEMD=OFF?

@ghost
Copy link
Author

ghost commented Oct 5, 2022

Did not test, can we also remove -DUSE_SYSTEMD=OFF?

Indeed, we can.

@bobby285271 bobby285271 merged commit 946f7ba into NixOS:master Oct 6, 2022
@ghost ghost deleted the pr/webkitgtk/ENABLE_JOURNALD_LOG=off branch October 6, 2022 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10.rebuild-darwin: 11-100 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant