[Ml-yokadi] update script path?

Sébastien Renard Sebastien.Renard at digitalfox.org
Thu Nov 12 07:53:22 CET 2009


Le mercredi 11 novembre 2009 15:36:48, Kartik Mistry a écrit :
> > I don't understand the issue. The path provided is computed to  be always
> > the good one (else if we have a bug :D). Why should we remove this
> > information ?
> 
> Thing is that, current path/update.py path will never be correct :)

The code is in the shareDirPath() function of utils.py module. It search if 
you are in a dev tarball or in installed package. 
The share directory is searched along the following path for installed 
version: 
  "/usr/share/yokadi",
  "/usr/local/share/yokadi",
  "/usr/local/yokadi/share/yokadi",
  "/opt/yokadi/share/yokadi"

But the given path in message is indeed false, it misses the update dir. 
Correct path is shareDir/*update*/update.py. That's fixed.

> > Moreover, the update folder path may differ from a package to another and
> > with source installation. We cannot have a fix path.
> 
> True.
> 
> > But we could add a link to the README.markdown file for further detail.
> 
> Putting update/README is enough. I'm putting REAME.Debian and adding
> db updating procedure in it.

I've added a symlink README => README.markdown as we did for main README file. 
Maybe we should only have README file and forget the markdown suffix ?

I also added a message to notify README existance when users have to upgrade 
his database.
 
> I've corrected some issues in manpages, will send patch in separate
> mail sometime in weekend..

Thanks. 

-- 
Sébastien


More information about the Ml-yokadi mailing list