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

install nits #1

Closed
f451-fbsd opened this issue Aug 26, 2022 · 1 comment
Closed

install nits #1

f451-fbsd opened this issue Aug 26, 2022 · 1 comment

Comments

@f451-fbsd
Copy link

Hi, thank you for creating this

On install, fakertc doesn't make the file it needs in order to work.

service fakertc start

Set RTC from: /var/db/fakertc: /usr/local/etc/rc.d/fakertc: WARNING: /var/db/fakertc is not readable

fix is:

touch /var/db/fakertc

service fakertc start

Set RTC from: /var/db/fakertc: Fri Aug 26 14:58:34 UTC 2022
.

[1] 14.0-CURRENT #1 main-n257229-e9a2e4d1d28: Wed Aug 10 16:00:33 B
ST 2022
[2] # git -C /usr/ports rev-list --first-parent --count main
592994

@ronaldklop
Copy link
Owner

Hi, thanks for your report.
It is a bit counter intuitive but actually stop should be run before start. On shutdown stop runs and saves the date. On boot start runs and it restores the date. So running start without stop does not make sense technically. But I understand that it is intuitive to start the tool after installation so I'm thinking about checking for a not existing file and let start save the date if the file does not exist. That seems more user friendly to me than a warning.

About your solution. When you "touch" the file it creates an empty file. Then start incidentally runs "date -u " (with the whitespace because the file is empty) and that just prints the current date. So it looks better but it is just a cosmetic difference.

I already created a patch but will need to test a little bit more.

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

No branches or pull requests

2 participants