Skip to content

cargo: exclude development scripts and fuzzing data#1332

Merged
BurntSushi merged 1 commit intomasterfrom
ag/exclude_unnecessary_stuff
Feb 3, 2026
Merged

cargo: exclude development scripts and fuzzing data#1332
BurntSushi merged 1 commit intomasterfrom
ag/exclude_unnecessary_stuff

Conversation

@BurntSushi
Copy link
Member

During a dependency review we noticed that different regex-* crate
includes various development scripts and sometimes fuzzing data. These
development scripts shouldn't be there as they might, at some point
become problematic. As of now they prevent any downstream user from
enabling the [bans.build.interpreted] option of cargo deny. The
fuzzing data are also not needed for anything.

I opted for using an explicit include list instead of an exclude list
to prevent these files from beeing included in the published packages
to make sure that everything that's included is an conscious choice.

Closes #1319

During a dependency review we noticed that different regex-* crate
includes various development scripts and sometimes fuzzing data. These
development scripts shouldn't be there as they might, at some point
become problematic. As of now they prevent any downstream user from
enabling the `[bans.build.interpreted]` option of cargo deny. The
fuzzing data are also not needed for anything.

I opted for using an explicit include list instead of an exclude list
to prevent these files from beeing included in the published packages
to make sure that everything that's included is an conscious choice.

Closes #1319
@BurntSushi BurntSushi force-pushed the ag/exclude_unnecessary_stuff branch from ae6e92a to 4b2b52a Compare February 3, 2026 13:23
@BurntSushi BurntSushi merged commit 6a81006 into master Feb 3, 2026
19 checks passed
@BurntSushi BurntSushi deleted the ag/exclude_unnecessary_stuff branch February 3, 2026 13:46
@BurntSushi
Copy link
Member Author

This PR is on crates.io in regex 1.12.3.

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