[pacman-dev] [PATCH] Compress manpages only after checking for $srcdir and $pkgdir
Move the manpage compression code from tidy_install to create_package. Since
tidy_install could remove a backup file, it should go before check_package.
Alternatively, we could move all the code from tidy_install into create_package
since those two are always called together.
Fixes FS#33318.
Signed-off-by: Chirantan Ekbote
On 11/01/13 13:00, Chirantan Ekbote wrote:
Move the manpage compression code from tidy_install to create_package. Since tidy_install could remove a backup file, it should go before check_package. Alternatively, we could move all the code from tidy_install into create_package since those two are always called together.
Fixes FS#33318.
Signed-off-by: Chirantan Ekbote
---
I definitely do not want this in create_package. What I would like to see is tidy_install rearranged with check_install added in. So tidy_install should do: removal operations - docs - purge - libtool - emptydirs check for missing backup files grep for srcdir/pkgdir zipman strip upx Eventually we should just move this all into some sort of modular hooklike format like: 10-docs 20-purge ... but we need to be set-up for makepkg splitting first...
From: Chirantan Ekbote
On 29/01/13 12:14, chirantan.ekbote@gmail.com wrote:
From: Chirantan Ekbote
Rearrange tidy_install so we first remove docs, unwanted files, libtool files, and empty directories. Then check for missing backup files and references to $srcdir and $pkgdir. Finally compress manpages, strip debug symbols, and compress executables with upx.
Fixes FS33318
Signed-off-by: Chirantan Ekbote
---
Looks good. Allan
On Mon, Jan 28, 2013 at 8:49 PM, Allan McRae
On 29/01/13 12:14, chirantan.ekbote@gmail.com wrote:
From: Chirantan Ekbote
Rearrange tidy_install so we first remove docs, unwanted files, libtool files, and empty directories. Then check for missing backup files and references to $srcdir and $pkgdir. Finally compress manpages, strip debug symbols, and compress executables with upx.
Fixes FS33318
Signed-off-by: Chirantan Ekbote
--- Looks good.
FS#xxx would look better than FSxxx, but too late. No auto-linking in the commit message on cgit (https://projects.archlinux.org/pacman.git/commit/?id=266b3dd706) unless you prefix it like that. (For a good example, see https://projects.archlinux.org/pacman.git/commit/?id=27067b1372) -Dan
On Mon, Jan 28, 2013 at 10:46 PM, Dan McGee
On Mon, Jan 28, 2013 at 8:49 PM, Allan McRae
wrote: On 29/01/13 12:14, chirantan.ekbote@gmail.com wrote:
From: Chirantan Ekbote
Rearrange tidy_install so we first remove docs, unwanted files, libtool files, and empty directories. Then check for missing backup files and references to $srcdir and $pkgdir. Finally compress manpages, strip debug symbols, and compress executables with upx.
Fixes FS33318
Signed-off-by: Chirantan Ekbote
--- Looks good.
FS#xxx would look better than FSxxx, but too late. No auto-linking in the commit message on cgit (https://projects.archlinux.org/pacman.git/commit/?id=266b3dd706) unless you prefix it like that. (For a good example, see https://projects.archlinux.org/pacman.git/commit/?id=27067b1372)
-Dan
Thanks, I'll remember that for the next one :-) +chirantan
participants (4)
-
Allan McRae
-
Chirantan Ekbote
-
chirantan.ekbote@gmail.com
-
Dan McGee