On Thu, Apr 13, 2006 at 10:28:59AM +0200, Jean-Marc Lasgouttes wrote: > > Hi Kayvan, > > Did you see this message? We need your input on why this special > cygwin code was needed...
All I remember is that it was needed at one time for the QT/Cygwin code I compiled from CVS (and it was in the instructions or README file for building QT/Cygwin at kde-cygwin.sf.net). I think if Abdel and others think that we can do without it, I'm all for it. I can't compile LyX right now on any of my platforms, but I have not had the time to fix the problems. My automated daily builds on my Redhat FC4 system are getting those damnable linker errors or the python lyx2lyx module issue (see ftp://ftp.sylvan.com/pub/lyx/devel/log for the latest). My Cygwin machine can not build nowadays due to some compiler errors due to the recent QT changes. I probably need to refresh my Cygwin and follow the latest Cygwin/QT/LyX build directions so I can get my Cygwin/Windows builds up to the latest. So, in short, I have no objections since I can't build right now anyways! :-) If anyone has any suggestions to help me get my daily builds working again, please feel free to email me. I'm going to upgrade that machine to Redhat FC5 in the next week or so, after which I will try again. ---Kayvan > To: lyx-devel@lists.lyx.org > From: Abdelrazak Younes <[EMAIL PROTECTED]> > Subject: [Patch] reduce linking time under windows (was Re: Question about > C++ warning ...) > Date: Wed, 12 Apr 2006 21:23:49 +0200 > User-Agent: Thunderbird 1.5 (Windows/20051201) > In-Reply-To: <[EMAIL PROTECTED]> > Xref: fantomas.inria.fr lyx-devel:80738 > > Abdelrazak Younes a écrit : > >Enrico Forestieri a écrit : > >>On Wed, Apr 12, 2006 at 02:45:00PM +0200, Abdelrazak Younes wrote: > >>>Enrico Forestieri a écrit : > >>>>>>That's -Wl,--export-all-symbols and simply passes > >>>>>>--export-all-symbols > >>>>>>to the linker. It does no good and simply will increase your linking > >>>>>>time. I suggest that you get rid of it. > >>>[..] > >>>>I was not talking about the warnings, sorry for the confusion. > >>>>What I said only applies to the final link step. I observed that > >>>>with -Wl,--export-all-symbols, the linking time noticeably increases. > >>>>Removing that option shortens the linking time for me without any > >>>>side effect. > >>>> > >>>>This is a sed command you can use to modify configure in order to > >>>>avoid "-Wl,--export-all-symbols" in LDFLAGS: > >>>> > >>>>sed ' # Omit -Wl,--export-all-symbols in LDFLAGS > >>>>/LDFLAGS=\"\$LDFLAGS > >>>>\$WIN32_LD_EXPORT_ALL_SYMBOLS\"/s/LDFLAGS/true\t\#LDFLAGS/' \ > >>>> ./configure > ./configure-new > >>>Wouldn't it be better to integrate that fix in the m4 macros? If in > >>>there that is... > >> > >>Perhaps... But I am not going to propose that, as I have to save > >>energy for other battles... ;-) > > > >Well, if it can help the production of lyx dynamically linked with Qt3 > >that would be great because as it is now the linking time is close to > >unbearable. I'll try to test if there's any improvement. > > It seems to :-) > ar+ranlib libqt2 was done in 10 minutes and less than 400 Megs. > final exe linking took less than 10 minutes and less than 400 Megs > > But, I had to modify the finale linking command because for whatever > reason it tried to link with "/usr/lib/libiconv.dll.a" which of course > does not exists. Correcting that to d:/mingw/lib/iconv.lib did the > trick! I am sure this is due to this patch or to something else. > > The resulting qt3 executable seems to work as usual, I can see no > difference. No need to say that this is a _huge_ improvement! > > As cited below, Enrico tested on cygwin and cygwin/X11 that this patch > does not cause any problem. So I would like to ask Angus principally his > opinion on this. If Kayvan could also tell us his reason for putting > that at the first place, that would be great. > > Abdel. > > >>Then, I don't know if this flag is going to help when dynamically > >>linking Qt, as I must link statically to it. For sure it doesn't > >>help when dinamically linking the X11 libs or any other library > >>(I just finished successfully building an X11 version of LyX/Cygwin > >>without that option and dynamically linking to libAiksaurus, > >>libaspell, libiconv, libXrandr, libXcursor, libXft, libXrender, > >>libfontconfig, libfreetype, libexpat, libXext, libSM, libICE, > >>libX11: The gain in linking time is awesome). > Index: config/cygwin.m4 > =================================================================== > --- config/cygwin.m4 (revision 13592) > +++ config/cygwin.m4 (working copy) > @@ -7,16 +7,6 @@ > cygwin* | mingw* | pw32* ) > # Export all symbols to Win32 DLL using MinGW 2.0 ld. > WIN32_LD_EXPORT_ALL_SYMBOLS='' > - AC_MSG_CHECKING([whether ld accepts --export-all-symbols]) > - if $LD --help 2>&1 | egrep 'export-all-symbols' > /dev/null; then > - WIN32_LD_EXPORT_ALL_SYMBOLS='-Wl,--export-all-symbols' > - AC_MSG_RESULT([yes]) > - else > - AC_MSG_RESULT([no]) > - fi > - if test x"$WIN32_LD_EXPORT_ALL_SYMBOLS" != x; then > - LDFLAGS="$LDFLAGS $WIN32_LD_EXPORT_ALL_SYMBOLS" > - fi > ;; > esac > ])
