-
-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
sfml: 2.5.1 -> 2.6.1; csfml: 2.5.2 -> 2.6.0 #312161
base: master
Are you sure you want to change the base?
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
@ofborg build sfml csfml |
Co-authored-by: Yohann Boniface <edhyjox@gmail.com>
Co-authored-by: Sigmanificient <edhyjox@gmail.com>
Co-authored-by: Sigmanificient <edhyjox@gmail.com>
Result of 3 packages marked as broken and skipped:
9 packages built:
|
Result of 1 package marked as broken and skipped:
25 packages built:
|
pkg-config is broken upsream. |
When building the
Following nix advise, I tried to build with these changes: --- a/pkgs/by-name/cs/csfml/package.nix 2024-05-21 09:41:18.068576345 +0200
+++ b/pkgs/by-name/cs/csfml/package.nix 2024-05-21 09:40:58.865882958 +0200
@@ -20,7 +20,11 @@
nativeBuildInputs = [ cmake ];
buildInputs = [ sfml ];
cmakeFlags = [ "-DCMAKE_MODULE_PATH=${sfml}/share/SFML/cmake/Modules/" ];
- patches = [ ./pkgconfig.patch ];
+
+ prePatch = ''
+ substituteInPlace tools/pkg-config/* \
+ --replace-fail 'libdir=''${exec_prefix}/@CMAKE_INSTALL_LIBDIR@' "libdir=@CMAKE_INSTALL_FULL_LIBDIR@"
+ '';
meta = {
homepage = "https://www.sfml-dev.org/"; It produces a valid |
dev out change broke csfml. Currently trying to fix the issue |
Removing multiple outputs as it is broken right now for SFML/CSFML |
Co-authored-by: Yohann Boniface <edhyjox@gmail.com>
@ofborg build sfml csfml |
When the build ends, the pr should be good to go. Waiting for review by maintainers |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/3979 |
We still have no answer from the maintainers of both sfml and csfml after 3 weeks. I'm adding myself as maintainer |
meta = with lib; { | ||
prePatch = '' | ||
substituteInPlace tools/pkg-config/* \ | ||
--replace-fail 'libdir=''${exec_prefix}/@CMAKE_INSTALL_LIBDIR@' "libdir=@CMAKE_INSTALL_FULL_LIBDIR@" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please try to upstream this
Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
Description of changes
Works fine on a few projects of mine on
x86_64-linux
Managed to get it working on
x86_64-darwin
CSFML & SFML packages don't get a lot of updates on the nixpkgs. I might add myself as a maintainer if we don't get a reply shortly
I also had to patch pkgconfig path in the cmake as it is broken upstream. A or fixes the issue for SFML package and I replicated the patch for CSFML.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.