Bug 180222

Summary:	portage double-compresses manpages
Product:	Gentoo Linux	Reporter:	Andre <andremuellerster>
Component:	New packages	Assignee:	Portage team <dev-portage>
Status:	RESOLVED FIXED
Severity:	trivial	CC:	tinaught
Priority:	High	Keywords:	InVCS
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	181949
Attachments:	make ecompressdir use -f for the automatic decompression phase

Description Andre 2007-05-29 14:24:49 UTC

The install of powertop-1.5 installs 
/usr/share/man/man1/powertop.1.gz.bz2
along with the proper 
/usr/share/man/man1/powertop.1.bz2

The first one is called on "man powertop"
and gives garbled output.

After deleting it, all is well.
So, that file should be left out.


Reproducible: Always

Steps to Reproduce:
1. emerge =powertop-1.5

Actual Results:  
# equery f powertop
[...]
/usr/share/man/man1/powertop.1.bz2
/usr/share/man/man1/powertop.1.gz.bz2


Expected Results:  
# equery f powertop
[...]
/usr/share/man/man1/powertop.1.bz2

Comment 1 Pietro Franchi 2007-05-29 16:18:58 UTC

this is because make install already installs the manual...
not sure what is the correct way of handling this, but maybe we should just leave out "doman powertop.1"

Comment 2 Stefan Schweizer (RETIRED) gentoo-dev

2007-05-30 00:03:45 UTC

I just worked around this with

        gunzip ${D}/usr/share/man/man1/powertop.1.gz

portage needs to eiher uncompress before bzipping or leave .gz files out of the bzipping.

Comment 3 Zac Medico gentoo-dev

2007-05-30 00:23:54 UTC

(In reply to comment #1)
> this is because make install already installs the manual...
> not sure what is the correct way of handling this, but maybe we should just
> leave out "doman powertop.1"

Yeah, "doman powertop.1" is redundant in this ebuild.

This is happening because we don't have -f in the default PORTAGE_COMPRESS_FLAGS anymore.  Without -f, duplicate files like this are not overwritten.  If we don't want add -f to the default PORTAGE_COMPRESS_FLAGS, I suppose we can manually check for duplicates and remove them or else give a QA warning that the ebuild is trying to install duplicate files.

Comment 4 Zac Medico gentoo-dev

2007-05-30 00:37:26 UTC

Created attachment 120666 [details, diff]
make ecompressdir use -f for the automatic decompression phase

This seems to solve the problem without altering the default PORTAGE_COMPRESS_FLAGS.

Comment 5 Zac Medico gentoo-dev

2007-05-31 01:29:16 UTC

This has been released in 2.1.2.9.