GNU bug report logs - #60430
[PATCH core-updates] gnu: icu4c: Update packaging style.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 60430 in the body.
You can then email your comments to 60430 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: guix-patches <at> gnu.org
Cc: ngraves <at> ngraves.fr
Subject: [PATCH] gnu: icu4c: Update packaging style.
Date: Fri, 30 Dec 2022 17:09:00 +0100

* gnu/packages/icu4c.scm (icu4c): Update packaging style.
---
 gnu/packages/icu4c.scm | 66 ++++++++++++++++++++----------------------
 1 file changed, 32 insertions(+), 34 deletions(-)

diff --git a/gnu/packages/icu4c.scm b/gnu/packages/icu4c.scm
index 1e4f66d956..1338c8f525 100644
--- a/gnu/packages/icu4c.scm
+++ b/gnu/packages/icu4c.scm
@@ -63,41 +63,39 @@ (define-public icu4c
    (inputs
     (list perl))
    (arguments
-    `(#:configure-flags
-      (list
+    (list
+     #:configure-flags
+     `(list
        "--enable-rpath"
-        ,@(if (%current-target-system)
-              '((string-append "--with-cross-build="
-                                (assoc-ref %build-inputs "icu4c-build-root")))
-              '()))
-      #:phases
-      (modify-phases %standard-phases
-        (add-after 'unpack 'chdir-to-source
-          (lambda _ (chdir "source") #t))
-        (add-after 'chdir-to-source 'update-LDFLAGS
-          (lambda _
-            ;; Do not create a "data-only" libicudata.so because it causes
-            ;; problems on some architectures (notably armhf and MIPS).
-            (substitute* "config/mh-linux"
-              (("LDFLAGSICUDT=-nodefaultlibs -nostdlib")
-               "LDFLAGSICUDT="))
-            #t))
-        ,@(if (target-riscv64?)
-            `((add-after 'unpack 'disable-failing-test
-                ;; It is unknown why this test is failing.
-                (lambda _
-                  (substitute* "source/test/intltest/numbertest_api.cpp"
-                    (("(TESTCASE_AUTO\\(unitUsage\\));" all)
-                     (string-append "//" all))))))
-            '())
-        (add-after 'install 'avoid-coreutils-reference
-          ;; Don't keep a reference to the build tools.
-          (lambda* (#:key outputs #:allow-other-keys)
-            (let ((out (assoc-ref outputs "out")))
-              (substitute* (find-files (string-append out "/lib/icu")
-                                       "\\.inc$")
-                (("INSTALL_CMD=.*/bin/install") "INSTALL_CMD=install"))
-              #t))))))
+       ,@(if (%current-target-system)
+             '((string-append "--with-cross-build="
+                              (assoc-ref %build-inputs "icu4c-build-root")))
+             '()))
+     #:phases
+     #~(modify-phases %standard-phases
+         (add-after 'unpack 'chdir-to-source
+           (lambda _ (chdir "source")))
+         (add-after 'chdir-to-source 'update-LDFLAGS
+           (lambda _
+             ;; Do not create a "data-only" libicudata.so because it causes
+             ;; problems on some architectures (notably armhf and MIPS).
+             (substitute* "config/mh-linux"
+               (("LDFLAGSICUDT=-nodefaultlibs -nostdlib")
+                "LDFLAGSICUDT="))))
+         #$@(if (target-riscv64?)
+                `((add-after 'unpack 'disable-failing-test
+                    ;; It is unknown why this test is failing.
+                    (lambda _
+                      (substitute* "source/test/intltest/numbertest_api.cpp"
+                        (("(TESTCASE_AUTO\\(unitUsage\\));" all)
+                         (string-append "//" all))))))
+                '())
+         (add-after 'install 'avoid-coreutils-reference
+           ;; Don't keep a reference to the build tools.
+           (lambda _
+             (substitute* (find-files (string-append #$output "/lib/icu")
+                                      "\\.inc$")
+               (("INSTALL_CMD=.*/bin/install") "INSTALL_CMD=install")))))))
    (synopsis "International Components for Unicode")
    (description
     "ICU is a set of C/C++ and Java libraries providing Unicode and
-- 
2.38.1

Message #8 received at 60430 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 60430 <at> debbugs.gnu.org, GNU Debbugs <control <at> debbugs.gnu.org>
Subject: Re: bug#60430: [PATCH] gnu: icu4c: Update packaging style.
Date: Tue, 21 Mar 2023 11:19:41 -0400

retitle 60430 [PATCH core-updates] gnu: icu4c: Update packaging style.
quit

Hello!

Nicolas Graves <ngraves <at> ngraves.fr> writes:

> * gnu/packages/icu4c.scm (icu4c): Update packaging style.
> ---
>  gnu/packages/icu4c.scm | 66 ++++++++++++++++++++----------------------
>  1 file changed, 32 insertions(+), 34 deletions(-)
>
> diff --git a/gnu/packages/icu4c.scm b/gnu/packages/icu4c.scm
> index 1e4f66d956..1338c8f525 100644
> --- a/gnu/packages/icu4c.scm
> +++ b/gnu/packages/icu4c.scm
> @@ -63,41 +63,39 @@ (define-public icu4c
>     (inputs
>      (list perl))
>     (arguments
> -    `(#:configure-flags
> -      (list
> +    (list
> +     #:configure-flags
> +     `(list
>         "--enable-rpath"
> -        ,@(if (%current-target-system)
> -              '((string-append "--with-cross-build="
> -                                (assoc-ref %build-inputs "icu4c-build-root")))
> -              '()))
> -      #:phases
> -      (modify-phases %standard-phases
> -        (add-after 'unpack 'chdir-to-source
> -          (lambda _ (chdir "source") #t))
> -        (add-after 'chdir-to-source 'update-LDFLAGS
> -          (lambda _
> -            ;; Do not create a "data-only" libicudata.so because it causes
> -            ;; problems on some architectures (notably armhf and MIPS).
> -            (substitute* "config/mh-linux"
> -              (("LDFLAGSICUDT=-nodefaultlibs -nostdlib")
> -               "LDFLAGSICUDT="))
> -            #t))
> -        ,@(if (target-riscv64?)
> -            `((add-after 'unpack 'disable-failing-test
> -                ;; It is unknown why this test is failing.
> -                (lambda _
> -                  (substitute* "source/test/intltest/numbertest_api.cpp"
> -                    (("(TESTCASE_AUTO\\(unitUsage\\));" all)
> -                     (string-append "//" all))))))
> -            '())
> -        (add-after 'install 'avoid-coreutils-reference
> -          ;; Don't keep a reference to the build tools.
> -          (lambda* (#:key outputs #:allow-other-keys)
> -            (let ((out (assoc-ref outputs "out")))
> -              (substitute* (find-files (string-append out "/lib/icu")
> -                                       "\\.inc$")
> -                (("INSTALL_CMD=.*/bin/install") "INSTALL_CMD=install"))
> -              #t))))))
> +       ,@(if (%current-target-system)
> +             '((string-append "--with-cross-build="
> +                              (assoc-ref %build-inputs "icu4c-build-root")))
> +             '()))
> +     #:phases
> +     #~(modify-phases %standard-phases
> +         (add-after 'unpack 'chdir-to-source
> +           (lambda _ (chdir "source")))
> +         (add-after 'chdir-to-source 'update-LDFLAGS
> +           (lambda _
> +             ;; Do not create a "data-only" libicudata.so because it causes
> +             ;; problems on some architectures (notably armhf and MIPS).
> +             (substitute* "config/mh-linux"
> +               (("LDFLAGSICUDT=-nodefaultlibs -nostdlib")
> +                "LDFLAGSICUDT="))))
> +         #$@(if (target-riscv64?)
> +                `((add-after 'unpack 'disable-failing-test
> +                    ;; It is unknown why this test is failing.
> +                    (lambda _
> +                      (substitute* "source/test/intltest/numbertest_api.cpp"
> +                        (("(TESTCASE_AUTO\\(unitUsage\\));" all)
> +                         (string-append "//" all))))))
> +                '())
> +         (add-after 'install 'avoid-coreutils-reference
> +           ;; Don't keep a reference to the build tools.
> +           (lambda _
> +             (substitute* (find-files (string-append #$output "/lib/icu")
> +                                      "\\.inc$")
> +               (("INSTALL_CMD=.*/bin/install") "INSTALL_CMD=install")))))))
>     (synopsis "International Components for Unicode")
>     (description
>      "ICU is a set of C/C++ and Java libraries providing Unicode and

This looks good, but it's a core-updates change: guix refresh -l
icu4c <at> 69 says 12433 packages would be rebuilt.

Also, you should update all the variants which inherits this one to also
use gexp expressions, as it may cause problems otherwise.

I'm adding 'core-updates' to the title to avoid merge mistakes.

Thanks for working on it!

-- 
Maxim

Message #15 received at 60430-done <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 60430-done <at> debbugs.gnu.org
Subject: Re: bug#60430: [PATCH core-updates] gnu: icu4c: Update packaging
 style.
Date: Sat, 20 Jan 2024 15:04:39 -0500

Hi,

Nicolas Graves <ngraves <at> ngraves.fr> writes:

> * gnu/packages/icu4c.scm (icu4c): Update packaging style.

That's now already been done on core-updates.

-- 
Thanks,
Maxim

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.