Hello Nick,
On Mon, 29 Jul 2013 09:31:55 -0700
Nick Bartos <nick at pistoncloud.com> wrote:
> The make install of makedumpfile-1.5.4 uses cp instead of the usual
> "install" command, which has some side effects of not pre-creating
> directories. This results in installs to an alternate DESTDIR fail because
> the parent directories do not exist.
>
> I have attached a patch which modifies make install to use the install
> utility.
>
> ---
> diff -U3 -r makedumpfile-1.5.4.orig/Makefile makedumpfile-1.5.4/Makefile
> --- makedumpfile-1.5.4.orig/Makefile 2013-07-01 08:08:49.000000000 +0000
> +++ makedumpfile-1.5.4/Makefile 2013-07-18 17:15:01.109058204 +0000
> @@ -86,8 +86,8 @@
> rm -f $(OBJ) $(OBJ_PART) $(OBJ_ARCH) makedumpfile makedumpfile.8.gz makedumpfile.conf.5.gz
>
> install:
> - cp makedumpfile ${DESTDIR}/bin
> - cp makedumpfile-R.pl ${DESTDIR}/bin
> - cp makedumpfile.8.gz ${DESTDIR}/usr/share/man/man8
> - cp makedumpfile.conf.5.gz ${DESTDIR}/usr/share/man/man5
> - cp makedumpfile.conf ${DESTDIR}/etc/makedumpfile.conf.sample
> + install -m 755 -d ${DESTDIR}/usr/bin ${DESTDIR}/usr/share/man/man5 ${DESTDIR}/usr/share/man/man8 ${DESTDIR}/etc
> + install -m 755 -t ${DESTDIR}/usr/bin makedumpfile makedumpfile-R.pl
> + install -m 644 -t ${DESTDIR}/usr/share/man/man8 makedumpfile.8.gz
> + install -m 644 -t ${DESTDIR}/usr/share/man/man5 makedumpfile.conf.5.gz
> + install -m 644 -D makedumpfile.conf ${DESTDIR}/etc/makedumpfile.conf.sample
I think /usr/sbin is better than /usr/bin as installation directory
according to Filesystem Hierarchy Standard.
4.10. /usr/sbin : Non-essential standard system binaries
4.10.1. Purpose
This directory contains any non-essential binaries used exclusively
by the system administrator. System administration programs that are
required for system repair, system recovery, mounting /usr, or other
essential functions must be placed in /sbin instead.
So may I fix that when I merge this patch ?
Thanks
Atsushi Kumagai
