Repository navigation
Conversation
Added more details to help with cygwin install, adding links to curl source code where relevant to help save people a little time.
The cygwin install process I used for the instructions are:
1. Windows Sandox
2. Install winget
3. Install cygwin
4. Download curl source code:
- curl https://mirrors.kernel.org/sources.redhat.com/cygwin/x86_64/release/curl/curl-8.13.0-1-src.tar.xz -o src.tar.xz
- tar -xJf src.tar.xz && rm src.tar.xz
- move curl-8.13.0-1.src\curl-8.13.0.tar.xz curl.tar.xz
- tar -xJf curl.tar.xz && rm curl.tar.xz
- move curl-8.13.0 curl
- cd curl
- sh configure --without-ssl
- make
- ${test_commands}
5. Run curl ${test_commands}:
- src\curl --help
- src\curl http://example.com
- src\curl -I http://example.com
- src\curl -H "accept-language: en-US,en;q=0.9" http://example.com
- src\curl -L http://example.com
jhauga
marked this pull request as draft
May 30, 2025 00:01
Correct errors from spellcheck, badwords-synopsis, and mischecks.
Correct errors from spellcheck, badwords-synopsis, and mischecks.
Correct error from spellcheck.
jhauga
marked this pull request as ready for review
May 30, 2025 00:44
Member
But they link to different anchors on the download page, which seems mostly wrong. Most of the specific packages listed on the download page are not source code at all so several links are definitely wrong. I think it is better to direct users to the curl release tarballs at the top of the download page. Probably then completely without any fragment in the URL. |
vszakats
reviewed
May 30, 2025
jhauga
marked this pull request as draft
May 30, 2025 21:45
Commit suggestions. Co-authored-by: Viktor Szakats <vszakats@users.noreply.github.com>
Removed the redundant links to curl source code.
jhauga
marked this pull request as ready for review
May 30, 2025 23:12
vszakats
reviewed
May 31, 2025
vszakats
reviewed
May 31, 2025
jhauga
marked this pull request as draft
May 31, 2025 05:41
Implement suggestions. Co-authored-by: Viktor Szakats <vszakats@users.noreply.github.com>
Tested build without suggested packages. It worked.
jhauga
marked this pull request as ready for review
May 31, 2025 06:28
Change cygwin setup-x86_64 syntax that provides better search search results.
Member
|
@vszakats is there a 👍 from you on this? |
vszakats
approved these changes
Jun 5, 2025
Member
Yes, looking good! |
Member
|
Thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added more details to help with cygwin install, adding links to curl source code (where relevant) to help save people a little time.
The cygwin install process I used for the instructions are:
Windows Sandox
Install winget
Install cygwin
Download curl source code:
curl https://mirrors.kernel.org/sources.redhat.com/cygwin/x86_64/release/curl/curl-8.13.0-1-src.tar.xz -o src.tar.xztar -xJf src.tar.xz && rm src.tar.xzmove curl-8.13.0-1.src\curl-8.13.0.tar.xz curl.tar.xztar -xJf curl.tar.xz && rm curl.tar.xzmove curl-8.13.0 curlcd curlsh configure --without-sslmake - ${test_commands}Run curl ${test_commands}:
-
src\curl --help-
src\curl http://example.com-
src\curl -I http://example.com-
src\curl -H "accept-language: en-US,en;q=0.9" http://example.com-
src\curl -L http://example.com