Skip to content
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

cmake: drop redundant assigments #14924

Closed
wants to merge 1 commit into from

Conversation

vszakats
Copy link
Member

@vszakats vszakats commented Sep 15, 2024

No need to set them ON again.

No need to set them to `ON` again.
@dfandrich
Copy link
Contributor

Analysis of PR #14924 at 9ca6a3a8:

Test 3207 failed, which has NOT been flaky recently, so there could be a real issue in the PR.

Generated by Testclutch

@vszakats vszakats closed this in c4f781e Sep 19, 2024
@vszakats vszakats deleted the cm-redundant-sets branch September 19, 2024 11:04
moritzbuhl pushed a commit to moritzbuhl/curl that referenced this pull request Sep 20, 2024
No need to set them `ON` again.

Closes curl#14924
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants