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

Fix provenance in case broken packages are in the environment #2595

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

maxnoe
Copy link
Member

@maxnoe maxnoe commented Jul 18, 2024

Should fix the docs failure seen in #2543

Copy link

Passed

Analysis Details

0 Issues

  • Bug 0 Bugs
  • Vulnerability 0 Vulnerabilities
  • Code Smell 0 Code Smells

Coverage and Duplications

  • Coverage No coverage information (94.20% Estimated after merge)
  • Duplications 0.00% Duplicated Code (0.60% Estimated after merge)

Project ID: cta-observatory_ctapipe_AY52EYhuvuGcMFidNyUs

View in SonarQube

@maxnoe maxnoe requested review from mexanick and LukasBeiske July 18, 2024 10:15
@maxnoe
Copy link
Member Author

maxnoe commented Jul 18, 2024

The failure was due to pypa/setuptools#4480 and pypa/setuptools#4482

In [1]: from importlib.metadata import distributions
In [2]: [d._path for d in distributions() if d.name is None]
Out[2]: 
[PosixPath('/home/mnoethe/Uni/CTA/ctapipe/.venv/lib/python3.11/site-packages/setuptools/_vendor/zipp-3.19.2.dist-info'),
 PosixPath('/home/mnoethe/Uni/CTA/ctapipe/.venv/lib/python3.11/site-packages/setuptools/_vendor/jaraco.text-3.12.1.dist-info'),
 PosixPath('/home/mnoethe/Uni/CTA/ctapipe/.venv/lib/python3.11/site-packages/setuptools/_vendor/backports.tarfile-1.2.0.dist-info'),
 PosixPath('/home/mnoethe/Uni/CTA/ctapipe/.venv/lib/python3.11/site-packages/setuptools/_vendor/wheel-0.43.0.dist-info'),
 PosixPath('/home/mnoethe/Uni/CTA/ctapipe/.venv/lib/python3.11/site-packages/setuptools/_vendor/inflect-7.3.1.dist-info'),
 PosixPath('/home/mnoethe/Uni/CTA/ctapipe/.venv/lib/python3.11/site-packages/setuptools/_vendor/autocommand-2.2.2.dist-info'),
 PosixPath('/home/mnoethe/Uni/CTA/ctapipe/.venv/lib/python3.11/site-packages/setuptools/_vendor/importlib_resources-6.4.0.dist-info'),
 PosixPath('/home/mnoethe/Uni/CTA/ctapipe/.venv/lib/python3.11/site-packages/setuptools/_vendor/jaraco.context-5.3.0.dist-info'),
 PosixPath('/home/mnoethe/Uni/CTA/ctapipe/.venv/lib/python3.11/site-packages/setuptools/_vendor/typeguard-4.3.0.dist-info'),
 PosixPath('/home/mnoethe/Uni/CTA/ctapipe/.venv/lib/python3.11/site-packages/setuptools/_vendor/importlib_metadata-8.0.0.dist-info'),
 PosixPath('/home/mnoethe/Uni/CTA/ctapipe/.venv/lib/python3.11/site-packages/setuptools/_vendor/jaraco.functools-4.0.1.dist-info')]

@maxnoe maxnoe merged commit 24201c2 into main Jul 18, 2024
13 of 14 checks passed
@maxnoe maxnoe added this to the 0.22.0 milestone Sep 12, 2024
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.

3 participants