[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