Skip to content

[v640][CMake] Init git before patches to ensure correct paths of patched files#22439

Merged
guitargeek merged 2 commits into
root-project:v6-40-00-patchesfrom
guitargeek:bp_1
May 31, 2026
Merged

[v640][CMake] Init git before patches to ensure correct paths of patched files#22439
guitargeek merged 2 commits into
root-project:v6-40-00-patchesfrom
guitargeek:bp_1

Conversation

@guitargeek

Copy link
Copy Markdown
Contributor

This fixes issue where the build directory is somewhere underneath a git repository, which affects git apply behaviour.

Adapted from 836cf1a, the equivalent change made in the master branch.

@guitargeek guitargeek self-assigned this May 31, 2026
@guitargeek guitargeek requested a review from bellenot as a code owner May 31, 2026 13:24
@guitargeek guitargeek changed the title [CMake] Init git before patches to ensure correct paths of patched files [v640][CMake] Init git before patches to ensure correct paths of patched files May 31, 2026
@github-actions

github-actions Bot commented May 31, 2026

Copy link
Copy Markdown

Test Results

    22 files      22 suites   3d 13h 9m 0s ⏱️
 3 853 tests  3 852 ✅ 0 💤 1 ❌
77 007 runs  77 006 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 7e2a756.

♻️ This comment has been updated with latest results.

)

* [win] Fix `git apply` failures in incremental builds
* [cmake] Use marker files to detect if the patch has been applied

(cherry picked from commit bfe9a83)
@guitargeek guitargeek added the clean build Ask CI to do non-incremental build on PR label May 31, 2026
This fixes issue where the build directory is somewhere underneath a git
repository, which affects `git apply` behaviour.

This is commit 836cf1a from master, partially applied.
@guitargeek guitargeek requested a review from dpiparo as a code owner May 31, 2026 16:08
@guitargeek guitargeek merged commit 428633d into root-project:v6-40-00-patches May 31, 2026
31 of 33 checks passed
@guitargeek guitargeek deleted the bp_1 branch May 31, 2026 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clean build Ask CI to do non-incremental build on PR pr:backport

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants