GNU bug report logs -
#49705
[PATCH] gnu: gmnisrv: Fix configure flags.
Previous Next
Reported by: Sarah Morgensen <iskarian <at> mgsn.dev>
Date: Fri, 23 Jul 2021 08:08:02 UTC
Severity: normal
Tags: patch
Done: Arun Isaac <arunisaac <at> systemreboot.net>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 49705 in the body.
You can then email your comments to 49705 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
guix-patches <at> gnu.org
:
bug#49705
; Package
guix-patches
.
(Fri, 23 Jul 2021 08:08:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Sarah Morgensen <iskarian <at> mgsn.dev>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Fri, 23 Jul 2021 08:08:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
Remove the dependency on /etc/mime.types (fixes #49233). Look in /etc
for configuration files rather than /gnu/store/...-gmnisrv.../etc.
* gnu/packages/web.scm (gmnisrv)[arguments]<#:phases> Move logic to...
<#:tests?>: ...here...
<#:make-flags>: ...and here.
<#:configure-flags>: Set --sysconfdir and --with-mimedb.
[propagated-inputs]: Move mailcap to...
[inputs]: ...here (/etc/mime.types is no longer needed).
---
Built, linted, and manually tested. Fixes <https://issues.guix.gnu.org/49233>.
gnu/packages/web.scm | 19 +++++++++----------
1 file changed, 9 insertions(+), 10 deletions(-)
diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm
index 2cc8619b19..12ba55cdc8 100644
--- a/gnu/packages/web.scm
+++ b/gnu/packages/web.scm
@@ -7985,20 +7985,19 @@ solution for any project's interface needs:
(file-name (git-file-name name version))))
(build-system gnu-build-system)
(arguments
- `(#:phases
- (modify-phases %standard-phases
- (add-before 'configure 'set-variables
- (lambda _
- (setenv "CC" "gcc")
- #t))
- (delete 'check))))
+ `(#:tests? #f ; no tests
+ #:configure-flags
+ (list "--sysconfdir=/etc"
+ (string-append "--with-mimedb="
+ (assoc-ref %build-inputs "mailcap")
+ "/etc/mime.types"))
+ #:make-flags '("CC=gcc")))
(inputs
- `(("openssl" ,openssl)))
+ `(("mailcap" ,mailcap)
+ ("openssl" ,openssl)))
(native-inputs
`(("pkg-config" ,pkg-config)
("scdoc" ,scdoc)))
- (propagated-inputs
- `(("mailcap" ,mailcap)))
(synopsis "Simple Gemini protocol server")
(description "gmnisrv is a simple Gemini protocol server written in C.")
(license (list license:gpl3+
base-commit: 89ea0918a4a6cc9c250b85c0b713e471b7769c48
--
2.31.1
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49705
; Package
guix-patches
.
(Sun, 25 Jul 2021 14:24:02 GMT)
Full text and
rfc822 format available.
Message #8 received at 49705 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Hi Sarah,
Thank you for improving our gmnisrv package!
> + #:make-flags '("CC=gcc")))
This should be (list (string-append "CC=" ,(cc-for-target))) so that the
package cross-compiles correctly. Hard-coding CC=gcc is now deprecated
practice.
I also feel that this patch should be split into at least two patches or
maybe even three. This way, the first line summary in the commit message
is much more descriptive of the change.
1. gnu: gmnisrv: Remove dependency on /etc/mime.types.
2. gnu: gmnisrv: Cross-compile.
3. gnu: gmnisrv: Disable tests using tests?.
WDYT? Feel free to combine patches 2 and 3 if that splitting is a bit
excessive.
Thanks,
Arun
[signature.asc (application/pgp-signature, inline)]
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49705
; Package
guix-patches
.
(Sun, 25 Jul 2021 20:39:01 GMT)
Full text and
rfc822 format available.
Message #11 received at 49705 <at> debbugs.gnu.org (full text, mbox):
Hi,
Same as before, but with s/gcc/(cc-for-target)/, and split apart.
Arun, thanks for the pointers (and the copyright line; how do I still
forget?)
--
Sarah Morgensen (4):
gnu: gmnisrv: Disable tests using #:tests?.
gnu: gmnisrv: Use #:make-flags, and cross-compile.
gnu: gmnisrv: Remove dependency on /etc/mime.types.
gnu: gmnisrv: Fix configuration search path.
gnu/packages/web.scm | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
base-commit: b81ab18404a684817166dbac260e5a8c68b35389
--
2.31.1
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49705
; Package
guix-patches
.
(Sun, 25 Jul 2021 20:40:02 GMT)
Full text and
rfc822 format available.
Message #14 received at 49705 <at> debbugs.gnu.org (full text, mbox):
gnu/packages/web.scm (gmnisrv)[arguments]<#:phases>: Undelete check
phase, and...
<#:tests?>: ...disable tests here instead.
---
gnu/packages/web.scm | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm
index f229397db4..0ac13112c2 100644
--- a/gnu/packages/web.scm
+++ b/gnu/packages/web.scm
@@ -7983,13 +7983,13 @@ solution for any project's interface needs:
(file-name (git-file-name name version))))
(build-system gnu-build-system)
(arguments
- `(#:phases
+ `(#:tests? #f ; no check target
+ #:phases
(modify-phases %standard-phases
(add-before 'configure 'set-variables
(lambda _
(setenv "CC" "gcc")
- #t))
- (delete 'check))))
+ #t)))))
(inputs
`(("openssl" ,openssl)))
(native-inputs
--
2.31.1
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49705
; Package
guix-patches
.
(Sun, 25 Jul 2021 20:41:02 GMT)
Full text and
rfc822 format available.
Message #17 received at 49705 <at> debbugs.gnu.org (full text, mbox):
gnu/packages/web.scm (gmnisrv)[arguments]<#:phases>{set-variables}: Move
'CC=' to...
<#:make-flags>: ...here, and support cross-compile with (cc-for-target).
---
gnu/packages/web.scm | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm
index 0ac13112c2..b106e7c8b6 100644
--- a/gnu/packages/web.scm
+++ b/gnu/packages/web.scm
@@ -7984,12 +7984,8 @@ solution for any project's interface needs:
(build-system gnu-build-system)
(arguments
`(#:tests? #f ; no check target
- #:phases
- (modify-phases %standard-phases
- (add-before 'configure 'set-variables
- (lambda _
- (setenv "CC" "gcc")
- #t)))))
+ #:make-flags
+ (list (string-append "CC=" ,(cc-for-target)))))
(inputs
`(("openssl" ,openssl)))
(native-inputs
--
2.31.1
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49705
; Package
guix-patches
.
(Sun, 25 Jul 2021 20:41:02 GMT)
Full text and
rfc822 format available.
Message #20 received at 49705 <at> debbugs.gnu.org (full text, mbox):
Remove the dependency on /etc/mime.types (fixes #49233).
gnu/packages/web.scm (gmnisrv)[arguments]<#:configure-flags>: Set
--with-mimedb to directly use mime.types from mailcap.
[propagated-inputs]: Move mailcap to...
[inputs]: ...here (/etc/mime.types no longer need in profiles).
---
gnu/packages/web.scm | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm
index b106e7c8b6..61465c5d44 100644
--- a/gnu/packages/web.scm
+++ b/gnu/packages/web.scm
@@ -7984,15 +7984,18 @@ solution for any project's interface needs:
(build-system gnu-build-system)
(arguments
`(#:tests? #f ; no check target
+ #:configure-flags
+ (list (string-append "--with-mimedb="
+ (assoc-ref %build-inputs "mailcap")
+ "/etc/mime.types"))
#:make-flags
(list (string-append "CC=" ,(cc-for-target)))))
(inputs
- `(("openssl" ,openssl)))
+ `(("mailcap" ,mailcap)
+ ("openssl" ,openssl)))
(native-inputs
`(("pkg-config" ,pkg-config)
("scdoc" ,scdoc)))
- (propagated-inputs
- `(("mailcap" ,mailcap)))
(synopsis "Simple Gemini protocol server")
(description "gmnisrv is a simple Gemini protocol server written in C.")
(license (list license:gpl3+
--
2.31.1
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49705
; Package
guix-patches
.
(Sun, 25 Jul 2021 20:41:03 GMT)
Full text and
rfc822 format available.
Message #23 received at 49705 <at> debbugs.gnu.org (full text, mbox):
Look in /etc for configuration files rather than
/gnu/store/...-gmnisrv.../etc.
gnu/packages/web.scm (gmnisrv)[arguments]<#:configure-flags>: Set
--sysconfdir.
---
gnu/packages/web.scm | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm
index 61465c5d44..e209f9d21d 100644
--- a/gnu/packages/web.scm
+++ b/gnu/packages/web.scm
@@ -7985,7 +7985,8 @@ solution for any project's interface needs:
(arguments
`(#:tests? #f ; no check target
#:configure-flags
- (list (string-append "--with-mimedb="
+ (list "--sysconfdir=/etc"
+ (string-append "--with-mimedb="
(assoc-ref %build-inputs "mailcap")
"/etc/mime.types"))
#:make-flags
--
2.31.1
Reply sent
to
Arun Isaac <arunisaac <at> systemreboot.net>
:
You have taken responsibility.
(Mon, 26 Jul 2021 21:19:01 GMT)
Full text and
rfc822 format available.
Notification sent
to
Sarah Morgensen <iskarian <at> mgsn.dev>
:
bug acknowledged by developer.
(Mon, 26 Jul 2021 21:19:02 GMT)
Full text and
rfc822 format available.
Message #28 received at 49705-done <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Hi Sarah,
> Same as before, but with s/gcc/(cc-for-target)/, and split apart.
Thank you! Pushed with only minor cosmetic changes.
> (and the copyright line; how do I still forget?)
I forget sometimes too. We really need a patch linter to check for this.
Regards,
Arun
[signature.asc (application/pgp-signature, inline)]
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Tue, 24 Aug 2021 11:24:03 GMT)
Full text and
rfc822 format available.
This bug report was last modified 2 years and 239 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.