[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 99938] kig fails to compile because of undefined symbols in
From:       Dominique Devriese <devriese () kde ! org>
Date:       2005-02-24 13:29:36
Message-ID: 20050224132936.9014.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=99938         




------- Additional Comments From devriese kde org  2005-02-24 14:29 -------
Anne-Marie Mahfouf writes:
> Obviously we had no time to read all that doc as the fix was
> urgent. I am of course no master in all that and I usually leave
> that sort of fixes to people who knows best. However I think it was
> fair to try by ourselves to solve this with the knowledge we have
> instead of asking top KDE people who are quite busy with svn and the
> release.  Having said the fix was stupid, you need to provide us
> with your fix in order to support this accusation. One can say that
> some of MY commits are stupid because I admit I am a beginner
> especially in ld and gcc but I react strongly when you say Albert's
> fix is stupid. I praise Albert for his work within the project. His
> code is high quality and his help unvaluable. If you follow the
> latest KDE developments you would know that Albert is someone highly
> respected within KDE. So, sorry, but reading that Albert made a
> stupid fix without having a proof of that is really making me angry.

I have already explained the problems with Albert's commit.  I don't
claim to be an expert on linker stuff, but I know enough to know that
his fix could at best hide the problem in the bug he was dealing with,
and raise the problem again for other bugs.  I have also sent you the
proper fix, and you have committed it yourself.  Can we please end
this discussion ?

cheers
domi
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic