[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: include_expsyms clobbers user files.
From: |
Ralf Wildenhues |
Subject: |
Re: include_expsyms clobbers user files. |
Date: |
Wed, 28 Sep 2005 11:03:49 +0200 |
User-agent: |
Mutt/1.5.11 |
Hi Peter,
* Peter Ekberg wrote on Tue, Sep 27, 2005 at 02:13:46PM CEST:
>
> (Using a real MUA for a change...)
:-)
> * Ralf Wildenhues wrote on Sunday, September 25, 2005 15:10 CEST:
> >
> > That's just because you try to avoid any extra copy, which requires
> > quite some logic, as seen in your proposed patch.
>
> I you don't mind the extra copy, here's a patch which does that instead.
Hmm. The improved simplicity is very appealing. However, I think the
*.uexp file should be removed after linking (see $delfiles).
(The fact that we don't fail on write errors is of course an issue in
both cases; lets worry about this later, not now.)
Also, you should make this conditional on nonempty $include_expsyms;
no need to punish all users here.
> > > Ok, here's a patch. I renamed the orig_export_symbols variable
> > > as it's now not always the original export_symbols value...
> >
> > I tried hard to find an error in the patch, but failed.
> > Please install, I think it's ok.
>
> Yes, I tested thoroughly, it was a mess.
I'm sorry.
> * libltdl/config/ltmain.m4sh (func_mode_link): Don't modify
> user provided symbols file.
Well, which approach do you like better? I'd have a slight preference
for this one, but don't care too much.
Cheers,
Ralf
- RE: include_expsyms clobbers user files., Peter Ekberg, 2005/09/22
- Re: include_expsyms clobbers user files., Ralf Wildenhues, 2005/09/25
- Re: include_expsyms clobbers user files., Peter Ekberg, 2005/09/27
- Re: include_expsyms clobbers user files.,
Ralf Wildenhues <=
- Re: include_expsyms clobbers user files., Peter Ekberg, 2005/09/28
- Re: include_expsyms clobbers user files., Ralf Wildenhues, 2005/09/29
- Re: include_expsyms clobbers user files., Peter Ekberg, 2005/09/29
- Re: include_expsyms clobbers user files., Ralf Wildenhues, 2005/09/29
- Re: include_expsyms clobbers user files., Ralf Wildenhues, 2005/09/30