Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 619936 - app-misc/socnetv-2.2: uses doman on compressed file
Summary: app-misc/socnetv-2.2: uses doman on compressed file
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: doman-compressed
  Show dependency tree
 
Reported: 2017-05-28 09:12 UTC by Michał Górny
Modified: 2022-10-18 03:27 UTC (History)
1 user (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2017-05-28 09:12:01 UTC 
The package uses doman/newman on a compressed file which is invalid per PMS:

      doman "${S}/man/${PN}.1.gz"
Comment 1 mimosinnet 2017-11-21 08:35:54 UTC 
I have read this [1] documentation, copied portage/app-misc/socnetv to my local repository, renamed the ebuild to socnetv-2.3.ebuild and successfully emerged socnetv-2.3. Equery show these files: 

---
equery files socnetv | grep man
/usr/share/man
/usr/share/man/man1
/usr/share/man/man1/socnetv.1.bz2
---

At the moment, ' doman "${S}/man/${PN}.1.gz" ' installs the compressed man page. As I understand it, the ebuild should uncompress socnetv.1.bz2 and install it in /usr/share/man/man1/. What would be the proper way of stating this in the ebuild?

Thanks! 


[1] https://wiki.gentoo.org/wiki/Basic_guide_to_write_Gentoo_Ebuilds
Comment 2 Agostino Sarubbo gentoo-dev 2021-12-13 16:13:34 UTC 
tinderbox has reproduced this issue with version 2.2 - Updating summary.
Comment 3 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-10-18 03:23:29 UTC 
(In reply to mimosinnet from comment #1)
> I have read this [1] documentation, copied portage/app-misc/socnetv to my
> local repository, renamed the ebuild to socnetv-2.3.ebuild and successfully
> emerged socnetv-2.3. Equery show these files: 
> 
> ---
> equery files socnetv | grep man
> /usr/share/man
> /usr/share/man/man1
> /usr/share/man/man1/socnetv.1.bz2
> ---
> 
> At the moment, ' doman "${S}/man/${PN}.1.gz" ' installs the compressed man
> page. As I understand it, the ebuild should uncompress socnetv.1.bz2 and
> install it in /usr/share/man/man1/. What would be the proper way of stating
> this in the ebuild?


So, the installed files were OK in your list because Portage covers it up (it decompresses-and-recompresses for you).

The fix is to gzip -d (gunzip/decompress/ungzip), then doman without .gz.
Comment 4 Larry the Git Cow gentoo-dev 2022-10-18 03:27:44 UTC 
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=9f8d422362e2ef012b66a5f0e581cfd22812f492

commit 9f8d422362e2ef012b66a5f0e581cfd22812f492
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2022-10-18 03:23:49 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-10-18 03:27:35 +0000

    app-misc/socnetv: don't call doman on compressed file
    
    Closes: https://bugs.gentoo.org/619936
    Signed-off-by: Sam James <sam@gentoo.org>

 app-misc/socnetv/{socnetv-2.2.ebuild => socnetv-2.2-r1.ebuild} | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)