-
Notifications
You must be signed in to change notification settings - Fork 46
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.
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
clang: update to 18.1.6 #115
Conversation
MSYS2 renamed the directory. Move our files to a matching directory in preparation for the update to version 18.1.6 Signed-off-by: Matthias Aßhauer <[email protected]>
b1155ef
to
9bc59bd
Compare
Needs git-for-windows/gfw-helper-github-app#81 before we can |
/deploy The workflow run was started. |
We need to be careful not to replace any trailing space and backslash when replacing the value of LLVM_TARGETS_TO_BUILD. MSYS2 also renamed the directory, so pkgname needs to be adapted slightly. Signed-off-by: Matthias Aßhauer <[email protected]>
Signed-off-by: Matthias Aßhauer <[email protected]>
9bc59bd
to
062f600
Compare
@@ -49,7 +49,8 @@ mv upstream/$pkgname/README-patches.md ./ || die "$0: failed to replace existing | |||
|
|||
sed -e "s/pkgrel=[.0-9]\+\(.*\)/pkgrel=$new_pkgrel\1/" \ | |||
-e 's/-DCMAKE_BUILD_TYPE=Release/-DCMAKE_BUILD_TYPE=MinSizeRel/' \ | |||
-e 's/-DLLVM_TARGETS_TO_BUILD=[^)]*/-DLLVM_TARGETS_TO_BUILD=Native/' \ | |||
-e 's/-DLLVM_TARGETS_TO_BUILD=[^)\\]*/-DLLVM_TARGETS_TO_BUILD=Native/' \ | |||
-e 's/-DLLVM_TARGETS_TO_BUILD=Native\\/-DLLVM_TARGETS_TO_BUILD=Native \\/' \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've thought about [^)\\ ]
instead of removing and reintroducing a potential space, but decided against it to not have this break again due to a random space in a future target list.
/deploy The workflow run was started. |
This fixes git-for-windows/git#4874