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
hugo: remove obsolete dependencies #49536
Conversation
@GrahamcOfBorg build hugo |
Success on x86_64-linux (full log) Attempted: hugo Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: hugo Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: hugo Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: hugo Partial log (click to expand)
|
I looked at the failure on
|
closing as duplicate of #49525. |
Sorry, I didn't see that. I'll reopen this so you can just rebase this branch on current master and change the commit message. The remaining changes should be in deps.nix. |
e37a0a1
to
084d126
Compare
@xeji done, please take another look. |
Success on x86_64-linux (full log) Attempted: hugo Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: hugo Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: hugo Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: hugo Partial log (click to expand)
|
Thanks. Actually the current deps.nix (without this PR) seems to match the |
if it matches go.mod, we should probably keep the upstream version in this case. Although I did test with mine and it seems to work fine. |
@xeji let me give https://github.com/adisbladis/vgo2nix a try and see what would it yield. |
I just did that. It had some crashes due to not understanding replace directives but when I manually edited the go.mod, vgo2nix essentially provided the current deps.nix. |
oh I see. In that case, let's let upstream deal with the dependencies. Closing this for now. |
Thank you! I think that go 1.11 modules will make packaging a lot easier if they are adopted, but they're still called an experimental feature... |
Definitely! I can't wait for it to go upstream. I think we can generate all of our go packages at that point. |
Motivation for this change
Hugo v0.50 was released 2 days ago. I also added myself to the maintainer list.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)/cc @schneefux