Re: [PATCH spice-streaming-agent v2 1/5] Install udev rule

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

 



> 
> On Mon, May 21, 2018 at 11:45:27AM +0100, Frediano Ziglio wrote:
> > The udev rule is used to do some action when the device is added to the
> > system. Current rule change the permission of the special file to allow to
> > open it by any user.
> 
> I thought this was a temporary 'hack' until we have a better way of
> handling this?
> 

Yes, you can note the rule is changed in a following patch.

> > Some systems use /lib/udev while others use /usr/lib/udev.
> > Allow to specify the full path to support both type of systems.
> 
> Maybe $ pkg-config udev --variable udevdir
> /usr/lib/udev
> could be used for that?
> 

I suppose we can use pkg-config to get the default.
About rpm and spec file is not clear which dependency to add to
get the pkg-config file. Maybe "pkgconfig(udev)" ?

> Christophe
> 

I think something like this could do (tested):


diff --git a/configure.ac b/configure.ac
index a75f22c..4e4f5c7 100644
--- a/configure.ac
+++ b/configure.ac
@@ -52,10 +52,12 @@ PKG_CHECK_MODULES(JPEG, libjpeg, , [
 ])
 AC_SUBST(JPEG_LIBS)

+PKG_CHECK_VAR([UDEVRULESDIR], [udev], [udevdir],
+[UDEVRULESDIR="$UDEVRULESDIR/rules.d"], [UDEVRULESDIR=/usr/lib/udev/rules.d])
+
 AC_ARG_WITH(udevrulesdir,
     [AS_HELP_STRING([--with-udevrulesdir=DIR], [udev rules.d directory])],
-    [UDEVRULESDIR="$withval"],
-    [UDEVRULESDIR=/usr/lib/udev/rules.d]
+    [UDEVRULESDIR="$withval"]
 )
 AC_SUBST(UDEVRULESDIR)

diff --git a/spice-streaming-agent.spec.in b/spice-streaming-agent.spec.in
index 5f4eec9..801e81e 100644
--- a/spice-streaming-agent.spec.in
+++ b/spice-streaming-agent.spec.in
@@ -1,5 +1,3 @@
-%{!?_udevrulesdir:%define _udevrulesdir /usr/lib/udev/rules.d}
-
 Name:           spice-streaming-agent
 Version:        @VERSION@
 Release:        1%{?dist}
@@ -13,6 +11,7 @@ BuildRequires:  libX11-devel libXfixes-devel
 BuildRequires:  libjpeg-turbo-devel
 BuildRequires:  catch-devel
 BuildRequires:  systemd-devel
+BuildRequires:  pkgconfig(udev)
 # we need /usr/sbin/semanage program which is available on different
 # packages depending on distribution
 Requires(post): /usr/sbin/semanage
@@ -34,7 +33,7 @@ agent plugins.
 %setup -q

 %build
-%configure --enable-tests --with-udevrulesdir=%{_udevrulesdir}
+%configure --enable-tests %{?_udevrulesdir:--with-udevrulesdir=%{_udevrulesdir}}
 make %{?_smp_mflags} V=1

 %check


Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]