[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: |
Thu, 22 Sep 2005 12:43:46 +0200 |
* Ralf Wildenhues wrote on Thursday, September 22, 2005 10:06 CEST:
> Hi Peter,
>
> * Peter Ekberg wrote on Thu, Sep 22, 2005 at 10:00:32AM CEST:
> > * Ralf Wildenhues wrote on Wednesday, September 21, 2005 22:37 CEST:
> > > * Peter Ekberg wrote on Mon, Sep 19, 2005 at 09:05:04PM CEST:
>
> > > > Thinking about it further, include_expsyms is perhaps not
> > > > buggy with skipped exports, as then the needed symbols perhaps
> > > > get exported anyway with the export_symbols_cmds functionality.
> > > > A check wouldn't hurt though...
>
> > 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.
> > So, zap it? Or is include_expsyms somehow a hook for "outside
> > users"?
>
> To be honest, I don't know. But it's nicely symmetrical with
> exclude_expsyms, and the latter is useful. I can imagine situations
> where the former can be useful.
Ok, here's a patch. I renamed the orig_export_symbols variable
as it's now not always the original export_symbols value...
* libltdl/config/ltmain.m4sh (func_mode_link): Don't modify
user provided symbols file when $include_expsyms is active.
Rename orig_export_symbols variable to filt_export_symbols.
Cheers,
Peter
head-leave-export-symbols-file-alone.patch
Description: head-leave-export-symbols-file-alone.patch
- RE: include_expsyms clobbers user files.,
Peter Ekberg <=
- 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, 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