Re: [Mlt-devel] Kdenlive build script

[Dan D. <da...@de...>]

On Sun, Nov 13, 2011 at 12:23 PM, j-b-m <j-...@us...> wrote:
> Hello Dan.
>
> I have updated the kdenlive build script that is hosted on mltframework.org to
> now use the kdenlive git repository instead of the expired svn one.
>
> I also made a change to set the KDEHOME environment variable, so that
> Kdenlive's config file is now saved inside the install folder instead of using
> the standard KDE config path, that should avoid cases where recompiling with
> the build script caused Kdenlive to us a previous install of MLT.

great improvement! The chunk below wil be executed for each subproject
and not just  kdenlive. It belongs in an if test "kdenlive" = "$1"

@@ -1135,6 +1135,18 @@
     fi
   else
     cmd make install || die "Unable to install $1"
+    if [ ! -d "$FINAL_INSTALL_DIR/.kde" ]; then
+      cmd mkdir "$FINAL_INSTALL_DIR/.kde"
+    fi
+    if test "$KDEHOME" != "" ; then
+      HOMEPATH="$KDEHOME"
+    else
+      HOMEPATH="$HOME/.kde"
+      if [ ! -d "$HOMEPATH" ]; then
+       HOMEPATH="$HOME/.kde4"
+      fi
+    fi
+    echo "KDE HOME FOLDER: $HOMEPATH"
   fi
   feedback_progress Done installing $1

----

I made this change as well as incremented the version and uploaded. Is
there a purpose of HOMEPATH other than feedback in the log?

> Can you put this new version online (or if you prefer we can host it on
> Kdenlive.org and you link to us - as you want) ?

Since I have multiple build scripts, some changes can be made to
multiple scripts, so I prefer to keep it on mltframework.org for now.

-- 
+-DRD-+