[PATCH] meson: src: Fix DESTDIR handling while creating dirs

[Pavel Hrdina]

On Wed, Sep 09, 2020 at 05:06:19PM +0200, Jan Kiszka wrote:
> On 09.09.20 16:38, Erik Skultety wrote:
> > On Mon, Sep 07, 2020 at 11:25:34PM +0200, Jan Kiszka wrote:
> >> From: Jan Kiszka <jan.kiszka at siemens.com>
> >>
> >> If the target path contains a link with an absolute path (e.g.
> >> /var/run -> /run), makedirs will target the wrong location. Resolve the
> >> path first, then append DESTDIR again if needed.
> >
> > Can you elaborate on this? os.makedirs should follow the symlinks before
> > actually creating the directory hierarchy, so I'm failing to see the problem
> > this patch is trying to address.
> >
> 
> mkdir -p /my/destdir/var /my/destdir/run
> ln -s /run /my/destdir/var/run
> meson build --prefix=/usr
> DESTDIR=/my/destdir ninja -C build install
> 
> -> PermissionError: [Errno 13] Permission denied:
> '/my/destdir/var/run/libvirt'

I would say don't do that as that's not purpose of DESTDIR and running
libvirt from the DESTDIR installation will not work as it will not find
all the required files.

What is you use-case or what are you trying to achieve?

Pavel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20200909/c611d2ab/attachment-0001.sig>