Skip to content

tool_getparam: add support for --longopt=value#17789

Closed
bagder wants to merge 2 commits intomasterfrom
bagder/curl-options-equals
Closed

tool_getparam: add support for --longopt=value#17789
bagder wants to merge 2 commits intomasterfrom
bagder/curl-options-equals

Conversation

@bagder
Copy link
Copy Markdown
Member

@bagder bagder commented Jul 1, 2025

If the long option name ends with an equals sign (=), the argument is the text following on its right side.

This makes the command line parser accept this common style in addition to the existing way to accept option arguments more similar to how other command line tools do.

Example: curl --user-agent=curl-2000 https://example.com/

Change a few existing tests to use this syntax: 206, 1333, 1335, 1442

@bagder bagder added cmdline tool feature-window A merge of this requires an open feature window labels Jul 1, 2025
@github-actions github-actions bot added the tests label Jul 1, 2025
@bagder bagder marked this pull request as ready for review July 8, 2025 10:49
@bagder bagder force-pushed the bagder/curl-options-equals branch from 5d9b89d to 6cd2249 Compare July 25, 2025 22:09
bagder added 2 commits July 27, 2025 17:38
If the long option name ends with an equals sign (`=`), the argument is
the text following on its right side.

This makes the command line parser accept this common style in addition
to the existing way to accept option arguments more similar to how other
command line tools do.

Example: `curl --user-agent=curl-2000 https://example.com/`

Change a few existing tests to use this syntax: 206, 1333, 1335, 1442
@bagder bagder force-pushed the bagder/curl-options-equals branch from 6cd2249 to 8d45c1b Compare July 27, 2025 15:39
@bagder bagder closed this in 9f9fecc Jul 27, 2025
@bagder bagder deleted the bagder/curl-options-equals branch July 27, 2025 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cmdline tool feature-window A merge of this requires an open feature window tests

Development

Successfully merging this pull request may close these issues.

1 participant