[Kdenlive-devel] Setting the paths for MLT?

[Mads Bondo Dydensborg]

søndag 28 September 2008 skrev jb:

> > Is there a way to override this? During install? As a command line
> > parameter?
> 

> I also just commited a change so that the renderer (kdenlive_render) is now 
> taken from the same folder as the running kdenlive instance.
> 
> If you still want to work on a patch, it's ok for me.

Hi jb.

Would you please consider this very small patch:
$ svn diff mainwindow.cpp
Index: mainwindow.cpp
===================================================================
--- mainwindow.cpp      (revision 2442)
+++ mainwindow.cpp      (working copy)
@@ -982,6 +982,13 @@

 void MainWindow::parseProfiles() {
     //kdDebug()<<" + + YOUR MLT INSTALL WAS FOUND IN: "<< MLT_PREFIX <<endl;
+    // If MLT_PREFIX is specified in the environment, it overrides the stored
+    // setting.
+    QString mltPrefix = getenv( "MLT_PREFIX" );
+    if ( mltPrefix != "" ) {
+      KdenliveSettings::setMltpath( mltPrefix + "/share/mlt/profiles/" );
+      KdenliveSettings::setRendererpath( mltPrefix + "/bin/inigo" );
+    }

     //KdenliveSettings::setDefaulttmpfolder();
     if (KdenliveSettings::mltpath().isEmpty()) {

AFAICT this works, and allows me to set MLT_PREFIX in the environment, and 
have kdenlive use it.

The drawback is that kdenlive stores this in its settings, but if it is only 
used from the wizard, not very many users should have problems with this.

Please let me know if you can accept it, as I need to update the wizard to use 
this. Most installs done by the wizard, using the append date options is 
broken, and I would really like to fix that :-)

Thanks

Mads

-- 
Mads Bondo Dydensborg   mads at dydensborg.dk   http://www.madsdydensborg.dk/

The US population is 4 % of the global one. They produce 23 % of the global 
pollution.