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

Update Python script instructions in README.md #879

Merged
merged 2 commits into from
Jan 12, 2023

Conversation

hkcountryman
Copy link
Contributor

Closes #847

I believe the WeeChat docs and default directories have changed since README.md was created.

@trygveaa trygveaa merged commit 6826508 into wee-slack:master Jan 12, 2023
@trygveaa
Copy link
Member

Thanks!

mkdir -p ~/.weechat/python/autoload
cd ~/.weechat/python
mkdir -p $XDG_DATA_HOME/weechat/python/autoload/
cd $XDG_DATA_HOME/weechat/python/
Copy link
Member

Choose a reason for hiding this comment

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

I think I'll change this to .local/share/weechat though, as I think most systems don't have $XDG_DATA_HOME set (and thus uses .local/share/weechat), and I want these commands to work verbatim for most people.

Could use a variable with a default value, but that's extra syntax not everyone will understand, plus it won't work in fish and other shells with different syntax from bash, so I don't think I'll do that.

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

Successfully merging this pull request may close these issues.

Autoload instructions should mention alternate path for XDG directories
2 participants