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

gh-122044: Don't error during gitignore filtering with no files #122045

Merged
merged 1 commit into from
Jul 27, 2024

Conversation

sethmlarson
Copy link
Contributor

@sethmlarson sethmlarson commented Jul 19, 2024

When a package doesn't have any files the script errors during gitignore filtering instead of the helpful error message.

@sethmlarson
Copy link
Contributor Author

Running this patch locally after removing the Modules/_blake2/ directory results in this message:

$ make regen-sbom
python3.13 ./Tools/build/generate_sbom.py
No valid paths found at path 'Modules/_blake2/impl/**' for package 'libb2
See 'https://devguide.python.org/developer-workflow/sbom' for more information.
make: *** [Makefile:3067: regen-sbom] Error 1

@sethmlarson sethmlarson requested a review from hugovk July 26, 2024 14:21
@sethmlarson
Copy link
Contributor Author

Hope you don't mind reviewing this one Hugo, fairly small one that fixes SBOMs when projects get removed (as is being done now for libb2).

@hugovk
Copy link
Member

hugovk commented Jul 27, 2024

Thanks! And backport to 3.12?

@hugovk hugovk merged commit 4e04d1a into python:main Jul 27, 2024
41 checks passed
@sethmlarson sethmlarson added needs backport to 3.12 bug and security fixes needs backport to 3.13 bugs and security fixes labels Jul 27, 2024
@miss-islington-app
Copy link

Thanks @sethmlarson for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Thanks @sethmlarson for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 27, 2024
…pythonGH-122045)

(cherry picked from commit 4e04d1a)

Co-authored-by: Seth Michael Larson <seth@python.org>
@bedevere-app
Copy link

bedevere-app bot commented Jul 27, 2024

GH-122354 is a backport of this pull request to the 3.12 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 27, 2024
…pythonGH-122045)

(cherry picked from commit 4e04d1a)

Co-authored-by: Seth Michael Larson <seth@python.org>
@bedevere-app bedevere-app bot removed the needs backport to 3.12 bug and security fixes label Jul 27, 2024
@bedevere-app
Copy link

bedevere-app bot commented Jul 27, 2024

GH-122355 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Jul 27, 2024
hugovk pushed a commit that referenced this pull request Jul 27, 2024
GH-122045) (#122354)

Co-authored-by: Seth Michael Larson <seth@python.org>
hugovk pushed a commit that referenced this pull request Jul 27, 2024
GH-122045) (#122355)

Co-authored-by: Seth Michael Larson <seth@python.org>
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.

2 participants