rattler icon indicating copy to clipboard operation
rattler copied to clipboard

[Feature Request] Add functionality to install package files

Open vnling opened this issue 1 year ago • 6 comments
trafficstars

Desired behavior: The matchspec parser can parse specs such as package.tar.bz2

Currently I run into unimplemented code when trying to parse package file specs, as described in https://github.com/mamba-org/rattler/issues/640. Can I ask if implementing this is on the roadmap for the team? If not, would a contribution to add this functionality need to handle anything additional on the package linking side, or is it just a matter of adding the matchspec parsing logic?

vnling avatar May 06 '24 12:05 vnling

TBH Im not entirely sure as I'm not entirely sure how it should work.

Do you have some examples and how you expect it would work?

But any contribution would be appreciated!

baszalmstra avatar May 06 '24 12:05 baszalmstra

I assume it would be replicating the logic in https://github.com/conda/conda/blob/2ca3d3532a176b4ece917d4cf39ab5ca47ffec6c/conda/models/match_spec.py#L631-L642? Happy to give it a shot, just wanted to know if it was something the team was working on already. Thanks!

vnling avatar May 06 '24 13:05 vnling

Thanks! Yeah, that looks simple enough. I think from the MatchSpec struct we are missing the url field. This should then probably also be added (and to NamelessMatchSpec).

baszalmstra avatar May 06 '24 13:05 baszalmstra

@vnling no one is currently working on this - feel free to give it a shot. You can also join our Discord in case you need any help :)

wolfv avatar May 06 '24 14:05 wolfv

@vnling just curious if you're working on this, if not I can probably throw up a PR since this will eliminate a lot of code on my side :)

iamthebot avatar May 14 '24 01:05 iamthebot

Hey @iamthebot, am not currently working on it, happy for you to!

vnling avatar May 14 '24 12:05 vnling

This is now supported through https://github.com/mamba-org/rattler/pull/725

baszalmstra avatar Jul 08 '24 09:07 baszalmstra