openhab / openhabian

openHABian - empowering the smart home, for Raspberry Pi and Debian systems
https://community.openhab.org/t/13379
ISC License
818 stars 251 forks source link

Remove NTP Setup from openHABian setup as it is now redundant #1775

Closed BigGeorgeTx closed 1 year ago

BigGeorgeTx commented 1 year ago

Some users have reported that NTP fails on the Pi installations due to systemd-timesyncd excess start attempts.

It appears that standard Raspberry Pi OS now includes NTP by default and the NTP setup in openHABian set up results in a second daemon, that cannot run because there is already one running.

This PR removes the call from openhabian-setup.sh and the routine from functions/system.bash.

mstormi commented 1 year ago

Could you please fix what I mentioned above so we can merge this in time for OH4 ? Thanks

BigGeorgeTx commented 1 year ago

What is the DCO? (I submitted the other pull request you requested).

On Mon, May 15, 2023 at 10:02 AM Markus Storm @.***> wrote:

@.**** requested changes on this pull request.

Could you please also remove install & uninstall options from menu.bash ?

If possible, also please fix the DCO. Thank you.

— Reply to this email directly, view it on GitHub https://github.com/openhab/openhabian/pull/1775#pullrequestreview-1426891137, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADEQZL425FY7X7QRONJVBDDXGJHQJANCNFSM6AAAAAAYATFFN4 . You are receiving this because you authored the thread.Message ID: @.***>

-- George Hyde Mobile +1-281-620-3039