Re: Language module support modified

Bram Moolenaar <[email protected]>
Newsgroups gmane.comp.tools.aap.devel
Message-ID <[email protected]>
Joerg Beyer wrote:

> adridg-FlD2LfDziEhmR6Xm/[email protected] schrieb am 30.09.03 11:55:20:
> > 
> > On Tue, 30 Sep 2003, Bram Moolenaar wrote:
> > > > Like someone else already pointed out, /usr/local is rather
> > > > unportable. I presume you mean "the OS-specific place for "extra
> > > > software"".
> > >
> > > And how would Aap find out where that is?
> > >
> > > I could add a variable in default.aap, but I found one place where the
> > > /usr/local/... directory is used without reading it: filetype detection.
> > > Perhaps this is something that must be done when installing Aap?
> > 
> > 
> > 	if os == "posix":
> > 		if test -d "/opt":
> > 			os_local_dir="/opt"
> > 		else:
> > 			os_local_dir="/usr/local"
> > 	else:
> > 		os_local_dir="C:/WINDOWS/SYSTEM"
> > 
> > That's a start, isn't it?
> 
> I would suggest to (at least additional) get this place from a
> configuration (maybe an env-var) - since many people are not willing
> to argue about the root directory of their installation, they have
> decided what is "right" a few years ago.

It's logical that a directory name that could differ from system to
system is configurable.  The main problem with this specific one is that
you can't look in a system-wide configuration file before you know where
they are located!

I don't like using environment variables, because each user would have
to set it.

The best method probably is to allow setting the directory when Aap is
installed, and use a sequence of checks for known places for the
default.

-- 
hundred-and-one symptoms of being an internet addict:
49. You never have to deal with busy signals when calling your ISP...because
    you never log off.

 /// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net   \\\
///          Creator of Vim - Vi IMproved -- http://www.Vim.org          \\\
\\\              Project leader for A-A-P -- http://www.A-A-P.org        ///
 \\\  Help AIDS victims, buy here: http://ICCF-Holland.org/click1.html  ///


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.