Re: [Rpm-maint] [rpm-software-management/rpm] Add --zlib-compress-debug-sections to find-debug-info.sh (#795)

2019-08-19 Thread Tom de Vries
> Is there some particular user / use-case for this? No, not for now. The benefit of using the --zlib-compress-debug-sections option is that installed debug info gets smaller. The drawback is that loading debug information get slower. This may be an acceptable trade-off for either an an

Re: [Rpm-maint] [rpm-software-management/rpm] Prefer '[] && []' to '[ -a ]' and '[] || []' to '[ -o ]' in tests (#810)

2019-08-19 Thread Panu Matilainen
When pushing updates to address comments, make a note of what you changed here. It's not always that obvious, and pushes without comments go easily unnoticed overall. ""Prefer '[] && []' to '[ -a ]' and '[] || []' to '[ -o ]'" describes the literal change done by the patch. This is not useful

Re: [Rpm-maint] [rpm-software-management/rpm] Add a dummy line to CI setup to allow controlling dnf gpgcheck per build (#819)

2019-08-19 Thread Panu Matilainen
Merged #819 into master. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/819#event-2565919051___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] Add a dummy line to CI setup to allow controlling dnf gpgcheck per build (#819)

2019-08-19 Thread Panu Matilainen
@pmatilai pushed 1 commit. b2decdfad1ad2d92e4fec1df597421d2313a76f3 Add a dummy line to CI setup to allow controlling dnf gpgcheck per build -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] Add a dummy line to CI setup to allow controlling dnf gpgcheck per build (#819)

2019-08-19 Thread Panu Matilainen
@pmatilai pushed 1 commit. 30fede268c054cac9e784524137e97f1e6a5e60b Add a dummy line to CI setup to allow controlling dnf gpgcheck per build -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] Add a dummy line to CI setup to allow controlling dnf gpgcheck per build (#819)

2019-08-19 Thread Panu Matilainen
@pmatilai pushed 1 commit. 341f740139ceb0f611aa6375119e70ca7a70775a Add a dummy line to CI setup to allow controlling dnf gpgcheck per build -- You are receiving this because you are subscribed to this thread. View it on GitHub:

[Rpm-maint] [rpm-software-management/rpm] Add a dummy line to CI setup to allow controlling dnf gpgcheck per build (#819)

2019-08-19 Thread Panu Matilainen
This doesn't do anything at all in itself because all the repositories have gpgcheck=1 by default. However adding this line allows disabling the gpgcheck for individual builds via Semaphore build settings, which allows us to deal with signatures on rawhide breaking twice a year causing our builds

Re: [Rpm-maint] [rpm-software-management/rpm] Fix segfault regression on empty script or description at end of spec (#816)

2019-08-19 Thread Panu Matilainen
If you look at the real commit messages, it's quite different to how GH renders it. The long commit hashes just don't align very nicely at < 78 characters per line. -- 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] Fix segfault regression on empty script or description at end of spec (#816)

2019-08-19 Thread Panu Matilainen
Merged #816 into master. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/816#event-2565729934___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] Add %{expr:...} macro for parsing expressions (#817)

2019-08-19 Thread Panu Matilainen
Oh and BTW, the reason for doing this *just now* is that it might be kinda necessary for resolving #804 in the macro space instead of placing all the calculations on the C side. Of course technically Lua could be used for those same calculations but we I don't want to make Lua a hard

Re: [Rpm-maint] [rpm-software-management/rpm] Add %{expr:...} macro for parsing expressions (#817)

2019-08-19 Thread Panu Matilainen
The error messages are simply the same as you get from spec %if conditionals, there are no messages added / changed in this PR. That's not to say the messages couldn't maybe be improved, but any change needs to account for the fact that they're shared between two quite different contexts.

Re: [Rpm-maint] [rpm-software-management/rpm] Add %{expr:...} macro for parsing expressions (#817)

2019-08-19 Thread Panu Matilainen
@pmatilai pushed 1 commit. 22331d303765509f1856352713a8a4de74b1085e Add %{expr:...} macro for parsing expressions -- You are receiving this because you are subscribed to this thread. View it on GitHub: