Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 180222 - portage double-compresses manpages
Summary: portage double-compresses manpages
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High trivial (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks: 181949
  Show dependency tree
 
Reported: 2007-05-29 14:24 UTC by Andre
Modified: 2007-05-31 01:29 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
make ecompressdir use -f for the automatic decompression phase (force_decompress.patch,713 bytes, patch)
2007-05-30 00:37 UTC, Zac Medico
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.