[Rpm-maint] [rpm-software-management/rpm] Test for the variable rather than it's expansion consistently in cmak… (PR #2506)

Panu Matilainen notifications at github.com
Tue May 9 09:31:26 UTC 2023


…e files

Testing for the expansion was a cmake newbie mistake that can go wrong in weird ways. Don't try to be clever with for-loops, just test for each case separately. This makes it greppable as well.
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/2506

-- Commit Summary --

  * Test for the variable rather than it's expansion consistently in cmake files

-- File Changes --

    M CMakeLists.txt (23)
    M docs/CMakeLists.txt (2)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/2506.patch
https://github.com/rpm-software-management/rpm/pull/2506.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2506
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/2506 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20230509/544eac8c/attachment-0001.html>


More information about the Rpm-maint mailing list