-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
oprofile: fix build with gcc14 #392898
oprofile: fix build with gcc14 #392898
Conversation
|
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.
I like these solutions way better than just silencing the warnings!
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 separate commits for each logical change. fixing build by patching in one commit, cleanup in another.
- add patch similar to upstream commit https://sourceforge.net/p/oprofile/oprofile/ci/b0acf9f0c0aac93bf6f3e196d7a52c9632ff4475 patch from commit does not work as is because tarball used for build has `configure` inside and changes to `configure.ac` do not affect it
- replace `--replace` with `--replace-fail` - replace `rec` with `finalAttrs`
84788a0
to
04f5647
Compare
Split into 2 commits:
|
|
oprofile: fix build with gcc14
https://sourceforge.net/p/oprofile/oprofile/ci/b0acf9f0c0aac93bf6f3e196d7a52c9632ff4475
patch from commit does not work as is because tarball used for build
has
configure
inside and changes toconfigure.ac
do not affect itoprofile: cleanup
--replace
with--replace-fail
rec
withfinalAttrs
Fixes build failure of
oprofile
, fails since2025-01-14
(update to GCC14 #356812):https://hydra.nixos.org/job/nixos/trunk-combined/nixpkgs.oprofile.x86_64-linux
https://hydra.nixos.org/build/293043082
Error log:
More logs from inside
./configure
withset -x
:Line that fails (does not produce output that is checked after):
From manual build of C code inside
./configure
(same error as in upstream commit message):Upstream commit patch is based on:
https://sourceforge.net/p/oprofile/oprofile/ci/b0acf9f0c0aac93bf6f3e196d7a52c9632ff4475
Tracking issue: #388196
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.