-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
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
syslinux: fix submodule clone #316878
syslinux: fix submodule clone #316878
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.
Tested that it works in a nix-shell
(with unpackPhase
and patchPhase
). Not sure I understand the problem, but being able to manipulate gnu-efi-src might be nice anyway, so LGTM.
--replace /usr/include/ "" | ||
--replace-fail /usr/include/ "" | ||
|
||
# needs to be manually put in place because submodule hash contains an extra g in the beginning gd34132e |
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'm not sure the g
is the problem here: the error message in the current behavior has fatal: Fetched in submodule path 'gnu-efi', but it did not contain d34132e62f666904158c7ec2f1eef5a9d5281c36. Direct fetching of that commit failed.
, so it seems to get the git hash just fine.
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.
Then my best next guess is, that sourceforge is weird. I didn't want to click through many pages to find the old release.
Is there a linked issue? What problem does that solve? I don't see a rationale anywhere about this, or a concise way to reproduce an issue. |
gnu-efi-src = fetchFromGitHub { | ||
owner = "ncroxon"; | ||
repo = "gnu-efi"; | ||
# see 363d61c4f112b972649b19d67e96b9321f738f00 in syslinux |
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.
# see 363d61c4f112b972649b19d67e96b9321f738f00 in syslinux | |
# see 363d61c4f112b972649b19d67e96b9321f738f00 in syslinux | |
# https://repo.or.cz/syslinux.git/commitdiff/363d61c4f112b972649b19d67e96b9321f738f00 |
I took some time into actually re-familiarizing myself with the derivation, then fixed the build so we use |
Description of changes
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.