helpers: fix install race
authorRoss Burton <ross.burton@intel.com>
Thu, 12 Nov 2015 19:46:44 +0000 (19:46 +0000)
committerSebastian Dröge <sebastian@centricular.com>
Fri, 13 Nov 2015 08:40:44 +0000 (09:40 +0100)
The install hook needs to be a install-data-hook not an install-exec-hook as the
helpers are installed into helperdir which is considered data (only path
variables with "exec" in are considered executables).

The explicit dependency on install-helpersPROGRAMS was an attempt at solving
this, but this causes occasional races where install-helpersPROGRAMS can run
twice in parallel (once via install-all, once via the hook's dependency).

https://bugzilla.gnome.org/show_bug.cgi?id=758029

libs/gst/helpers/Makefile.am

index 0ae9ab0..2530a34 100644 (file)
@@ -23,7 +23,7 @@ gst_ptp_helper_CFLAGS = $(GST_OBJ_CFLAGS) $(GIO_CFLAGS)
 gst_ptp_helper_LDADD = $(GST_OBJ_LIBS) $(GIO_LIBS) $(CAP_LIBS)
 endif
 
-install-exec-hook: install-helpersPROGRAMS
+install-data-hook:
 if HAVE_PTP
 if HAVE_PTP_HELPER_SETUID
        - chown root $(DESTDIR)$(helpersdir)/gst-ptp-helper