Skip to content

docs/cmdline-opts: document the dotless config path #10849

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

Closed
wants to merge 1 commit into from

Conversation

rpigott
Copy link
Contributor

@rpigott rpigott commented Mar 28, 2023

The real xdg config path is $XDG_CONFIG_HOME/curlrc, without the dot. The dotless name seems preferable, so let's match the documentation to the behavior.


curl 8.0.1 doesn't seem to care about $XDG_CONFIG_HOME/.curlc but I was happy to find it does find $XDG_CONFIG_HOME/curlrc. Let's document that.

The real xdg config path is $XDG_CONFIG_HOME/curlrc, without the dot.
The dotless name seems preferable, so let's match the documentation to
the behavior.
Copy link
Member

@bagder bagder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeps, I believe that was just a typo. I don't think it ever used the dot-version in there.

@bagder bagder closed this in 8d9c1a8 Mar 28, 2023
@bagder
Copy link
Member

bagder commented Mar 28, 2023

Thanks!

@rpigott rpigott deleted the fix-curlrc-doc branch March 28, 2023 17:46
bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
The real xdg config path is $XDG_CONFIG_HOME/curlrc, without the dot.
The dotless name seems preferable, so let's match the documentation to
the behavior.

Closes curl#10849
@keithbowes
Copy link

Yeps, I believe that was just a typo. I don't think it ever used the dot-version in there.

Actually, in my distro's supplied version, 7.88.1, $XDG_CONFIG_HOME/.curlrc works but $XDG_CONFIG_HOME/curlrc doesn't. It seems that it originally used the dot-version and then was fixed later (see #12129). Therefore, the man page is technically incorrect; in fact $XDG_CONFIG_HOME/curlrc hasn't been used since 7.73.0, but:

  • $XDG_CONFIG_HOME/.curlrc (7.73.0–7.81.0)
  • $XDG_CONFIG_HOME/.curclrc (if $XDG_CONFIG_HOME is set) or $XDG_CONFIG_HOME/curlrc (if $XDG_CONFIG_HOME is not set) (7.82.0–7.88.1)
  • $XDG_CONFIG_HOME/curlrc (8.0+)

@rpigott
Copy link
Contributor Author

rpigott commented Mar 25, 2025

I'm confused.

So curl changed the config path to include a dot contrary to the documentation? Shouldn't that just be reverted?

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