Skip to content

Allow glob targets#342

Open
cg-223 wants to merge 4 commits into
masterfrom
glob-targets
Open

Allow glob targets#342
cg-223 wants to merge 4 commits into
masterfrom
glob-targets

Conversation

@cg-223
Copy link
Copy Markdown
Collaborator

@cg-223 cg-223 commented Jan 24, 2026

Allows the use of "*" and "?" in targets.

Veeeery hacky, had to change a lot of stuff. I think I got it all working without any log spam.

I added a new optional property to patterns and regexes, silent, suppressing the warning for a patch not applying to its target. This defaults to false, except when the target is exactly *.

@cg-223
Copy link
Copy Markdown
Collaborator Author

cg-223 commented Jan 24, 2026

Dumps are an issue, using a "*" target causes everything to be dumped, as if the game was ran with --dump-all.

@cg-223
Copy link
Copy Markdown
Collaborator Author

cg-223 commented Jan 24, 2026

Worked around that by only dumping files that actually get modified by patches, this has the potential downside of not dumping files that have patches exclusively miss on them.

I don't see that being an issue as if no patches were applied at all to the file you don't need the dump (you can just look at the original file)

@Aurelius7309
Copy link
Copy Markdown

wtf, so many clopenings

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.

4 participants