----------------------------------------
> Date: Tue, 27 Aug 2013 13:02:17 -0400
> From: [email protected]
> To: [email protected]
> Subject: Re: [aur-general] Proofreading request
>
> # Using `install` to create directories is common. I believe `makepkg` sets
> # `umask` appropriately though so `makedir` should work.
> install -dm755 "$pkgdir/etc"
> install -dm755 "$pkgdir/usr/bin"
> install -dm755 "$pkgdir/usr/share/licenses/$pkgname"
> # Whitespace is nice.
>
> # The `-D` switch is not necessary here.
> install -m755 feather "$pkgdir/usr/bin/feather"
> install -m644 feather.yaml.dist "$pkgdir/etc/feather.yaml"
> # You should put LICENSE in the `source` array.
> install -m644 ../../LICENSE "$pkgdir/usr/share/licenses/$pkgname/LICENSE"
> }

Why. Leave the -D switch and get rid of the extra step of making the dirs 
altogether. Much simpler and cleaner.                                           

Reply via email to