Skip to content
This repository has been archived by the owner on Apr 29, 2022. It is now read-only.

Use xdg-user-dirs as default paths #105

Closed
wants to merge 229 commits into from
Closed

Use xdg-user-dirs as default paths #105

wants to merge 229 commits into from

Conversation

qwIvan
Copy link

@qwIvan qwIvan commented Feb 22, 2017

No description provided.

@jomo
Copy link
Owner

jomo commented Feb 22, 2017

Didn't you just change the default configuration, which is, by definition, configurable anyways?

@jomo jomo closed this Feb 22, 2017
@qwIvan
Copy link
Author

qwIvan commented Feb 22, 2017

If I doesn't configuration:
/usr/bin/imgur-screenshot: line 504: cd: /home/ivan/Pictures: No such file or directory

~/Pictures is the default XDG_PICTURES_DIR is English, but in Chinese, the path is ~/图片.

See in https://www.freedesktop.org/wiki/Software/xdg-user-dirs/

@jomo jomo changed the title default pictures path maybe not in ~/Pictures, for instance in the Chinese language environment Use xdg-user-dirs as default paths Feb 22, 2017
@jomo
Copy link
Owner

jomo commented Feb 22, 2017

Sorry, I may have misunderstood this. Your PR title confused me.

@jomo jomo reopened this Feb 22, 2017
@derimagia
Copy link

Was looking if there was an issue for this. One thing for logs can you confirm there isn't a way to change this?

log_file="${HOME}/.imgur-screenshot.log"
looks hardcoded to me. XDG doesn't really have a standard for this but I think a env variable would be good to have.

@jomo
Copy link
Owner

jomo commented May 15, 2017

Not sure how many more comments to put around the config section 😛

@derimagia
Copy link

Wow. Was paying attention too much to this ticket and then saw credentials.conf and so I thought it couldn't be in that. Sigh, sorry for that...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet