Skip to content

fix: Patch commands fail trying to re-apply the same patch twice #664

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: Patch commands fail trying to re-apply the same patch twice #664

wants to merge 1 commit into from

Conversation

BohdanBuinich
Copy link

Fixes #618

@BohdanBuinich
Copy link
Author

@TheLartians, Could you please review this change and let me know if you agree with it?

@aaeberharter
Copy link

I pulled CPM.cmake from the PR and it works for me.
cmake version 3.31.6-msvc6 (bundled with Visual Studio 2022)

Thank you

@christopherbate christopherbate mentioned this pull request Jun 12, 2025
Closed
@aaeberharter
Copy link

PATCH_OPTIONS are mentioned in README.md but are not used in CPM.cmake. After this PR is merged we should discuss how to add meaningful patch options.

@BohdanBuinich BohdanBuinich force-pushed the feat/fix_patch_command branch from 5060d97 to 3c3ebf1 Compare June 16, 2025 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] Patch commands fail trying to re-apply the same patch twice
2 participants
@BohdanBuinich @aaeberharter