Is it possible to disable `vendorSha256` with buildGoModule to make updates easier with niv?

bbigras · July 11, 2020, 5:50pm

I’m using niv with git-actions but I would like to avoid having to update vendorSha256 on every dependency update.

From the doc it says to run go mod vendor. But that’s on the upstream repo right? Can I do it in a package using fetchFromGitHub without modifying the upstream repo?

github.com

NixOS/nixpkgs/blob/a8fd3c7edddcd51e833d3ad1f2fdf3174e608902/doc/languages-frameworks/go.xml#L70-L78

    
      
           <para>
             <varname>vendorSha256</varname> can also take <varname>null</varname> as an input.
          
          
   When `null` is used as a value, rather than fetching the dependencies
             and vendoring them, we use the vendoring included within the source repo.
             If you'd like to not have to update this field on dependency changes, 
             run `go mod vendor` in your source repo and set 'vendorSha256 = null;'
           </para>
          </section>

colemickens · July 12, 2020, 9:31pm

Hi @bbigras. I apologize for not answering your question directly (though I was just wondering the same about vendorSha256 and cargoSha256 recently [related to adopting flakes]). But, I can offer this that might help. You can use nix-prefetch to get those values, as an example: nixpkgs-wayland/update.sh at 9b5a54a5e449debd28cf1bb074e4a694b4fc9630 · colemickens/nixpkgs-wayland · GitHub That’s the relevant part of the script I wrote to update nixpkgs-wayland packages which includes git/hg repos, and go/rust packages, and updates are automated. I don’t love this being in bash though, and generally I wish I could lean on niv/flakes for similar reasons as you.

jonringer · July 13, 2020, 7:10pm

I think the correct long-term action would be to make niv aware of vendor sha’s, similar to nixpkgs-update is now aware of them as well.

bbigras · July 13, 2020, 8:34pm

Thank you very much @colemickens. I’ll give it a try.

bbigras · July 13, 2020, 8:35pm

This make sense. I’ll open a feature request. Thanks!