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

cirrus: use make LDFLAGS=-all-static instead of curl_LDFLAGS #9633

Closed
wants to merge 1 commit into from

Conversation

jay
Copy link
Member

@jay jay commented Oct 2, 2022

  • Correct the use of -all-static for static Windows CI builds.

curl_LDFLAGS was removed from the makefile when metalink support was removed. LDFLAGS=-all-static is passed to make only, because it is not a valid option for configure compilation tests.

Closes #xxxx

- Correct the use of -all-static for static Windows CI builds.

curl_LDFLAGS was removed from the makefile when metalink support was
removed. LDFLAGS=-all-static is passed to make only, because it is not a
valid option for configure compilation tests.

Closes #xxxx
@jay jay added the CI Continuous Integration label Oct 2, 2022
@jay jay requested a review from mback2k October 2, 2022 21:22
@jay jay closed this in 258ee73 Oct 5, 2022
@jay jay deleted the rm_curl_LDFLAGS branch October 5, 2022 07:51
jquepi pushed a commit to jquepi/curl.1.555 that referenced this pull request Oct 24, 2022
- Correct the use of -all-static for static Windows CI builds.

curl_LDFLAGS was removed from the makefile when metalink support was
removed. LDFLAGS=-all-static is passed to make only, because it is not a
valid option for configure compilation tests.

Closes curl/curl#9633
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration hacktoberfest-accepted
Development

Successfully merging this pull request may close these issues.

None yet

1 participant