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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dev-python/snakemake: version bump #1323

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mschubert
Copy link
Contributor

No description provided.

@mschubert mschubert force-pushed the snakemake branch 2 times, most recently from c24c911 to 6762c65 Compare February 18, 2025 11:44
Copy link
Member

@Nowa-Ammerlaan Nowa-Ammerlaan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fails on my end with:

assets.AssetDownloadError: Failed to download asset https://raw.githubusercontent.com/snakemake/snakemake/main/LICENSE.md: <urlopen error [Errno -2] Name or service not known>

@mschubert
Copy link
Contributor Author

mschubert commented Feb 24, 2025

Fails on my end with:

assets.AssetDownloadError: Failed to download asset https://raw.githubusercontent.com/snakemake/snakemake/main/LICENSE.md: <urlopen error [Errno -2] Name or service not known>

It seems that setup.py downloads a range of assets, which will fail if there is no internet connection during the installation.

I'm not sure if/how this is fixable, I'll check the pypi tarball.

We should be able to fix that by providing all the missing files as additional sources.

Is there an example of another ebuild that already does this?

@Nowa-Ammerlaan
Copy link
Member

Is there an example of another ebuild that already does this?

The usual solution is to use the pypi release tarball instead of the github tag tarball. But not sure if that will resolve the problem in this case.

@mschubert
Copy link
Contributor Author

I can confirm that the pypi tarball does not include the assets; I'm open to suggestions what a good solution is here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants