Re: [libvirt PATCH v2 1/1] meson: Drop RPATH usage

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 19, 2020 at 10:30 AM Andrea Bolognani <abologna@xxxxxxxxxx> wrote:
>
> On Wed, 2020-08-19 at 15:10 +0200, Pavel Hrdina wrote:
> > On Wed, Aug 19, 2020 at 02:58:07PM +0200, Andrea Bolognani wrote:
> > > > > We can add an option like it was proposed in V1 but with the following
> > > > > changes. In meson.build we would have this:
> > > > >
> > > > >     if get_option('rpath')
> > > > >       libvirt_rpath = libdir
> > > > >     else
> > > > >       libvirt_rpath = ''
> > > > >     endif
> > > > >
> > > > > and all places with install_rpath would use libvirt_rpath instead of
> > > > > libdir directly and we would not have to have the craze if-else.
> > > >
> > > > Yeah that would be simpler.
> > >
> > > ... I don't think this would work: using
> > >
> > >   install_rpath: ''
> > >
> > > will result in an empty RPATH being set, not in RPATH being missing.
> > > So it's a different behavior than the one we want.
> >
> > I specifically tried this to make sure it works and also this is from
> > meson code:
> >
> >     self.install_rpath = kwargs.get('install_rpath', '')
> >
> > so yes it works.
>
> I tried again and I stand corrected :)
>
> I'll send a new patch in a minute.
>

Fedora's libtool is also slightly tweaked to minimize RPATH stuff
being set. Across the board, generally RPATH values are set to ""
(CMake, libtool, etc.), though I think Meson doesn't have an
equivalent to set globally at the %meson macro level...

-- 
真実はいつも一つ!/ Always, there's only one truth!





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux