Skip to content

Commit

Permalink
fix: files included twice when they match include_spec (#857)
Browse files Browse the repository at this point in the history
Currently if a file matches the explicit `sdist.include` it is included
twice in the source distribution.

This causes an error with some package managers such as `uv` (See
abetlen/llama-cpp-python#1670 (comment)).
  • Loading branch information
abetlen authored Aug 13, 2024
1 parent 8fe9acb commit 22ffdb9
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/scikit_build_core/build/_file_processor.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,7 @@ def each_unignored_file(
# Always include something included
if include_spec.match_file(p):
yield p
continue

# Ignore from global ignore
if exclude_spec.match_file(p):
Expand Down

0 comments on commit 22ffdb9

Please sign in to comment.