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

Bug 672916

Summary: sys-apps/portage: calls lz4 incorrectly for multiple files (was: dev-tcltk/blt-2.4z-r13: ecompress-file failed with PORTAGE_COMPRESS="lz4")
Product: Portage Development Reporter: Oscar Andrea Merandi <oscar.andrea>
Component: CoreAssignee: Portage team <dev-portage>
Status: RESOLVED FIXED    
Severity: normal CC: mgorny, oscar.andrea, sam
Priority: Normal Keywords: InVCS
Version: unspecified   
Hardware: AMD64   
OS: Linux   
See Also: https://bugs.gentoo.org/show_bug.cgi?id=667072
https://bugs.gentoo.org/show_bug.cgi?id=783714
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 859439    
Attachments: log /var/tmp/portage/dev-tcltk/blt-2.4z-r13/temp/build.log
emerge_info.txt

Description Oscar Andrea Merandi 2018-12-11 08:33:58 UTC
blt-2.4z-r13 fails the compilation, and in the logs you may notice asking whether to overwrite or not /var/tmp/portage/dev-tcltk/blt-2.4z-r13/image/usr/share/doc/blt-2.4z-r13/README
Also manually deleting /var/tmp/portage and trying to compile again, the situation does not change.

Reproducible: Always
Comment 1 Oscar Andrea Merandi 2018-12-11 08:35:32 UTC
Created attachment 557556 [details]
log /var/tmp/portage/dev-tcltk/blt-2.4z-r13/temp/build.log
Comment 2 Oscar Andrea Merandi 2018-12-11 08:36:59 UTC
Created attachment 557558 [details]
emerge_info.txt
Comment 3 Tupone Alfredo gentoo-dev 2018-12-11 20:25:17 UTC
The error show up during the merge phase.

Need to involve the portage team
Comment 4 Zac Medico gentoo-dev 2018-12-11 20:43:54 UTC
This is triggered with PORTAGE_COMPRESS="lz4".
Comment 5 Oscar Andrea Merandi 2018-12-12 10:22:48 UTC
(In reply to Zac Medico from comment #4)
> This is triggered with PORTAGE_COMPRESS="lz4".

Yes, without compressing with lz4 the software is compiled.
Comment 6 genBTC 2022-08-17 22:18:39 UTC
This bug is still reproducible on the current package: dev-tcltk/blt-2.5.3-r1 with fully updated system. 
Same e-compress file failed, but only when PORTAGE_COMPRESS="lz4".
Comment 7 Larry the Git Cow gentoo-dev 2022-08-19 17:59:23 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/proj/portage.git/commit/?id=0275734583c28a5a9e9e5c07a2724269efe5d1a5

commit 0275734583c28a5a9e9e5c07a2724269efe5d1a5
Author:     Philipp Rösner <philipp.roesner@student.uni-halle.de>
AuthorDate: 2022-08-19 15:46:03 +0000
Commit:     Mike Gilbert <floppym@gentoo.org>
CommitDate: 2022-08-19 17:31:17 +0000

    bin/ecompress: add lz4 command line options
    
    Calling lz4 with mutliple files results in warnings of the form:
    
    `Warning : <path>/<file> won't be used !
    Do you want multiple input files (-m) ?`
    
    Adding the `-m` option to the lz4 call resolves this issue.
    Additionally, the `--rm` option will remove the source files after a
    successful compression.
    
    Bug: https://bugs.gentoo.org/672916
    Closes: https://github.com/gentoo/portage/pull/892
    Signed-off-by: Philipp Rösner <rndxelement@protonmail.com>
    Signed-off-by: Mike Gilbert <floppym@gentoo.org>

 NEWS          | 3 +++
 bin/ecompress | 4 ++++
 2 files changed, 7 insertions(+)
Comment 8 Larry the Git Cow gentoo-dev 2022-08-23 00:39:11 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=328977ffff1aa2d13cd3d8f7b5cb01eea6d634f5

commit 328977ffff1aa2d13cd3d8f7b5cb01eea6d634f5
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2022-08-23 00:38:45 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-08-23 00:39:03 +0000

    sys-apps/portage: add 3.0.35
    
    Closes: https://bugs.gentoo.org/859439
    Closes: https://bugs.gentoo.org/672916
    Closes: https://bugs.gentoo.org/858791
    Closes: https://bugs.gentoo.org/862600
    Closes: https://bugs.gentoo.org/862606
    Closes: https://bugs.gentoo.org/863422
    Closes: https://bugs.gentoo.org/864382
    Closes: https://bugs.gentoo.org/865635
    Signed-off-by: Sam James <sam@gentoo.org>

 sys-apps/portage/Manifest              |   1 +
 sys-apps/portage/portage-3.0.35.ebuild | 273 +++++++++++++++++++++++++++++++++
 2 files changed, 274 insertions(+)