[Mesa-dev] [PATCH 1/2] bin/install_megadrivers: fix DESTDIR and -D*-path

Dylan Baker dylan at pnwbakers.com
Thu Apr 19 18:10:21 UTC 2018


Quoting Dylan Baker (2018-04-16 14:28:42)
> Quoting Dylan Baker (2018-04-13 08:46:46)
> > Quoting Dylan Baker (2018-04-09 14:02:51)
> > > This fixes -Ddri-drivers-path, -Dvdpau-libs-path, etc. with DESTDIR when
> > > those paths are absolute. Currently due to the way python's os.path.join
> > > handles absolute paths these will ignore DESTDIR, which is bad. This
> > > fixes them to be relative to DESTDIR if that is set.
> > > 
> > > Fixes: 3218056e0eb375eeda470058d06add1532acd6d4
> > >        ("meson: Build i965 and dri stack")
> > > Signed-off-by: Dylan Baker <dylan.c.baker at intel.com>
> > > ---
> > >  bin/install_megadrivers.py | 8 ++++++--
> > >  1 file changed, 6 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/bin/install_megadrivers.py b/bin/install_megadrivers.py
> > > index 7931a544bd2..c04a2a3eb34 100755
> > > --- a/bin/install_megadrivers.py
> > > +++ b/bin/install_megadrivers.py
> > > @@ -1,6 +1,6 @@
> > >  #!/usr/bin/env python
> > >  # encoding=utf-8
> > > -# Copyright © 2017 Intel Corporation
> > > +# Copyright © 2017-2018 Intel Corporation
> > >  
> > >  # Permission is hereby granted, free of charge, to any person obtaining a copy
> > >  # of this software and associated documentation files (the "Software"), to deal
> > > @@ -35,7 +35,11 @@ def main():
> > >      parser.add_argument('drivers', nargs='+')
> > >      args = parser.parse_args()
> > >  
> > > -    to = os.path.join(os.environ.get('MESON_INSTALL_DESTDIR_PREFIX'), args.libdir)
> > > +    if os.path.isabs(args.libdir):
> > > +        to = os.path.join(os.environ.get('DESTDIR', '/'), args.libdir[1:])
> > > +    else:
> > > +        to = os.path.join(os.environ['MESON_INSTALL_DESTDIR_PREFIX'], args.libdir)
> > > +
> > >      master = os.path.join(to, os.path.basename(args.megadriver))
> > >  
> > >      if not os.path.exists(to):
> > > -- 
> > > 2.17.0
> > > 
> > 
> > ping. The first patch is really needed for 18.0.1
> > 
> > Dylan
> 
> anybody?
> 

I'm pushing these today if no one speaks up.

Dylan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: signature
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180419/59d17182/attachment-0001.sig>


More information about the mesa-dev mailing list