unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#56331] [PATCH] gnu: Graft tzdata to install leap-seconds.list.
@ 2022-07-01  1:53 Antero Mejr via Guix-patches via
       [not found] ` <handler.56331.B.165664042219716.ack@debbugs.gnu.org>
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Antero Mejr via Guix-patches via @ 2022-07-01  1:53 UTC (permalink / raw)
  To: 56331

* gnu/packages/base.scm (tzdata)[replacement]: New field.
(tzdata-leap-seconds-list): New private variable.
---
 gnu/packages/base.scm | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/gnu/packages/base.scm b/gnu/packages/base.scm
index 4bdc3e7792..b265b429d5 100644
--- a/gnu/packages/base.scm
+++ b/gnu/packages/base.scm
@@ -1305,6 +1305,7 @@ (define-public tzdata
     ;; This package should be kept in sync with python-pytz in (gnu packages
     ;; time).
     (version "2022a")
+    (replacement tzdata-leap-seconds-list)
     (source (origin
              (method url-fetch)
              (uri (string-append
@@ -1390,6 +1391,19 @@ (define-public tzdata
 and daylight-saving rules.")
     (license public-domain)))
 
+(define tzdata-leap-seconds-list
+  (package
+    (inherit tzdata)
+    (arguments
+     (substitute-keyword-arguments (package-arguments tzdata)
+       ((#:phases phases)
+        #~(modify-phases #$phases
+            (add-after 'install 'copy-leap-seconds-list
+              (lambda _
+                (copy-file "leap-seconds.list"
+                           (string-append
+                            #$output "/share/zoneinfo/leap-seconds.list"))))))))))
+
 ;;; A "fixed" version of tzdata, which is used in the test suites of glib and R
 ;;; and a few other places. We can update this whenever we are able to rebuild
 ;;; thousands of packages (for example, in a core-updates rebuild). This package
-- 
2.36.1





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

* [bug#56331] Acknowledgement ([PATCH] gnu: Graft tzdata to install leap-seconds.list.)
       [not found] ` <handler.56331.B.165664042219716.ack@debbugs.gnu.org>
@ 2022-07-01  2:08   ` Antero Mejr via Guix-patches via
  0 siblings, 0 replies; 5+ messages in thread
From: Antero Mejr via Guix-patches via @ 2022-07-01  2:08 UTC (permalink / raw)
  To: 56331

leap-seconds.list is required by the hare programming language (which is
currently being packaged) for accurate datetime calculations.

It is optionally used by ntpd with the "leapfile" directive in ntpd.conf.




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

* [bug#56331] [PATCH] gnu: Graft tzdata to install leap-seconds.list.
  2022-07-01  1:53 [bug#56331] [PATCH] gnu: Graft tzdata to install leap-seconds.list Antero Mejr via Guix-patches via
       [not found] ` <handler.56331.B.165664042219716.ack@debbugs.gnu.org>
@ 2022-07-01  9:43 ` ( via Guix-patches via
  2022-07-01 13:03 ` [bug#56331] [PATCH] gnu: Update tzdata-for-tests Antero Mejr via Guix-patches via
  2 siblings, 0 replies; 5+ messages in thread
From: ( via Guix-patches via @ 2022-07-01  9:43 UTC (permalink / raw)
  To: Antero Mejr, 56331

On Fri Jul 1, 2022 at 2:53 AM BST, Antero Mejr via Guix-patches via wrote:
> @@ -1305,6 +1305,7 @@ (define-public tzdata
>      ;; This package should be kept in sync with python-pytz in (gnu packages
>      ;; time).
>      (version "2022a")
> +    (replacement tzdata-leap-seconds-list)
>      (source (origin
>               (method url-fetch)
>               (uri (string-append
> @@ -1390,6 +1391,19 @@ (define-public tzdata
>  and daylight-saving rules.")
>      (license public-domain)))

There's no need to graft it, as long as you make sure tzdata-for-tests
isn't rebuilt.




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

* [bug#56331] [PATCH] gnu: Update tzdata-for-tests.
  2022-07-01  1:53 [bug#56331] [PATCH] gnu: Graft tzdata to install leap-seconds.list Antero Mejr via Guix-patches via
       [not found] ` <handler.56331.B.165664042219716.ack@debbugs.gnu.org>
  2022-07-01  9:43 ` [bug#56331] [PATCH] gnu: Graft tzdata to install leap-seconds.list ( via Guix-patches via
@ 2022-07-01 13:03 ` Antero Mejr via Guix-patches via
  2022-07-01 19:49   ` Liliana Marie Prikler
  2 siblings, 1 reply; 5+ messages in thread
From: Antero Mejr via Guix-patches via @ 2022-07-01 13:03 UTC (permalink / raw)
  To: 56331; +Cc: paren

* gnu/packages/base.scm (tzdata-for-tests): Install leap-seconds.list.
---
 gnu/packages/base.scm | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/base.scm b/gnu/packages/base.scm
index 4bdc3e7792..a1d57611ee 100644
--- a/gnu/packages/base.scm
+++ b/gnu/packages/base.scm
@@ -1398,7 +1398,20 @@ (define-public tzdata
 ;;;
 ;;; Please make this a hidden-package if it is different from the primary tzdata
 ;;; package.
-(define-public tzdata-for-tests tzdata)
+(define-public tzdata-for-tests
+  (hidden-package
+   (package
+     (inherit tzdata)
+     (arguments
+      (substitute-keyword-arguments (package-arguments tzdata)
+        ((#:phases phases)
+         #~(modify-phases #$phases
+             (add-after 'install 'copy-leap-seconds-list
+               (lambda _
+                 (copy-file "leap-seconds.list"
+                            (string-append
+                             #$output
+                             "/share/zoneinfo/leap-seconds.list")))))))))))
 
 (define-public libiconv
   (package
-- 
2.36.1





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

* [bug#56331] [PATCH] gnu: Update tzdata-for-tests.
  2022-07-01 13:03 ` [bug#56331] [PATCH] gnu: Update tzdata-for-tests Antero Mejr via Guix-patches via
@ 2022-07-01 19:49   ` Liliana Marie Prikler
  0 siblings, 0 replies; 5+ messages in thread
From: Liliana Marie Prikler @ 2022-07-01 19:49 UTC (permalink / raw)
  To: Antero Mejr, 56331; +Cc: paren

Am Freitag, dem 01.07.2022 um 13:03 +0000 schrieb Antero Mejr:
> * gnu/packages/base.scm (tzdata-for-tests): Install leap-
> seconds.list.
> ---
>  gnu/packages/base.scm | 15 ++++++++++++++-
>  1 file changed, 14 insertions(+), 1 deletion(-)
> 
> diff --git a/gnu/packages/base.scm b/gnu/packages/base.scm
> index 4bdc3e7792..a1d57611ee 100644
> --- a/gnu/packages/base.scm
> +++ b/gnu/packages/base.scm
> @@ -1398,7 +1398,20 @@ (define-public tzdata
>  ;;;
>  ;;; Please make this a hidden-package if it is different from the
> primary tzdata
>  ;;; package.
> -(define-public tzdata-for-tests tzdata)
> +(define-public tzdata-for-tests
> +  (hidden-package
> +   (package
> +     (inherit tzdata)
> +     (arguments
> +      (substitute-keyword-arguments (package-arguments tzdata)
> +        ((#:phases phases)
> +         #~(modify-phases #$phases
> +             (add-after 'install 'copy-leap-seconds-list
> +               (lambda _
> +                 (copy-file "leap-seconds.list"
> +                            (string-append
> +                             #$output
> +                             "/share/zoneinfo/leap-
> seconds.list")))))))))))
That's exactly the patch you weren't supposed to write.  It bumps
tzdata-for-tests, thus causing mass rebuilds, but doesn't bump tzdata.

Cheers




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

end of thread, other threads:[~2022-07-01 19:50 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-07-01  1:53 [bug#56331] [PATCH] gnu: Graft tzdata to install leap-seconds.list Antero Mejr via Guix-patches via
     [not found] ` <handler.56331.B.165664042219716.ack@debbugs.gnu.org>
2022-07-01  2:08   ` [bug#56331] Acknowledgement ([PATCH] gnu: Graft tzdata to install leap-seconds.list.) Antero Mejr via Guix-patches via
2022-07-01  9:43 ` [bug#56331] [PATCH] gnu: Graft tzdata to install leap-seconds.list ( via Guix-patches via
2022-07-01 13:03 ` [bug#56331] [PATCH] gnu: Update tzdata-for-tests Antero Mejr via Guix-patches via
2022-07-01 19:49   ` Liliana Marie Prikler

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).