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
lua5.1: fix broken source fetch #40748
Conversation
url = "http://anonscm.debian.org/cgit/pkg-lua/lua5.1.git/plain/debian/patches/" | ||
+ "0004-Fix-stack-overflow-in-vararg-functions.patch?id=b75a2014db2ad65683521f7bb295bfa37b48b389"; | ||
url = "https://sources.debian.org/data/main/l/lua5.1/5.1.5-8.1/debian/patches/" | ||
+ "0004-Fix-stack-overflow-in-vararg-functions.patch"; | ||
sha256 = "05i5vh53d9i6dy11ibg9i9qpwz5hdm0s8bkx1d9cfcvy80cm4c7f"; |
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.
This seems to build fine, but nix-prefetch-url https://sources.debian.org/data/main/l/lua5.1/5.1.5-8.1/debian/patches/0004-Fix-stack-overflow-in-vararg-functions.patch
returns a different sha256. Since the old URL doesn't exist anymore I'm not sure if I'm doing something wrong with my command or if this is a different patch, with the same filename, though the pkg seems to build. Any hints?
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.
anonscm.debian.org
was moved and restructured, see #39927. You can try to find the original patch somewhere in salsa.debian.org
. Unfortunately the sources.debian.org
Debian URLs are not under version control, so they're not stable either and should not be used with fetchpatch
.
Currently the best solution for these Debian patches is to download the correct patch manually and add it to nixpkgs.
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.
Aha, thanks for the context; I've downloaded the file and inlined it now so that it won't go stale again.
@GrahamcOfBorg build lua5_1 |
No attempt on x86_64-linux (full log) The following builds were skipped because they don't evaluate on x86_64-linux: lua5_1 Partial log (click to expand)
|
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: lua5_1 Partial log (click to expand)
|
The upstream src URL for the patch appears to no longer exist. Per discussion in NixOS#39927, the upstream URL is not stable, so this commit inlines the patch in the nixpkgs src tree.
@GrahamcOfBorg build lua5_1 |
Success on x86_64-darwin (full log) Attempted: lua5_1 Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: lua5_1 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: lua5_1 Partial log (click to expand)
|
Thanks! |
The upstream src URL for the patch appears to no longer exist. Per discussion in NixOS#39927, the upstream URL is not stable, so this commit inlines the patch in the nixpkgs src tree. (cherry picked from commit 17f5001)
The upstream src URL for the patch appears to no longer exist; moved to a
different debian location.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)