Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

gitlab-workhorse: migrate to using buildGoPackage #52581

Conversation

vdemeester
Copy link
Member

@vdemeester vdemeester commented Dec 20, 2018

Motivation for this change

Part of #52469, make the gitlab-workhorst derivation using buildGoPackage 馃懠

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

Signed-off-by: Vincent Demeester <vincent@sbr.pm>
@flokli
Copy link
Contributor

flokli commented Dec 24, 2018

This breaks pkgs/applications/version-management/gitlab/update.py, for gitlab-workhorse (like it's already broken for gitlab-shell, see description of #50962):

For some weird reason in how go packages are being built, meta.position for gitlab-shell points to pkgs/development/go-modules/generic/default.nix, so update-source-version isn't able to bump this automatically (but we show the version and hash to bump to on the console).

Since then, the discovered location meta.position seems to have moved again: `

/nix/store/5chw603011946kg8w42961zsp4p4shr9-common-updater-scripts/bin/.update-source-version-wrapped: error: Couldn't locate old source hash '1r75jj0xb4jv5fq2ihxk0vlv43gsk523zx86076mwph1g75gi1nz' (or it appeared more than once) in '/home/flokli/dev/nixos/nixpkgs-2/lib/attrsets.nix'!

I agree it makes a lot of sense to switch to buildGoPackage, but updating gitlab and all dependencies is a bit complicated to do manually - so having tooling for it is nice.

Any idea how to fix update-source-version for buildGoPackage derivations?

@vdemeester
Copy link
Member Author

ohh damn 馃槗 馃
I'll try to look at it (after the holidays though 馃懠)

Let me close this one for now then 馃槈

@vdemeester vdemeester closed this Dec 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants