[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: include_expsyms clobbers user files.
From: |
Peter Ekberg |
Subject: |
Re: include_expsyms clobbers user files. |
Date: |
Tue, 27 Sep 2005 14:13:46 +0200 |
User-agent: |
Mutt/1.5.10i |
Hi!
(Using a real MUA for a change...)
* Ralf Wildenhues wrote on Sunday, September 25, 2005 15:10 CEST:
> * Peter Ekberg wrote on Thu, Sep 22, 2005 at 12:43:46PM CEST:
> > * Ralf Wildenhues wrote on Thursday, September 22, 2005 10:06 CEST:
> > > * Peter Ekberg wrote on Thu, Sep 22, 2005 at 10:00:32AM CEST:
> > > > But looking closer, include_expsyms is a disaster area. Since it
> > > > isn't used, my vote is to zap it completely.
> > > >
> > > > The problem is that it potentially clobbers user provided files,
> > > > so that building repeatedly will eat up all your disk space
> > > > eventually (but that isn't the worst problem with clobbering
> > > > user provided files of course).
> > >
> > > Why not make a copy of the user-provided file and append to that?
> > > C'mon, that isn't rocket science. Unless there are other reasons
> > > against include_expsyms, that is.
> >
> > Well, it seemed to complicate matters for little reason, that's all.
>
> 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.
> > 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.
* libltdl/config/ltmain.m4sh (func_mode_link): Don't modify
user provided symbols file.
Cheers,
Peterr
head-leave-export-symbols-file-alone-2.patch
Description: Text document
- 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 <=
- Re: include_expsyms clobbers user files., Ralf Wildenhues, 2005/09/28
- 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