From: Maxim Cournoyer <maxim.cournoyer@gmail.com>
To: jgart <jgart@dismail.de>
Cc: 50833-done@debbugs.gnu.org, "Ludovic Courtès" <ludo@gnu.org>,
"Sarah Morgensen" <iskarian@mgsn.dev>
Subject: bug#50833: [PATCH] Add Bower (notmuch curses email client)
Date: Fri, 02 Sep 2022 10:05:25 -0400 [thread overview]
Message-ID: <87o7vx7ty2.fsf@gmail.com> (raw)
In-Reply-To: nobody's message of "(unknown date)"
Hi,
jgart <jgart@dismail.de> writes:
> * gnu/packages/mail.scm (bower): New variable.
>
> Hi Maxim,
>
> Here's a version 3 patch set.
>
> I bumped bower to version 1.0 which was released a few days ago.
Thanks.
I've made the following changes:
--8<---------------cut here---------------start------------->8---
modified gnu/packages/mail.scm
@@ -1517,51 +1517,50 @@ (define-public bower
(source
(origin
(method git-fetch)
- (uri
- (git-reference
- (url home-page)
- (commit version)))
+ (uri (git-reference
+ (url home-page)
+ (commit version)))
(file-name (git-file-name name version))
(sha256
(base32 "0vcsbxlsvr2wv3c7sfr3yj21kbqy259skpxg00vf5bdkbc8qknq4"))))
(build-system gnu-build-system)
(arguments
- (list
- #:make-flags
- #~(list
- "bower"
- "man"
- (string-append "CC=" #+(cc-for-target))
- (string-append "prefix=" #$output))
- #:phases
- #~(modify-phases %standard-phases
- (delete 'configure)
- (add-after 'unpack 'patch-executables
- (lambda* (#:key inputs #:allow-other-keys)
- (substitute* "src/detect_mime_type.m"
- (("\"file")
- (string-append "\"" (search-input-file inputs "/bin/file"))))
- (substitute* "src/compose.m"
- (("\"base64")
- (string-append "\"" (search-input-file inputs "/bin/base64"))))
- (substitute* "src/prog_config.m"
- (("\\(\"false")
- (string-append "(\"" (search-input-file inputs "/bin/false")))
- (("\\(\"notmuch\"")
- (string-append "(\"" (search-input-file inputs "/bin/notmuch") "\""))
- (("/usr/bin/sendmail")
- (search-input-file inputs "/sbin/sendmail")))))
- (replace 'check
- (lambda* (#:key tests? #:allow-other-keys)
- (when tests?
- (with-directory-excursion "tests"
- (invoke "make")))))
- (replace 'install
- (lambda* _
- (let ((bin (string-append #$output "/bin"))
- (man (string-append #$output "/share/man/man1")))
- (install-file "bower" bin)
- (install-file "bower.1" man)))))))
+ (list
+ #:make-flags #~(list "bower" "man"
+ (string-append "CC=" #+(cc-for-target))
+ (string-append "prefix=" #$output))
+ #:parallel-tests? #f ;parallelism breaks test suite
+ #:phases
+ #~(modify-phases %standard-phases
+ (delete 'configure)
+ (add-after 'unpack 'patch-executables
+ (lambda* (#:key inputs #:allow-other-keys)
+ (substitute* "src/detect_mime_type.m"
+ (("\"file\"")
+ (format #f "~s" (search-input-file inputs "bin/file"))))
+ (substitute* "src/compose.m"
+ (("\"base64\"")
+ (format #f "~s" (search-input-file inputs "bin/base64"))))
+ (substitute* "src/prog_config.m"
+ (("shell_quoted\\(\"false\")")
+ (format #f "shell_quoted(~s)"
+ (search-input-file inputs "bin/false")))
+ (("shell_quoted\\(\"notmuch\")")
+ (format #f "shell_quoted(~s)"
+ (search-input-file inputs "bin/notmuch")))
+ (("/usr/bin/sendmail")
+ (search-input-file inputs "/sbin/sendmail")))))
+ (replace 'check
+ (lambda* (#:key parallel-tests? tests? #:allow-other-keys)
+ (when tests?
+ (invoke "make" "-C" "tests"
+ "-j" (if parallel-tests?
+ (number->string (parallel-job-count))
+ "1")))))
+ (replace 'install
+ (lambda* _
+ (install-file "bower" (string-append #$output "/bin"))
+ (install-file "bower.1" (string-append #$output
+ "/share/man/man1")))))))
(native-inputs
(list diffutils
gawk
@@ -1569,14 +1568,14 @@ (define-public bower
pandoc
util-linux))
(inputs
- (list gpgme
- coreutils
+ (list coreutils
+ gpgme
+ ncurses
notmuch
- sendmail
- ncurses))
- (synopsis "Terminal client for the notmuch email system")
- (description "@code{bower} is a curses frontend for the notmuch email
-system. It is written in @{mercury}.")
+ sendmail))
+ (synopsis "Terminal client for the Notmuch email system")
+ (description "@code{bower} is a curses front-end for the Notmuch email
+system, written in the Mercury language.")
(license license:gpl3+)))
--8<---------------cut here---------------end--------------->8---
I also added:
--8<---------------cut here---------------start------------->8---
(properties `((cpe-name . "bower-cpe-refers-to-a-different-bower")))
--8<---------------cut here---------------end--------------->8---
After realizing the CPE database referred to a different package with
the same name, reporting a false positive.
And pushed!
Thank you,
Closing.
Maxim
prev parent reply other threads:[~2022-09-02 14:06 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-26 23:08 [bug#50833] Add bower (it's not what you're thinking) jgart via Guix-patches via
2021-09-26 23:11 ` [bug#50833] [PATCH] gnu: Add bower jgart via Guix-patches via
2021-09-27 1:01 ` Sarah Morgensen
2021-09-29 1:12 ` jgart via Guix-patches via
2021-10-01 2:11 ` Sarah Morgensen
2021-10-01 3:31 ` jgart via Guix-patches via
2021-10-02 7:58 ` jgart via Guix-patches via
2021-10-25 4:32 ` jgart via Guix-patches via
2021-10-25 12:25 ` [bug#50833] Add bower (it's not what you're thinking) Ludovic Courtès
2022-04-20 21:07 ` [bug#50833] [PATCH] Add Bower (notmuch curses email client) Maxim Cournoyer
2022-05-05 14:31 ` jgart via Guix-patches via
2022-07-07 18:01 ` Maxim Cournoyer
2022-07-08 0:23 ` jgart via Guix-patches via
2022-07-08 10:33 ` Munyoki Kilyungi
2022-07-08 18:47 ` jgart via Guix-patches via
2022-07-08 19:42 ` jgart via Guix-patches via
2022-07-14 8:54 ` Munyoki Kilyungi
2022-07-11 16:09 ` Maxim Cournoyer
2022-07-15 1:50 ` jgart via Guix-patches via
2021-09-29 1:12 ` [bug#50833] [PATCH] gnu: Add bower jgart via Guix-patches via
2021-10-01 3:31 ` jgart via Guix-patches via
2022-07-20 5:18 ` [bug#50833] [PATCH v2] " jgart via Guix-patches via
2022-07-20 11:59 ` Maxime Devos
2022-07-24 17:45 ` jgart via Guix-patches via
2022-08-21 6:53 ` [bug#50833] [PATCH v3] " jgart via Guix-patches via
2022-09-02 14:05 ` Maxim Cournoyer [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87o7vx7ty2.fsf@gmail.com \
--to=maxim.cournoyer@gmail.com \
--cc=50833-done@debbugs.gnu.org \
--cc=iskarian@mgsn.dev \
--cc=jgart@dismail.de \
--cc=ludo@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.