[PATCH] makedumpfile-1.5.4 "make install" assumes directories exist

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux