On Mon, 2018-03-26 at 18:45 -0600, Keith Packard wrote:
> Adam Jackson <a...@redhat.com> writes:
> 
> > Signed-off-by: Adam Jackson <a...@redhat.com>
> > +if get_option('suid_wrapper')
> > +    executable('Xorg.wrap',
> > +        'xorg-wrapper.c',
> > +        include_directories: [inc, xorg_inc],
> > +        dependencies: xorg_deps,
> > +        c_args: xorg_c_args,
> > +        install: true,
> > +        install_dir: get_option('libexecdir'),
> > +        # install_mode: ['r-sr-xr-x', 0, 0],
> 
> I assume your package files fix the permissions? If I'm reading the
> autotools version, there's a chown and chmod for that?

Yes, we just set the thing to be suid. Annoyingly install_mode: works
for the install_data() verb but not for executable(). There's a bug:

https://github.com/mesonbuild/meson/issues/3328

Probably this path should add a warning() to alert the builder?

- ajax
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to