Re: [PATCH]: Re: [Nautilus-list] Re: must not ship Nautilus 1.0.4 until we fix FAM support

[Yoann Vandoorselaere <yoann mandrakesoft com>]

Maciej Stachowiak <mjs noisehavoc org> writes:

> On 17Jun2001 01:41PM (+0200), Yoann Vandoorselaere wrote:
> > 
> > [Put Ramiro in CC for approval]
> > 
> > The problem was the wrong assumption that g_module_build_path() would 
> > lookup the module for you. It does not.   
> > 
> > Can I commit ?
> > 
> 
> Hi Yoann,
> 
> Your patch is not correct. g_module_build_path will result in a name
> like "libfam.so", which should get looked up in LD_LIBRARY_PATH and
> the system hardcoded library locations by g_module_open (). 

This is wrong.
In several distribution, there is only a .so if you installed the
developpement package. This patch permit to lookup libfam.so.*.

> Your patch
> will look up the library only in "/usr" and "/usr/lib", 

You mean /usr/local/lib & /usr/lib

> which is wrong
> when you have fam built in a custom prefix (as for example I do)

This imply adding a configuration option to Nautilus to tell where
the fam library is...

> , in
> addition to unnecessarily scanning a bunch of huge directories.

We don't have the choise. Please read the rest of the thread.
We need this lookup.

> Further, Nautilus is meant to work with a particular version of FAM,
> which appears as libfam.so.0 on Linux, and your patch just checks for
> "libfam" as a prefix, so it could find "libfam.so.1" which would not
> work, or heck, even "libfamily.so".

libfam.so will not work... As far as I remember Darin said in a previous 
mail (search the archives) that the lookup function would be the good way 
to go.

You can't assume a given version will be installed on a system,
so we have to rely on binary compatibility.

> A better fix might be to pass "libfam.so.0" instead of "fam" to
> g_module_open, but only on systems that use this style of shared
> library naming (I'm not sure what a good way to tell is).

Yes, and as soon the fam will be distributed using package and that
FAM soname will change, Nautilus will broke. No, I seriously think
we should rely on binary compatibility, which will give us more chance
of having Nautilus using FAM on whatever system than otherwise...

So, what do we do ?

-- 
Yoann Vandoorselaere | I worry about my child and the Internet all the time,
MandrakeSoft         | even though she's too  young to have logged on yet.
                     | Here's what I worry about. I worry that 10 or 15  years
                     | from now, she will come to me and say 'Daddy, where were
                     | you when they  took freedom of the press away from the
                     | Internet?'  -- Mike Godwin