Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 953092

Summary: app-arch/patool: Use app-arch/7zip if applicable
Product: Gentoo Linux Reporter: NRK <nrk>
Component: Current packagesAssignee: Alfred Wingate <parona>
Status: RESOLVED FIXED    
Severity: normal CC: nrk, proxy-maint
Priority: Normal Keywords: PullRequest
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://github.com/gentoo/gentoo/pull/41593
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on: 953858    
Bug Blocks: 942397    

Description NRK 2025-04-03 16:13:01 UTC
This currently has a dependency on app-arch/p7zip. But the script seems to recognize 7zz executable [0] already so maybe it can be changed to `|| ( app-arch/7zip-24.09 app-arch/p7zip )` instead.

[0]: https://github.com/wummel/patool/blob/8c8b4fcd649a36d4e76e422e259f1544b7e0a9bd/patoolib/__init__.py#L161


See also: https://bugs.gentoo.org/942397
Comment 1 Larry the Git Cow gentoo-dev 2025-04-14 23:29:14 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0254b513dd6c9210f989d37ce3882c967c2f0e56

commit 0254b513dd6c9210f989d37ce3882c967c2f0e56
Author:     Alfred Wingate <parona@protonmail.com>
AuthorDate: 2025-04-14 18:00:41 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2025-04-14 23:28:48 +0000

    app-arch/patool: use app-arch/7zip if possible
    
    7zip is also keyworded on x86 now due to https://bugs.gentoo.org/953098.
    
    7zip + || ( 7zip[symlink] p7zip ) is to get maximum test coverage as
    patool supports both 7zz and 7z.
    
    Bug: https://bugs.gentoo.org/953092
    Signed-off-by: Alfred Wingate <parona@protonmail.com>
    Closes: https://github.com/gentoo/gentoo/pull/41593
    Signed-off-by: Sam James <sam@gentoo.org>

 app-arch/patool/patool-4.0.0.ebuild | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)