wx build process improvements, including added checks and outdated changes after...
authorkevino@webkit.org <kevino@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Dec 2007 17:36:57 +0000 (17:36 +0000)
committerkevino@webkit.org <kevino@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Dec 2007 17:36:57 +0000 (17:36 +0000)
Reviewed by Alp Toker

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

WebKitTools/ChangeLog
WebKitTools/wx/build-wxwebkit

index 5baf7de..800e093 100644 (file)
@@ -1,3 +1,17 @@
+2007-12-18  Kevin Ollivier  <kevino@theolliviers.com>
+
+        Reviewed by Alp Toker.
+
+        Build related fixes.
+
+        * wx/build-wxwebkit:
+        - Check to make sure the user is running a supported wx port
+        - More robust checks for Cygwin
+        - Only run install-unix-extras on Mac, it's not the right solution
+          for Linux distros.
+        - Remove outdated Linux instructions after successful build
+        - Clean Bakefile-generated files during a clean operation
+
 2007-12-18  Darin Adler  <darin@apple.com>
 
         * Scripts/make-js-test-wrappers: Don't generate a wrapper for select-options-remove.js.
index a2cb426..71b8432 100755 (executable)
@@ -76,6 +76,13 @@ else
         echo "Make sure wx-config is on your PATH and points to a valid wxWidgets installation."
         exit 1
     fi
+    
+    CONFIG=`wx-config --selected-config`
+    if [ "${CONFIG:0:4}" != "gtk2" -a "${CONFIG:0:3}" != "mac" ]; then
+        echo "ERROR: This configuration '$CONFIG' is not currently supported by wxWebKit. Please use the win, mac, or gtk2 port depending on your platform."
+        exit 1
+    fi
+    
     if [ ! -d "$WX_PREFIX/share/bakefile" ]; then
         echo "ERROR: wxWidgets installation does not have wxpresets."
         echo "wx-config is pointing to an installation that does not have Bakefile presets installed. Run `wx-config --version` to ensure your wxWidgets is of version 2.8+."
@@ -188,7 +195,7 @@ else
         
         # since the buildbot will wipe the build tree clean sometimes, we need to reinstall
         # the dependencies if they aren't installed.
-        if [ ! "${OSTYPE:0:6}" == "cygwin" ]; then
+        if [ "${OSTYPE:0:6}" == "darwin" ]; then
             $WK_ROOT/WebKitTools/wx/install-unix-extras
         fi
         
@@ -220,10 +227,10 @@ if [ $do_build == 1 -o $do_clean == 1 ]; then
         WXGC_DEFINE="-DWTF_USE_WXGC=1"
     fi
     WX_EXT=
-    if [ "$OSTYPE" == "cygwin" -a $wxdebug == 1 ]; then
+    if [ "${OSTYPE:0:6}" == "cygwin" -a $wxdebug == 1 ]; then
         WX_EXT=d
     fi
-    if [ "$OSTYPE" == "cygwin" -a $wxpython == 1 ]; then
+    if [ "${OSTYPE:0:6}" == "cygwin" -a $wxpython == 1 ]; then
         other_args=WX_PYTHON=1
         WX_EXT=h
         wxdebug=1
@@ -313,10 +320,6 @@ if [ $do_build == 1 -o $do_clean == 1 ]; then
         echo ""
         echo "Next steps:"
         echo ""
-        if [ "${OSTYPE:0:5}" = "linux" ]; then
-            echo "-- Add /usr/local/lib and $WEBKITOUTPUTDIR to your LD_LIBRARY_PATH environment variable."
-            echo ""
-        fi
         echo "-- Run '$WK_ROOT/WebKitTools/Scripts/run-javascriptcore-tests --wx' to ensure JSCore tests pass."
         echo ""
         echo "-- Run $WEBKITOUTPUTDIR/$BROWSERAPP to test your wxWebKit build."
@@ -325,4 +328,9 @@ if [ $do_build == 1 -o $do_clean == 1 ]; then
     fi
 fi
 
+if [ $do_clean == 1 ]; then
+    cd $WK_ROOT/WebKit/wx
+    bakefile_gen --clean
+fi
+
 cd $olddir