symbolic icon indicating copy to clipboard operation
symbolic copied to clipboard

Cargo: lock the dependencies

Open RaitoBezarius opened this issue 4 years ago • 4 comments

I am currently packaging symbolic in NixOS and we often need lock files to pin all dependencies, it seems like this repository do not have it which would require us to maintain the Cargo.lock ourselves, do you think it is possible to add it upstream?

Thank you for your work :)

RaitoBezarius avatar Oct 10 '21 14:10 RaitoBezarius

May I ask which part exactly you want to package?

In either case, I think one binary artifact that we ship is symbolic-cabi as part of the python package. Also for that usecase, I think its fair to add a lockfile. Let me run that by the team as well.

Swatinem avatar Oct 11 '21 12:10 Swatinem

It doesn't really make sense for a library project to have a Cargo.lock file though, does it? it specifies the versions of our dependencies which it can handle and it is up to the binary using this to decide on the shape of their Cargo.lock file.

flub avatar Oct 11 '21 12:10 flub

Agreed regarding locking the whole project, @flub I was under the assumption that the symbolic-cabi is a binary, and it would be good to lock it, @Swatinem

If it does not make any sense, I can still lock it upstream, thanks !

RaitoBezarius avatar Oct 12 '21 06:10 RaitoBezarius

We used to have Cargo.lock checked in for the C-ABI binary. When we moved everything into a workspace, that was dropped. There's nothing wrong with bringing back Cargo.lock; however, we should ensure that it is regularly updated in CI so that we do not miss incompatibilities. If it's not a big deal to pin this upstream, that sounds like the best compromise.

jan-auer avatar Oct 15 '21 06:10 jan-auer