data: elm: images: fix double data file definitions leading to races
authorStefan Schmidt <stefan@osg.samsung.com>
Wed, 28 Sep 2016 12:33:18 +0000 (14:33 +0200)
committerStefan Schmidt <stefan@osg.samsung.com>
Wed, 28 Sep 2016 12:33:18 +0000 (14:33 +0200)
For a while now I had strange races during make install on Jenkins where some
image files either already existed or chmod was called on files not there yet.

It took some digging but it turns out commit
8dcd5207cc61e604efc942a62137bdeda4060aa0 broke this as a side effect. The black
magic that already installed the files JP refers to is sitting in
data/Makefile.am. It is plain autofoo stuff and just includes the images
Makefile to get the list of files.

JP was correct though that the glayer files have not been installed into the
correct subdir. Fixing this as well here.

data/Makefile.am
data/elementary/images/Makefile.am

index 1df0122..8645929 100644 (file)
@@ -200,7 +200,7 @@ include elementary/images/Makefile.am
 elementaryimagesdir = $(datadir)/elementary/images
 elementaryimages_DATA = $(elementary_images_files)
 
-elementaryimages_glayerdir = $(datadir)/elementary/images
+elementaryimages_glayerdir = $(datadir)/elementary/images/g_layer
 elementaryimages_glayer_DATA = $(elementary_images_glayer_files)
 
 EXTRA_DIST += $(elementary_images_files) $(elementary_images_glayer_files)
index 5c0f0eb..9909d82 100644 (file)
@@ -88,11 +88,3 @@ elementary_images_glayer_files = \
        elementary/images/g_layer/flick_5.png \
        elementary/images/g_layer/long_tap_5.png \
        elementary/images/g_layer/tap_4.png
-
-elementary_images_filesdir = $(datadir)/elementary/images
-elementary_images_files_DATA = $(elementary_images_files)
-
-elementary_images_glayer_filesdir = $(datadir)/elementary/images/g_layer
-elementary_images_glayer_files_DATA = $(elementary_images_glayer_files)
-
-EXTRA_DIST += $(elementary_images_files_DATA) $(elementary_images_glayer_files_DATA)