Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] chrony: fix wrong option in the default config.
@ 2019-10-30  7:24 voidlinux-github
  2019-10-30  7:40 ` [PR PATCH] [Updated] " voidlinux-github
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: voidlinux-github @ 2019-10-30  7:24 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1117 bytes --]

There is a new pull request by travankor against master on the void-packages repository

https://github.com/travankor/void-packages chrony
https://github.com/void-linux/void-packages/pull/15936

chrony: fix wrong option in the default config.
The current option rtcfile in the default config conflicts with runit's stage 3 shutdown script.
The stage 3 script calls hwclock -w, while chronyd in RTC mode wants to take control of /dev/rtc to measure RTC drift.
Furthermore the default chronyd runit service does not take advantage of the Linux RTC functionality with the -s option.
See for more info: https://chrony.tuxfamily.org/faq.html#_i_want_to_use_code_chronyd_code_s_rtc_support_must_i_disable_code_hwclock_code

To maintain compatibility with the stage 3 shutdown, switch rtcfile to the rtcsync option.
Example config file #1 already sets the correct options that work out of the box so use that example file instead of editing the annotated one.
Example config file #3 will instead be installed to /usr/share/examples.

A patch file from https://github.com/void-linux/void-packages/pull/15936.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-chrony-15936.patch --]
[-- Type: text/x-diff, Size: 2165 bytes --]

From 78183aeb9f3e87392b83dd7c83b995e733289606 Mon Sep 17 00:00:00 2001
From: travankor <travankor@tuta.io>
Date: Fri, 18 Oct 2019 23:46:32 -0700
Subject: [PATCH] chrony: fix wrong option in the default config.

The current option rtcfile in the default config conflicts with runit's stage 3 shutdown script.
The stage 3 script calls hwclock -w, while chronyd in RTC mode wants to take control of /dev/rtc to measure RTC drift.
Furthermore the default chronyd runit service does not take advantage of the Linux RTC functionality with the -s option.
See for more info: https://chrony.tuxfamily.org/faq.html#_i_want_to_use_code_chronyd_code_s_rtc_support_must_i_disable_code_hwclock_code

To maintain compatibility with the stage 3 shutdown, switch rtcfile to the rtcsync option.
Example config file #1 already sets the correct options that work out of the box so use that example file instead of editing the annotated one.
Example config file #3 will instead be installed to /usr/share/examples.
---
 srcpkgs/chrony/template | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/srcpkgs/chrony/template b/srcpkgs/chrony/template
index 6d629a9260c..75142b66101 100644
--- a/srcpkgs/chrony/template
+++ b/srcpkgs/chrony/template
@@ -2,7 +2,7 @@
 # When Updating: Please confirm the upstream config still refers to make_dirs
 pkgname=chrony
 version=3.5
-revision=1
+revision=2
 build_style=gnu-configure
 configure_args="--without-nss --enable-scfilter
  --with-sendmail=/usr/bin/sendmail"
@@ -29,12 +29,8 @@ alternatives="
  ntpd:ntpd:/etc/sv/chronyd"
 
 post_install() {
-	vconf examples/chrony.conf.example3 chrony.conf
-	# Modify default configuration so it works out-of-the-box
-	vsed -e 's,! pool pool.ntp.org iburst,pool pool.ntp.org iburst,' -i ${DESTDIR}/etc/chrony.conf
-	vsed -e 's,! rtcfile /var/lib/chrony/rtc,rtcfile /var/lib/chrony/rtc,' -i ${DESTDIR}/etc/chrony.conf
-	vsed -e 's,! makestep 1.0 3,makestep 1.0 3,' -i ${DESTDIR}/etc/chrony.conf
-	vsconf examples/chrony.conf.example1
+	vconf examples/chrony.conf.example1 chrony.conf
 	vsconf examples/chrony.conf.example2
+	vsconf examples/chrony.conf.example3
 	vsv chronyd
 }

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PR PATCH] [Updated] chrony: fix wrong option in the default config.
  2019-10-30  7:24 [PR PATCH] chrony: fix wrong option in the default config voidlinux-github
  2019-10-30  7:40 ` [PR PATCH] [Updated] " voidlinux-github
@ 2019-10-30  7:40 ` voidlinux-github
  2019-10-30  7:43 ` voidlinux-github
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: voidlinux-github @ 2019-10-30  7:40 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1122 bytes --]

There is an updated pull request by travankor against master on the void-packages repository

https://github.com/travankor/void-packages chrony
https://github.com/void-linux/void-packages/pull/15936

chrony: fix wrong option in the default config.
The current option rtcfile in the default config conflicts with runit's stage 3 shutdown script.
The stage 3 script calls hwclock -w, while chronyd in RTC mode wants to take control of /dev/rtc to measure RTC drift.
Furthermore the default chronyd runit service does not take advantage of the Linux RTC functionality with the -s option.
See for more info: https://chrony.tuxfamily.org/faq.html#_i_want_to_use_code_chronyd_code_s_rtc_support_must_i_disable_code_hwclock_code

To maintain compatibility with the stage 3 shutdown, switch rtcfile to the rtcsync option.
Example config file #1 already sets the correct options that work out of the box so use that example file instead of editing the annotated one.
Example config file #3 will instead be installed to /usr/share/examples.

A patch file from https://github.com/void-linux/void-packages/pull/15936.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-chrony-15936.patch --]
[-- Type: text/x-diff, Size: 2112 bytes --]

From 63e717daab349bb55ee343b422fa49e9f55f1b4e Mon Sep 17 00:00:00 2001
From: travankor <travankor@tuta.io>
Date: Fri, 18 Oct 2019 23:46:32 -0700
Subject: [PATCH] chrony: fix wrong option in the default config.

The option rtcfile in the default config conflicts with runit's stage 3.
Chronyd in RTC mode takes control of /dev/rtc to measure RTC drift.
Stage 3 calls hwclock -w, which messes up chrony's drift calculation.
Furthermore the current chronyd runit service  needs the -s option,
to effectively use the RTC feature.
See for more info:
https://chrony.tuxfamily.org/faq.html#_i_want_to_use_code_chronyd_code_s_rtc_support_must_i_disable_code_hwclock_code

Switch rtcfile to the rtcsync option, to be consistent with stage 3.
Example config file #1 sets the right options that work out of the box.
So use that example file instead of editing the annotated one.
Example config file #3 will instead be installed to /usr/share/examples.
---
 srcpkgs/chrony/template | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/srcpkgs/chrony/template b/srcpkgs/chrony/template
index 6d629a9260c..75142b66101 100644
--- a/srcpkgs/chrony/template
+++ b/srcpkgs/chrony/template
@@ -2,7 +2,7 @@
 # When Updating: Please confirm the upstream config still refers to make_dirs
 pkgname=chrony
 version=3.5
-revision=1
+revision=2
 build_style=gnu-configure
 configure_args="--without-nss --enable-scfilter
  --with-sendmail=/usr/bin/sendmail"
@@ -29,12 +29,8 @@ alternatives="
  ntpd:ntpd:/etc/sv/chronyd"
 
 post_install() {
-	vconf examples/chrony.conf.example3 chrony.conf
-	# Modify default configuration so it works out-of-the-box
-	vsed -e 's,! pool pool.ntp.org iburst,pool pool.ntp.org iburst,' -i ${DESTDIR}/etc/chrony.conf
-	vsed -e 's,! rtcfile /var/lib/chrony/rtc,rtcfile /var/lib/chrony/rtc,' -i ${DESTDIR}/etc/chrony.conf
-	vsed -e 's,! makestep 1.0 3,makestep 1.0 3,' -i ${DESTDIR}/etc/chrony.conf
-	vsconf examples/chrony.conf.example1
+	vconf examples/chrony.conf.example1 chrony.conf
 	vsconf examples/chrony.conf.example2
+	vsconf examples/chrony.conf.example3
 	vsv chronyd
 }

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PR PATCH] [Updated] chrony: fix wrong option in the default config.
  2019-10-30  7:24 [PR PATCH] chrony: fix wrong option in the default config voidlinux-github
@ 2019-10-30  7:40 ` voidlinux-github
  2019-10-30  7:40 ` voidlinux-github
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: voidlinux-github @ 2019-10-30  7:40 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1122 bytes --]

There is an updated pull request by travankor against master on the void-packages repository

https://github.com/travankor/void-packages chrony
https://github.com/void-linux/void-packages/pull/15936

chrony: fix wrong option in the default config.
The current option rtcfile in the default config conflicts with runit's stage 3 shutdown script.
The stage 3 script calls hwclock -w, while chronyd in RTC mode wants to take control of /dev/rtc to measure RTC drift.
Furthermore the default chronyd runit service does not take advantage of the Linux RTC functionality with the -s option.
See for more info: https://chrony.tuxfamily.org/faq.html#_i_want_to_use_code_chronyd_code_s_rtc_support_must_i_disable_code_hwclock_code

To maintain compatibility with the stage 3 shutdown, switch rtcfile to the rtcsync option.
Example config file #1 already sets the correct options that work out of the box so use that example file instead of editing the annotated one.
Example config file #3 will instead be installed to /usr/share/examples.

A patch file from https://github.com/void-linux/void-packages/pull/15936.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-chrony-15936.patch --]
[-- Type: text/x-diff, Size: 2112 bytes --]

From 63e717daab349bb55ee343b422fa49e9f55f1b4e Mon Sep 17 00:00:00 2001
From: travankor <travankor@tuta.io>
Date: Fri, 18 Oct 2019 23:46:32 -0700
Subject: [PATCH] chrony: fix wrong option in the default config.

The option rtcfile in the default config conflicts with runit's stage 3.
Chronyd in RTC mode takes control of /dev/rtc to measure RTC drift.
Stage 3 calls hwclock -w, which messes up chrony's drift calculation.
Furthermore the current chronyd runit service  needs the -s option,
to effectively use the RTC feature.
See for more info:
https://chrony.tuxfamily.org/faq.html#_i_want_to_use_code_chronyd_code_s_rtc_support_must_i_disable_code_hwclock_code

Switch rtcfile to the rtcsync option, to be consistent with stage 3.
Example config file #1 sets the right options that work out of the box.
So use that example file instead of editing the annotated one.
Example config file #3 will instead be installed to /usr/share/examples.
---
 srcpkgs/chrony/template | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/srcpkgs/chrony/template b/srcpkgs/chrony/template
index 6d629a9260c..75142b66101 100644
--- a/srcpkgs/chrony/template
+++ b/srcpkgs/chrony/template
@@ -2,7 +2,7 @@
 # When Updating: Please confirm the upstream config still refers to make_dirs
 pkgname=chrony
 version=3.5
-revision=1
+revision=2
 build_style=gnu-configure
 configure_args="--without-nss --enable-scfilter
  --with-sendmail=/usr/bin/sendmail"
@@ -29,12 +29,8 @@ alternatives="
  ntpd:ntpd:/etc/sv/chronyd"
 
 post_install() {
-	vconf examples/chrony.conf.example3 chrony.conf
-	# Modify default configuration so it works out-of-the-box
-	vsed -e 's,! pool pool.ntp.org iburst,pool pool.ntp.org iburst,' -i ${DESTDIR}/etc/chrony.conf
-	vsed -e 's,! rtcfile /var/lib/chrony/rtc,rtcfile /var/lib/chrony/rtc,' -i ${DESTDIR}/etc/chrony.conf
-	vsed -e 's,! makestep 1.0 3,makestep 1.0 3,' -i ${DESTDIR}/etc/chrony.conf
-	vsconf examples/chrony.conf.example1
+	vconf examples/chrony.conf.example1 chrony.conf
 	vsconf examples/chrony.conf.example2
+	vsconf examples/chrony.conf.example3
 	vsv chronyd
 }

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PR PATCH] [Updated] chrony: fix wrong option in the default config.
  2019-10-30  7:24 [PR PATCH] chrony: fix wrong option in the default config voidlinux-github
                   ` (2 preceding siblings ...)
  2019-10-30  7:43 ` voidlinux-github
@ 2019-10-30  7:43 ` voidlinux-github
  2019-10-30 14:56 ` [PR PATCH] [Merged]: " voidlinux-github
  4 siblings, 0 replies; 6+ messages in thread
From: voidlinux-github @ 2019-10-30  7:43 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1122 bytes --]

There is an updated pull request by travankor against master on the void-packages repository

https://github.com/travankor/void-packages chrony
https://github.com/void-linux/void-packages/pull/15936

chrony: fix wrong option in the default config.
The current option rtcfile in the default config conflicts with runit's stage 3 shutdown script.
The stage 3 script calls hwclock -w, while chronyd in RTC mode wants to take control of /dev/rtc to measure RTC drift.
Furthermore the default chronyd runit service does not take advantage of the Linux RTC functionality with the -s option.
See for more info: https://chrony.tuxfamily.org/faq.html#_i_want_to_use_code_chronyd_code_s_rtc_support_must_i_disable_code_hwclock_code

To maintain compatibility with the stage 3 shutdown, switch rtcfile to the rtcsync option.
Example config file #1 already sets the correct options that work out of the box so use that example file instead of editing the annotated one.
Example config file #3 will instead be installed to /usr/share/examples.

A patch file from https://github.com/void-linux/void-packages/pull/15936.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-chrony-15936.patch --]
[-- Type: text/x-diff, Size: 2109 bytes --]

From 82c5e29f7a824107a8ac7937db22877d0c208d40 Mon Sep 17 00:00:00 2001
From: travankor <travankor@tuta.io>
Date: Fri, 18 Oct 2019 23:46:32 -0700
Subject: [PATCH] chrony: fix wrong option in the default config.

The option rtcfile in the default config conflicts with runit's stage 3.
Chronyd in RTC mode takes control of /dev/rtc to measure RTC drift.
Stage 3 calls hwclock -w, which messes up chrony's drift calculation.
Furthermore the current chronyd runit service needs the -s option
to effectively use the RTC feature.

See for more info:
https://chrony.tuxfamily.org/faq.html#_i_want_to_use_code_chronyd_code_s_rtc_support_must_i_disable_code_hwclock_code

Switch rtcfile to the rtcsync option, to be consistent with stage 3.
Example config file 1 sets the right options that work out of the box.
So use that example file instead of editing the annotated one.
Example config file 3 will instead be installed to /usr/share/examples.
---
 srcpkgs/chrony/template | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/srcpkgs/chrony/template b/srcpkgs/chrony/template
index 6d629a9260c..75142b66101 100644
--- a/srcpkgs/chrony/template
+++ b/srcpkgs/chrony/template
@@ -2,7 +2,7 @@
 # When Updating: Please confirm the upstream config still refers to make_dirs
 pkgname=chrony
 version=3.5
-revision=1
+revision=2
 build_style=gnu-configure
 configure_args="--without-nss --enable-scfilter
  --with-sendmail=/usr/bin/sendmail"
@@ -29,12 +29,8 @@ alternatives="
  ntpd:ntpd:/etc/sv/chronyd"
 
 post_install() {
-	vconf examples/chrony.conf.example3 chrony.conf
-	# Modify default configuration so it works out-of-the-box
-	vsed -e 's,! pool pool.ntp.org iburst,pool pool.ntp.org iburst,' -i ${DESTDIR}/etc/chrony.conf
-	vsed -e 's,! rtcfile /var/lib/chrony/rtc,rtcfile /var/lib/chrony/rtc,' -i ${DESTDIR}/etc/chrony.conf
-	vsed -e 's,! makestep 1.0 3,makestep 1.0 3,' -i ${DESTDIR}/etc/chrony.conf
-	vsconf examples/chrony.conf.example1
+	vconf examples/chrony.conf.example1 chrony.conf
 	vsconf examples/chrony.conf.example2
+	vsconf examples/chrony.conf.example3
 	vsv chronyd
 }

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PR PATCH] [Updated] chrony: fix wrong option in the default config.
  2019-10-30  7:24 [PR PATCH] chrony: fix wrong option in the default config voidlinux-github
  2019-10-30  7:40 ` [PR PATCH] [Updated] " voidlinux-github
  2019-10-30  7:40 ` voidlinux-github
@ 2019-10-30  7:43 ` voidlinux-github
  2019-10-30  7:43 ` voidlinux-github
  2019-10-30 14:56 ` [PR PATCH] [Merged]: " voidlinux-github
  4 siblings, 0 replies; 6+ messages in thread
From: voidlinux-github @ 2019-10-30  7:43 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1122 bytes --]

There is an updated pull request by travankor against master on the void-packages repository

https://github.com/travankor/void-packages chrony
https://github.com/void-linux/void-packages/pull/15936

chrony: fix wrong option in the default config.
The current option rtcfile in the default config conflicts with runit's stage 3 shutdown script.
The stage 3 script calls hwclock -w, while chronyd in RTC mode wants to take control of /dev/rtc to measure RTC drift.
Furthermore the default chronyd runit service does not take advantage of the Linux RTC functionality with the -s option.
See for more info: https://chrony.tuxfamily.org/faq.html#_i_want_to_use_code_chronyd_code_s_rtc_support_must_i_disable_code_hwclock_code

To maintain compatibility with the stage 3 shutdown, switch rtcfile to the rtcsync option.
Example config file #1 already sets the correct options that work out of the box so use that example file instead of editing the annotated one.
Example config file #3 will instead be installed to /usr/share/examples.

A patch file from https://github.com/void-linux/void-packages/pull/15936.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-chrony-15936.patch --]
[-- Type: text/x-diff, Size: 2109 bytes --]

From 82c5e29f7a824107a8ac7937db22877d0c208d40 Mon Sep 17 00:00:00 2001
From: travankor <travankor@tuta.io>
Date: Fri, 18 Oct 2019 23:46:32 -0700
Subject: [PATCH] chrony: fix wrong option in the default config.

The option rtcfile in the default config conflicts with runit's stage 3.
Chronyd in RTC mode takes control of /dev/rtc to measure RTC drift.
Stage 3 calls hwclock -w, which messes up chrony's drift calculation.
Furthermore the current chronyd runit service needs the -s option
to effectively use the RTC feature.

See for more info:
https://chrony.tuxfamily.org/faq.html#_i_want_to_use_code_chronyd_code_s_rtc_support_must_i_disable_code_hwclock_code

Switch rtcfile to the rtcsync option, to be consistent with stage 3.
Example config file 1 sets the right options that work out of the box.
So use that example file instead of editing the annotated one.
Example config file 3 will instead be installed to /usr/share/examples.
---
 srcpkgs/chrony/template | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/srcpkgs/chrony/template b/srcpkgs/chrony/template
index 6d629a9260c..75142b66101 100644
--- a/srcpkgs/chrony/template
+++ b/srcpkgs/chrony/template
@@ -2,7 +2,7 @@
 # When Updating: Please confirm the upstream config still refers to make_dirs
 pkgname=chrony
 version=3.5
-revision=1
+revision=2
 build_style=gnu-configure
 configure_args="--without-nss --enable-scfilter
  --with-sendmail=/usr/bin/sendmail"
@@ -29,12 +29,8 @@ alternatives="
  ntpd:ntpd:/etc/sv/chronyd"
 
 post_install() {
-	vconf examples/chrony.conf.example3 chrony.conf
-	# Modify default configuration so it works out-of-the-box
-	vsed -e 's,! pool pool.ntp.org iburst,pool pool.ntp.org iburst,' -i ${DESTDIR}/etc/chrony.conf
-	vsed -e 's,! rtcfile /var/lib/chrony/rtc,rtcfile /var/lib/chrony/rtc,' -i ${DESTDIR}/etc/chrony.conf
-	vsed -e 's,! makestep 1.0 3,makestep 1.0 3,' -i ${DESTDIR}/etc/chrony.conf
-	vsconf examples/chrony.conf.example1
+	vconf examples/chrony.conf.example1 chrony.conf
 	vsconf examples/chrony.conf.example2
+	vsconf examples/chrony.conf.example3
 	vsv chronyd
 }

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PR PATCH] [Merged]: chrony: fix wrong option in the default config.
  2019-10-30  7:24 [PR PATCH] chrony: fix wrong option in the default config voidlinux-github
                   ` (3 preceding siblings ...)
  2019-10-30  7:43 ` voidlinux-github
@ 2019-10-30 14:56 ` voidlinux-github
  4 siblings, 0 replies; 6+ messages in thread
From: voidlinux-github @ 2019-10-30 14:56 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 919 bytes --]

There's a merged pull request on the void-packages repository

chrony: fix wrong option in the default config.
https://github.com/void-linux/void-packages/pull/15936

Description:
The option rtcfile in the default config conflicts with runit's stage 3.
Chronyd in RTC mode takes control of /dev/rtc to measure RTC drift.
Stage 3 calls hwclock -w, which messes up chrony's drift calculation.
Furthermore the current chronyd runit service needs the -s option
to effectively use the RTC feature.

See for more info:
https://chrony.tuxfamily.org/faq.html#_i_want_to_use_code_chronyd_code_s_rtc_support_must_i_disable_code_hwclock_code

Switch rtcfile to the rtcsync option, to be consistent with stage 3.
Example config file 1 sets the right options that work out of the box.
So use that example file instead of editing the annotated one.
Example config file 3 will instead be installed to /usr/share/examples.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2019-10-30 14:56 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-30  7:24 [PR PATCH] chrony: fix wrong option in the default config voidlinux-github
2019-10-30  7:40 ` [PR PATCH] [Updated] " voidlinux-github
2019-10-30  7:40 ` voidlinux-github
2019-10-30  7:43 ` voidlinux-github
2019-10-30  7:43 ` voidlinux-github
2019-10-30 14:56 ` [PR PATCH] [Merged]: " voidlinux-github

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).