From: "Léo Le Bouter via Guix-patches via" <guix-patches@gnu.org>
To: Rovanion Luckey <rovanion.luckey@gmail.com>, 47435@debbugs.gnu.org
Subject: [bug#47435] gnu: mail: Make the sendmail package actually output its binary
Date: Sat, 27 Mar 2021 19:07:52 +0100 [thread overview]
Message-ID: <c390fc4d7a78fce9c3fb95e3244344cf21f4eea9.camel@zaclys.net> (raw)
In-Reply-To: <CAAaf0CCUVMmrADMj9Ow200z=jJuzf=W1jPCgwT78gHEKUonSKA@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 3302 bytes --]
On Sat, 2021-03-27 at 18:34 +0100, Rovanion Luckey wrote:
> This is an attempt at getting the sendmail package working. Currently
> installing the package does not make the `sendmail` executable appear
> in `$PATH`:
>
>
> > └$ guix environment --ad-hoc sendmail
> > └$ sendmail
> > bash: sendmail: command not found
>
> This patch fixes that.
>
Thanks a lot for working on this!
It seems the commit message does not follow the ChangeLog guidelines as
used by GNU Guix, I suggest using this:
> gnu: sendmail: Fix output's directory structure.
>
> * gnu/packages/mail.scm (sendmail)[arguments]<#:phases>: Properly
> specify output directories in replacement 'configure and 'pre-
> install.
Look at the commit history of GNU Guix for more examples and
inspiration.
> Though there are questions that still need to be resolved. While this
> patch does fix what it sets out to fix, the solutions aren't
> neccicerily correct. It configures a "helpfile" to be put in
> "$out/usr/share" and the same goes for its "statistics" file. I have
> no idea what these does and where they should go in reality. If
> someone happens to know, please do tell.
If we look at the output of these commands:
With your patch applied:
$ tree $(./pre-inst-env guix build sendmail)
/gnu/store/ygz55p6g8z4kjbp6418w385jzy53324h-sendmail-8.15.2
├── bin
│ ├── hoststat -> /gnu/store/ygz55p6g8z4kjbp6418w385jzy53324h-
sendmail-8.15.2/bin/sendmail
│ ├── mailq -> /gnu/store/ygz55p6g8z4kjbp6418w385jzy53324h-sendmail-
8.15.2/bin/sendmail
│ ├── newaliases -> /gnu/store/ygz55p6g8z4kjbp6418w385jzy53324h-
sendmail-8.15.2/bin/sendmail
│ ├── purgestat -> /gnu/store/ygz55p6g8z4kjbp6418w385jzy53324h-
sendmail-8.15.2/bin/sendmail
│ ├── sendmail
│ └── vacation
├── etc
│ └── mail
│ ├── sendmail.cf
│ └── submit.cf
├── libexec
├── sbin
│ ├── editmap
│ ├── mailstats
│ ├── makemap
│ └── praliases
└── share
├── doc
│ └── sendmail-8.15.2
│ └── LICENSE
├── helpfile
└── statistics
7 directories, 16 files
Without your patch applied:
$ tree $(./pre-inst-env guix build sendmail)
/gnu/store/yn6dlixw6cnbmgjggk640vbv8hidfhi7-sendmail-8.15.2
├── etc
│ └── mail
│ ├── helpfile
│ ├── sendmail.cf
│ ├── statistics
│ └── submit.cf
├── share
│ └── doc
│ └── sendmail-8.15.2
│ └── LICENSE
└── usr
├── bin
│ ├── hoststat -> /usr/sbin/sendmail
│ ├── mailq -> /usr/sbin/sendmail
│ ├── newaliases -> /usr/sbin/sendmail
│ ├── purgestat -> /usr/sbin/sendmail
│ └── vacation
└── sbin
├── editmap
├── mailstats
├── makemap
├── praliases
├── sendmail
└── smrsh
8 directories, 16 files
We can see helpfile and statistics file go into etc/mail, then sendmail
goes into sbin, smrsh is missing in sbin.
Léo
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2021-03-27 18:09 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-27 17:34 [bug#47435] gnu: mail: Make the sendmail package actually output its binary Rovanion Luckey
2021-03-27 18:07 ` Léo Le Bouter via Guix-patches via [this message]
2021-03-27 18:22 ` Leo Famulari
2021-03-28 11:57 ` Rovanion Luckey
2021-03-29 14:24 ` Rovanion Luckey
2021-03-30 2:05 ` Léo Le Bouter via Guix-patches via
2021-03-30 2:13 ` Léo Le Bouter via Guix-patches via
2021-04-09 19:30 ` Rovanion Luckey
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=c390fc4d7a78fce9c3fb95e3244344cf21f4eea9.camel@zaclys.net \
--to=guix-patches@gnu.org \
--cc=47435@debbugs.gnu.org \
--cc=lle-bout@zaclys.net \
--cc=rovanion.luckey@gmail.com \
/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.