Re: [Rpm-maint] [rpm-software-management/rpm] [RFC] Make "%patchlist -f patches" work. (#874)

2019-10-02 Thread Peter Jones
> Wouldn't this also make sense for `%sourcelist` too? Yeah - and if you look at the code, it does it for both, I just didn't mention one in the commit message. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] rpmbuild: %patch: fix a memory leak (#873)

2019-10-02 Thread Peter Jones
Yeah, I noticed that it still leaks on multiple uses, and there may be better solutions these days. I chose to stop here for an entirely selfish reason - using -b twice is clearly wrong to do, with only invoking it once, that leak is chaff that shows up when I'm running valgrind to check my

Re: [Rpm-maint] [rpm-software-management/rpm] Support uncompressed/reconstructed payloads (#861)

2019-10-02 Thread Panu Matilainen
PR #880 is the first piece of the puzzle, was easier (and different) from what I originally thought. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[Rpm-maint] [rpm-software-management/rpm] confusing message about too few disk space (#879)

2019-10-02 Thread Michael Schroeder
This pops up from time to time as a bug report. The message rpm prints for too low disk space/too few inodes is a bit confusing: ``` foo.noarch needs 377MB on the /usr/src filesystem ``` But the disk has more than 377 MB free space. What's meant is that it needs 377 MB *more* disk space. Could

[Rpm-maint] [rpm-software-management/rpm] Add uncompressed payload digest to packages (#880)

2019-10-02 Thread Panu Matilainen
First piece of the puzzle to better support reconstructed payloads (#861) The actual work is pretty trivial here, but I dont like the name suggested in the ticket (too long), and I dont much the one used here much better, hence RFC only at the moment. Uncompressed is such a long annoying word,

Re: [Rpm-maint] [rpm-software-management/rpm] Added option to rpmbuild to check the build dependencies (-bd, -td, -rd) (#876)

2019-10-02 Thread Panu Matilainen
I concur, having special-purpose switch(es) for this seems nicer and way more obvious than using --nobuild to achieve the same :+1: -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] Change in behaviour from 4.14 to 4.15 when calling rpmbuild --nobuild (#875)

2019-10-02 Thread Panu Matilainen
It does smell like a regression, but then it's not clear whether --nobuild was ever intended to be used stand-alone. By appearance more like a disabler similar to --nocheck and --noprep etc, to be used with some other build modes. Eg 'rpmbuild -bb --nobuild ' does still work for checking

[Rpm-maint] [rpm-software-management/rpm] Don't insert rpmlib() dependencies for dynamic buildrequires (#878)

2019-10-02 Thread Panu Matilainen
rpmlib() dependencies are an installability barrier, and dynamic buildrequires do not qualify: the package cannot be correctly *built* with older rpm versions, but they will simply fail to parse the spec due to unknown tag. However rpmlib(DynamicBuildRequires) as it is injected by rpm 4.15.0