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

Add support for snap application using config file #619

Merged
merged 2 commits into from
Sep 8, 2021
Merged

Add support for snap application using config file #619

merged 2 commits into from
Sep 8, 2021

Conversation

kallydev
Copy link
Contributor

@kallydev kallydev commented Sep 5, 2021

Resolved #617. It can now use config file under $HOME path via -c or --config parameter.

@dev4u
Copy link

dev4u commented Sep 5, 2021

#617 读取当前目录文件,应该自觉加上./。不加./表示从系统设置的搜索目录读取。

@kallydev kallydev marked this pull request as ready for review September 5, 2021 08:01
@kallydev
Copy link
Contributor Author

kallydev commented Sep 5, 2021

This PR is theoretically correct, but I don't have a build environment available to test it right now. If anyone is interested in this, here are the steps needed to test it. I recommend using Ubuntu amd64 as the test environment, and CPU needs to support KVM extensions (macOS Big Sur's VMware Funsion and some public cloud servers don't support it).

git clone https://github.com/kallydev/shadowsocks-rust
cd shadowsocks-rust
git checkout patch
sudo apt install -y snapd
snap install snapcraft
snapcraft
shadowsocks-rust.ssserver -c ./config.json

@zonyitoo zonyitoo requested a review from DuckSoft September 5, 2021 14:52
@zonyitoo zonyitoo merged commit f717984 into shadowsocks:master Sep 8, 2021
atkdef pushed a commit to atkdef/shadowsocks-rust that referenced this pull request Sep 24, 2021
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.

No permission to read configuration files after installation with snap
4 participants