[Rpm-maint] [rpm-software-management/rpm] implement %elif (#613)
pavlinamv
notifications at github.com
Mon Dec 17 19:33:56 UTC 2018
This pull request contains 6 patches:
- patches 1, 2, 3: contain refactoring of the existing without any functional change. It is a preparation for the patch 5.
- patch 4: improves "Bad %if condition" error message
- patch 5: adds support of %elif operators (%elif, %elifarch, %elifos)
- patch 6: adds tests for %elif operators
If it will be requested, I can add patch 7 that restricts usage of %elif, %elifarch and %elifos such that: In scope of %if you can use only %elif, in scope of %ifos you can use only %elifos and similarly for %ifarch.
You can view, comment on, or merge this pull request online at:
https://github.com/rpm-software-management/rpm/pull/613
-- Commit Summary --
* Refactor variable "match" to have clear meaning
* Move checking whether %if condition will be resolved to the right place
* Refactor expression expansion evaluation in parseSpec
* Make "Bad %elif condition" error message more informative
* Support %elif operators (#311)
* Add tests for %elif, %elifos, %elifarch
-- File Changes --
M build/parseSpec.c (85)
M build/rpmbuild_internal.h (2)
M build/spec.c (2)
A tests/data/SPECS/iftest.spec (116)
M tests/rpmmacro.at (104)
-- Patch Links --
https://github.com/rpm-software-management/rpm/pull/613.patch
https://github.com/rpm-software-management/rpm/pull/613.diff
--
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/613
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20181217/60f0f34e/attachment.html>
More information about the Rpm-maint
mailing list