1. Patchew
  2. Libvirt
  3. View series

[libvirt] [PATCH] Fix src/ pkg-config API XML paths

Cole Robinson posted 1 patch 4 years, 4 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/a3097f61587e95b679cd4029982bcb5c5e6ee161.1575900167.git.crobinso@redhat.com
src/libvirt-lxc.pc.in  | 2 +-
src/libvirt-qemu.pc.in | 2 +-
src/libvirt.pc.in      | 6 +++---
3 files changed, 5 insertions(+), 5 deletions(-)
[libvirt] [PATCH] Fix src/ pkg-config API XML paths
Posted by Cole Robinson 4 years, 4 months ago 
The .pc files in src/ are intended for use with the ./run script,
to ease building bindings against an uninstalled libvirt build.
The pointer to the API XML files is incorrect though, it needs to
point into the build tree.

This fixes use of the run script for building libvirt-python, ex:

  /path/to/libvirt.git/run ./setup.py build

Signed-off-by: Cole Robinson <crobinso@redhat.com>
---
 src/libvirt-lxc.pc.in  | 2 +-
 src/libvirt-qemu.pc.in | 2 +-
 src/libvirt.pc.in      | 6 +++---
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/libvirt-lxc.pc.in b/src/libvirt-lxc.pc.in
index 27e3de4244..9243bcf903 100644
--- a/src/libvirt-lxc.pc.in
+++ b/src/libvirt-lxc.pc.in
@@ -9,7 +9,7 @@ includedir=@abs_top_builddir@/include
 source_includedir=@abs_top_srcdir@/include
 datarootdir=@abs_top_builddir@
 
-libvirt_lxc_api=@datadir@/docs/libvirt-lxc-api.xml
+libvirt_lxc_api=@abs_top_builddir@/docs/libvirt-lxc-api.xml
 
 Name: libvirt
 Version: @VERSION@
diff --git a/src/libvirt-qemu.pc.in b/src/libvirt-qemu.pc.in
index 3eb9ad4bc1..a4f9e65c03 100644
--- a/src/libvirt-qemu.pc.in
+++ b/src/libvirt-qemu.pc.in
@@ -9,7 +9,7 @@ includedir=@abs_top_builddir@/include
 source_includedir=@abs_top_srcdir@/include
 datarootdir=@abs_top_builddir@
 
-libvirt_qemu_api=@datadir@/docs/libvirt-qemu-api.xml
+libvirt_qemu_api=@abs_top_builddir@/docs/libvirt-qemu-api.xml
 
 Name: libvirt
 Version: @VERSION@
diff --git a/src/libvirt.pc.in b/src/libvirt.pc.in
index 6ff1ce6272..a16e455210 100644
--- a/src/libvirt.pc.in
+++ b/src/libvirt.pc.in
@@ -9,12 +9,12 @@ includedir=@abs_top_builddir@/include
 source_includedir=@abs_top_srcdir@/include
 datarootdir=@abs_top_builddir@
 
-libvirt_api=@datadir@/docs/libvirt-api.xml
+libvirt_api=@abs_top_builddir@/docs/libvirt-api.xml
 
 # For compat with older versions - also present
 # in libvirt-qemu.pc / libvirt-lxc.pc
-libvirt_qemu_api=@datadir@/docs/libvirt-qemu-api.xml
-libvirt_lxc_api=@datadir@/docs/libvirt-lxc-api.xml
+libvirt_qemu_api=@abs_top_builddir@/docs/libvirt-qemu-api.xml
+libvirt_lxc_api=@abs_top_builddir@/docs/libvirt-lxc-api.xml
 
 Name: libvirt
 Version: @VERSION@
-- 
2.23.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] Fix src/ pkg-config API XML paths
Posted by Daniel P. Berrangé 4 years, 4 months ago 
On Mon, Dec 09, 2019 at 09:02:56AM -0500, Cole Robinson wrote:
> The .pc files in src/ are intended for use with the ./run script,
> to ease building bindings against an uninstalled libvirt build.
> The pointer to the API XML files is incorrect though, it needs to
> point into the build tree.
> 
> This fixes use of the run script for building libvirt-python, ex:
> 
>   /path/to/libvirt.git/run ./setup.py build
> 
> Signed-off-by: Cole Robinson <crobinso@redhat.com>
> ---
>  src/libvirt-lxc.pc.in  | 2 +-
>  src/libvirt-qemu.pc.in | 2 +-
>  src/libvirt.pc.in      | 6 +++---
>  3 files changed, 5 insertions(+), 5 deletions(-)

Sigh, I must have been blind when doing

ommit 54fff1856b9c9854fc3f649a52103e30a116c896
Author: Daniel P. Berrangé <berrange@redhat.com>
Date:   Mon Dec 2 13:18:01 2019 +0000

    src: fix pkg-config path for API XML files
    

/me wonders why we have a 'datarootdir' parameter in the .pc
files at all ? Wonder if this was a typo and meant to be
'datadir', or were the variable all meant to reference
'datarootdir' instead of 'datadir'

eg libvirt_lxc_api=${datarootdir}/docs/libvirt-lxc-api.xml


> 
> diff --git a/src/libvirt-lxc.pc.in b/src/libvirt-lxc.pc.in
> index 27e3de4244..9243bcf903 100644
> --- a/src/libvirt-lxc.pc.in
> +++ b/src/libvirt-lxc.pc.in
> @@ -9,7 +9,7 @@ includedir=@abs_top_builddir@/include
>  source_includedir=@abs_top_srcdir@/include
>  datarootdir=@abs_top_builddir@
>  
> -libvirt_lxc_api=@datadir@/docs/libvirt-lxc-api.xml
> +libvirt_lxc_api=@abs_top_builddir@/docs/libvirt-lxc-api.xml
>  
>  Name: libvirt
>  Version: @VERSION@
> diff --git a/src/libvirt-qemu.pc.in b/src/libvirt-qemu.pc.in
> index 3eb9ad4bc1..a4f9e65c03 100644
> --- a/src/libvirt-qemu.pc.in
> +++ b/src/libvirt-qemu.pc.in
> @@ -9,7 +9,7 @@ includedir=@abs_top_builddir@/include
>  source_includedir=@abs_top_srcdir@/include
>  datarootdir=@abs_top_builddir@
>  
> -libvirt_qemu_api=@datadir@/docs/libvirt-qemu-api.xml
> +libvirt_qemu_api=@abs_top_builddir@/docs/libvirt-qemu-api.xml
>  
>  Name: libvirt
>  Version: @VERSION@
> diff --git a/src/libvirt.pc.in b/src/libvirt.pc.in
> index 6ff1ce6272..a16e455210 100644
> --- a/src/libvirt.pc.in
> +++ b/src/libvirt.pc.in
> @@ -9,12 +9,12 @@ includedir=@abs_top_builddir@/include
>  source_includedir=@abs_top_srcdir@/include
>  datarootdir=@abs_top_builddir@
>  
> -libvirt_api=@datadir@/docs/libvirt-api.xml
> +libvirt_api=@abs_top_builddir@/docs/libvirt-api.xml
>  
>  # For compat with older versions - also present
>  # in libvirt-qemu.pc / libvirt-lxc.pc
> -libvirt_qemu_api=@datadir@/docs/libvirt-qemu-api.xml
> -libvirt_lxc_api=@datadir@/docs/libvirt-lxc-api.xml
> +libvirt_qemu_api=@abs_top_builddir@/docs/libvirt-qemu-api.xml
> +libvirt_lxc_api=@abs_top_builddir@/docs/libvirt-lxc-api.xml
>  
>  Name: libvirt
>  Version: @VERSION@
> -- 
> 2.23.0
> 
> --
> libvir-list mailing list
> libvir-list@redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] Fix src/ pkg-config API XML paths
Posted by Cole Robinson 4 years, 4 months ago 
On 12/9/19 9:08 AM, Daniel P. Berrangé wrote:
> On Mon, Dec 09, 2019 at 09:02:56AM -0500, Cole Robinson wrote:
>> The .pc files in src/ are intended for use with the ./run script,
>> to ease building bindings against an uninstalled libvirt build.
>> The pointer to the API XML files is incorrect though, it needs to
>> point into the build tree.
>>
>> This fixes use of the run script for building libvirt-python, ex:
>>
>>   /path/to/libvirt.git/run ./setup.py build
>>
>> Signed-off-by: Cole Robinson <crobinso@redhat.com>
>> ---
>>  src/libvirt-lxc.pc.in  | 2 +-
>>  src/libvirt-qemu.pc.in | 2 +-
>>  src/libvirt.pc.in      | 6 +++---
>>  3 files changed, 5 insertions(+), 5 deletions(-)
> 
> Sigh, I must have been blind when doing
> 
> ommit 54fff1856b9c9854fc3f649a52103e30a116c896
> Author: Daniel P. Berrangé <berrange@redhat.com>
> Date:   Mon Dec 2 13:18:01 2019 +0000
> 
>     src: fix pkg-config path for API XML files
>     
> 
> /me wonders why we have a 'datarootdir' parameter in the .pc
> files at all ? Wonder if this was a typo and meant to be
> 'datadir', or were the variable all meant to reference
> 'datarootdir' instead of 'datadir'
> 
> eg libvirt_lxc_api=${datarootdir}/docs/libvirt-lxc-api.xml
> 

Sure I can do that. Or push this and send a follow up patch removing the
unused variables, unless we expect apps could be consuming them in some
way? Your call

Thanks,
Cole

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] Fix src/ pkg-config API XML paths
Posted by Daniel P. Berrangé 4 years, 4 months ago 
On Mon, Dec 09, 2019 at 07:26:13PM -0500, Cole Robinson wrote:
> On 12/9/19 9:08 AM, Daniel P. Berrangé wrote:
> > On Mon, Dec 09, 2019 at 09:02:56AM -0500, Cole Robinson wrote:
> >> The .pc files in src/ are intended for use with the ./run script,
> >> to ease building bindings against an uninstalled libvirt build.
> >> The pointer to the API XML files is incorrect though, it needs to
> >> point into the build tree.
> >>
> >> This fixes use of the run script for building libvirt-python, ex:
> >>
> >>   /path/to/libvirt.git/run ./setup.py build
> >>
> >> Signed-off-by: Cole Robinson <crobinso@redhat.com>
> >> ---
> >>  src/libvirt-lxc.pc.in  | 2 +-
> >>  src/libvirt-qemu.pc.in | 2 +-
> >>  src/libvirt.pc.in      | 6 +++---
> >>  3 files changed, 5 insertions(+), 5 deletions(-)
> > 
> > Sigh, I must have been blind when doing
> > 
> > ommit 54fff1856b9c9854fc3f649a52103e30a116c896
> > Author: Daniel P. Berrangé <berrange@redhat.com>
> > Date:   Mon Dec 2 13:18:01 2019 +0000
> > 
> >     src: fix pkg-config path for API XML files
> >     
> > 
> > /me wonders why we have a 'datarootdir' parameter in the .pc
> > files at all ? Wonder if this was a typo and meant to be
> > 'datadir', or were the variable all meant to reference
> > 'datarootdir' instead of 'datadir'
> > 
> > eg libvirt_lxc_api=${datarootdir}/docs/libvirt-lxc-api.xml
> > 
> 
> Sure I can do that. Or push this and send a follow up patch removing the
> unused variables, unless we expect apps could be consuming them in some
> way? Your call

Just push this immediate fix to start with.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] Fix src/ pkg-config API XML paths
Posted by Daniel P. Berrangé 4 years, 4 months ago 
On Mon, Dec 09, 2019 at 09:02:56AM -0500, Cole Robinson wrote:
> The .pc files in src/ are intended for use with the ./run script,
> to ease building bindings against an uninstalled libvirt build.
> The pointer to the API XML files is incorrect though, it needs to
> point into the build tree.
> 
> This fixes use of the run script for building libvirt-python, ex:
> 
>   /path/to/libvirt.git/run ./setup.py build
> 
> Signed-off-by: Cole Robinson <crobinso@redhat.com>
> ---
>  src/libvirt-lxc.pc.in  | 2 +-
>  src/libvirt-qemu.pc.in | 2 +-
>  src/libvirt.pc.in      | 6 +++---
>  3 files changed, 5 insertions(+), 5 deletions(-)

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


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

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

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.