Fix header ifdef check for config-post.h in VPATH build

[libvirt] [PATCH] Fix header ifdef check for config-post.h in VPATH build

Daniel P. Berrangé posted 1 patch 5 years, 4 months ago

Download mbox

Test syntax-check passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20181214140944.744028-1-berrange@redhat.com

build-aux/header-ifdef.pl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Expand all Fold all

[libvirt] [PATCH] Fix header ifdef check for config-post.h in VPATH build

Posted by Daniel P. Berrangé 5 years, 4 months ago

We must do a substring match, not an exact match since
there can be an arbitrary virtual path prepended.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---

Pushed as a build fix

 build-aux/header-ifdef.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/build-aux/header-ifdef.pl b/build-aux/header-ifdef.pl
index 74b4c0246b..ccabf14055 100644
--- a/build-aux/header-ifdef.pl
+++ b/build-aux/header-ifdef.pl
@@ -85,7 +85,7 @@ while (<>) {
     }
 
     if ($mistake ||
-        $ARGV eq "config-post.h" ||
+        $ARGV =~ /config-post\.h$/ ||
         $ARGV =~ /vbox_(CAPI|XPCOM)/) {
         $state = $STATE_EOF;
         next;
-- 
2.19.2

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list