[EFL][GTK] Install script not working on Debian 9 (testing) and Ubuntu 16.10
authorclopez@igalia.com <clopez@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 13 Aug 2016 15:32:43 +0000 (15:32 +0000)
committerclopez@igalia.com <clopez@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 13 Aug 2016 15:32:43 +0000 (15:32 +0000)
https://bugs.webkit.org/show_bug.cgi?id=160809

Reviewed by Carlos Garcia Campos.

* efl/install-dependencies: Factorize the previous logic for handling the php5/7
case, and use it also for the new package names of libpng and libgeoclue.
* gtk/install-dependencies: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@204445 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/efl/install-dependencies
Tools/gtk/install-dependencies

index 0a85909..d6469c8 100644 (file)
@@ -1,3 +1,14 @@
+2016-08-13  Carlos Alberto Lopez Perez  <clopez@igalia.com>
+
+        [EFL][GTK] Install script not working on Debian 9 (testing) and Ubuntu 16.10
+        https://bugs.webkit.org/show_bug.cgi?id=160809
+
+        Reviewed by Carlos Garcia Campos.
+
+        * efl/install-dependencies: Factorize the previous logic for handling the php5/7
+        case, and use it also for the new package names of libpng and libgeoclue.
+        * gtk/install-dependencies: Ditto.
+
 2016-08-12  Gyuyoung Kim  <gyuyoung.kim@webkit.org>
 
         Skip to check directories existence in build-webkit
index 77880ad..c472e9b 100755 (executable)
@@ -48,6 +48,16 @@ function checkInstaller {
     printNotSupportedMessageAndExit
 }
 
+# If the package $1 is available, prints it. Otherwise prints $2.
+# Useful for handling when a package is renamed on new versions of Debian/Ubuntu.
+function aptIfElse {
+    if apt-cache show $1 &>/dev/null; then
+        echo $1
+    else
+        echo $2
+    fi
+}
+
 function installDependenciesWithApt {
     # These are dependencies necessary for building WebKitEFL.
     apt-get install \
@@ -67,7 +77,7 @@ function installDependenciesWithApt {
         libffi-dev \
         libfreetype6-dev \
         libgcrypt11-dev \
-        libgeoclue-dev \
+        $(aptIfElse libgeoclue-2-dev libgeoclue-dev) \
         libgif-dev \
         libgl1-mesa-dev \
         libgnutls28-dev \
@@ -80,7 +90,7 @@ function installDependenciesWithApt {
         libmpg123-dev \
         liborc-0.4-dev \
         libp11-kit-dev \
-        libpng12-dev \
+        $(aptIfElse libpng-dev libpng12-dev) \
         libpulse-dev \
         libsqlite3-dev \
         libssl-dev \
@@ -110,14 +120,9 @@ function installDependenciesWithApt {
     # These are dependencies necessary for running tests.
     apt-get install \
         apache2 \
+        $(aptIfElse libapache2-mod-php7.0 libapache2-mod-php5) \
         libruby \
         xvfb
-
-    if apt-cache show libapache2-mod-php7.0 &>/dev/null; then
-        apt-get install libapache2-mod-php7.0
-    else
-        apt-get install libapache2-mod-php5
-    fi
 }
 
 checkInstaller
index d85eff8..cb69d93 100755 (executable)
@@ -75,6 +75,16 @@ function installDependenciesWithBrew {
          xz
 }
 
+# If the package $1 is available, prints it. Otherwise prints $2.
+# Useful for handling when a package is renamed on new versions of Debian/Ubuntu.
+function aptIfElse {
+    if apt-cache show $1 &>/dev/null; then
+        echo $1
+    else
+        echo $2
+    fi
+}
+
 function installDependenciesWithApt {
     # These are dependencies necessary for building WebKitGTK+.
     packages=" \
@@ -95,7 +105,7 @@ function installDependenciesWithApt {
         libedit-dev \
         libenchant-dev \
         libfaad-dev \
-        libgeoclue-dev \
+        $(aptIfElse libgeoclue-2-dev libgeoclue-dev) \
         libgirepository1.0-dev \
         libgl1-mesa-dev \
         libgl1-mesa-glx \
@@ -109,7 +119,7 @@ function installDependenciesWithApt {
         libnotify-dev \
         libopus-dev \
         libpango1.0-dev \
-        libpng12-dev \
+        $(aptIfElse libpng-dev libpng12-dev) \
         libpulse-dev \
         librsvg2-dev \
         libsecret-1-dev \
@@ -135,6 +145,7 @@ function installDependenciesWithApt {
         curl \
         dbus-x11 \
         libapache2-mod-bw \
+        $(aptIfElse libapache2-mod-php7.0 libapache2-mod-php5) \
         libgpg-error-dev \
         psmisc \
         pulseaudio-utils \
@@ -198,14 +209,7 @@ function installDependenciesWithApt {
         git-svn \
         subversion"
 
-    if apt-cache show libapache2-mod-php7.0 &>/dev/null; then
-        packages="$packages libapache2-mod-php7.0"
-    else
-        packages="$packages libapache2-mod-php5"
-    fi
-
     apt-get install $packages
-
 }
 
 function installDependenciesWithPacman {