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
gdb: Flexible target matching for darwin #36378
Conversation
I filed this with the base branch of |
Please target master instead, cherry-picking happens in the other direction. |
Thanks for tackling this! 👍. |
Outside of the nix-build the target is `x86_64-apple-darwin17.4.0`, while inside the target is `x86_64-apple-darwin`. This difference causes the fallback target configuration for darwin, which disables gdb. Add a patch to make the target matching more flexible.
4508761
to
4c76a21
Compare
Retargeted to master. |
@GrahamcOfBorg build gdb |
Success on aarch64-linux (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on x86_64-darwin (full log) Partial log (click to expand)
|
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.
Thanks!
It looks like that commit broke the build for ldc on Darwin because gdb is used in some tests and doesn't work without a signed binary. https://hydra.nixos.org/build/72723666/nixlog/5
I guess this should be fixed as soon as #38624 is merged. |
We should disable the gdb related tests, we already do that in some other places. |
I think you misunderstood. |
Outside of the nix-build the target is
x86_64-apple-darwin17.4.0
,while inside the target is
x86_64-apple-darwin
. This differencecauses the fallback target configuration for darwin, which disables
gdb. Add a patch to make the target matching more flexible.
This feels like patching a symptom more than a cause, is there a more principled alternative?
Motivation for this change
#35677
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)