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

Revert #579 #604

Merged
merged 2 commits into from
Mar 11, 2023
Merged

Revert #579 #604

merged 2 commits into from
Mar 11, 2023

Conversation

WaterLemons2k
Copy link
Contributor

@WaterLemons2k WaterLemons2k commented Mar 11, 2023

What does this PR do?

Motivation

In Windows, the %USERPROFILE% for normal user and SYSTEM user are different, revert #579 to fix this issue.

Additional Notes

Caused by #579.

In Windows, the home directories for normal user and SYSTEM user are different. Add `-c` option to fix this issue.
@jeessy2
Copy link
Owner

jeessy2 commented Mar 11, 2023

有些系统通过os.UserHomeDir()可能获取不到目录,考虑还原以前那个版本。获取得了就返回,获取不到就上级目录,也不提示获取失败

比如:群晖

This reverts commit 8c4974a.
@WaterLemons2k WaterLemons2k changed the title fix: different configFilePath for normal and SYSTEM users Revert #579 Mar 11, 2023
@WaterLemons2k
Copy link
Contributor Author

有些系统通过os.UserHomeDir()可能获取不到目录,考虑还原以前那个版本。获取得了就返回,获取不到就上级目录,也不提示获取失败

比如:群晖

群晖获取的是 /var/services/homes 吧。

@jeessy2
Copy link
Owner

jeessy2 commented Mar 11, 2023

具体没测,矿神套件那个连报三个获取失败。

@jeessy2 jeessy2 merged commit 51cce0d into jeessy2:master Mar 11, 2023
@WaterLemons2k WaterLemons2k deleted the configFilePath branch March 11, 2023 10:14
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.

安装服务后启动,会卡在“第一次运行将等待 20 秒后运行 (等待网络)”
2 participants