<html>
<head>
</head>
<body>
Hi Harry,<br>
<br>
COMPLAIN! ;-)<br>
always the same problem since ages ;-)<br>
Please don't do it this way, because it would take again 3 weeks till the
next one reports a bug like:<br>
<br>
"Some lib-entry will not be recognized!"<br>
<br>
Imagine the following entry done with the textedit field:<br>
"-lqt -lX11 -lkfile"<br>
<br>
First it recognises -X11 and the new ldadd is:<br>
"-lqt-lkfile"<br>
<br>
Isn't it?<br>
<br>
<br>
Another problem is that there has to be a blank at the end of ldadd, which
always causes an additional blank everytime you call project options.<br>
<br>
Please try to make a path like:<br>
<br>
1) before entering in this if-clauses add a blank at the beginning and the
end of ldadd<br>
<br>
2) use sth like<br>
<pre wrap="">ldadd = ldadd.replace( QRegExp(" -lX11 "), " " );<br></pre>
3) strip left and right bounded white spaces after all checkings were done,
so the rest can be used as context for the text editfield.<br>
<br>
<br>
Ciao<br>
<br>
Walter<br>
<br>
Harald Fernengel wrote:<br>
<blockquote type="cite" cite="mid:200203212008.40919.harry@kdevelop.org">
<pre wrap="">Hi,<br><br>this is pretty straight-forward, if we encounter " -lm" in the additional <br>libraries edit we replace it with "" and check the "libmath" checkbox. <br>Problem: " -lmylib" gets "ylib" which is pretty stupid (Bug #39538).<br><br>If no one complains I'll commit in a few hours so it gets into the release...<br><br>Best regards,<br>Harry</pre>
<pre wrap=""><br><hr width="90%" size="4"><br>Index: cprjoptionsdlg.cpp<br>===================================================================<br>RCS file: /home/kde/kdevelop/kdevelop/Attic/cprjoptionsdlg.cpp,v<br>retrieving revision 1.26.2.21.2.22.2.13<br>diff -u -3 -p -r1.26.2.21.2.22.2.13 cprjoptionsdlg.cpp<br>--- cprjoptionsdlg.cpp 2002/03/02 17:53:34 1.26.2.21.2.22.2.13<br>+++ cprjoptionsdlg.cpp 2002/03/21 19:02:56<br>@@ -304,7 +304,7 @@ void CPrjOptionsDlg::addLinkerPage()<br> l_X11->setText("X11");<br> l_X11->setChecked(ldadd.find(" -lX11 ") != -1);<br> if (l_X11->isChecked())<br>- ldadd = ldadd.replace( QRegExp(" -lX11"), "" );<br>+ ldadd = ldadd.replace( QRegExp(" -lX11 "), "" );<br> QWhatsThis::add(l_X11, i18n("X11 basics "));<br> <br> l_Xext=new QCheckBox(libs_group,"l_Xext");<br>@@ -312,7 +312,7 @@ void CPrjOptionsDlg::addLinkerPage()<br> l_Xext->setText("Xext");<br> l_Xext->setChecked(ldadd.find(" -lXext ") != -1);<br> if (l
_Xext->isChecked())<br>- ldadd = ldadd.replace( QRegExp(" -lXext"), "" );<br>+ ldadd = ldadd.replace( QRegExp(" -lXext "), "" );<br> QWhatsThis::add(l_Xext, i18n("X11 extensions "));<br> <br> l_qt=new QCheckBox(libs_group,"l_qt");<br>@@ -321,7 +321,7 @@ void CPrjOptionsDlg::addLinkerPage()<br> l_qt->setChecked((ldadd.find(" -lqt ") != -1) || (ldadd.find(" $(LIB_QT) ") != -1));<br> if (l_qt->isChecked())<br> {<br>- ldadd = ldadd.replace( QRegExp(" -lqt"), "" );<br>+ ldadd = ldadd.replace( QRegExp(" -lqt "), "" );<br> ldadd = ldadd.replace( QRegExp(" \\$\\(LIB_QT\\)"), "" );<br> }<br> QWhatsThis::add(l_qt, i18n("Qt"));<br>@@ -343,7 +343,7 @@ void CPrjOptionsDlg::addLinkerPage()<br> l_kdeui->setChecked((ldadd.find(" -lkdeui ") != -1)|| (ldadd.find(" $(LIB_KDEUI) ") != -1));<br> if (l_kdeui->isChecked())<br> {<br>- ldadd = ldadd.replace( QRegExp(" -lkdeui"), "" );<br>+ ldadd = ldadd.replace( QRegExp(" -lkdeui "), "" );<br>
ldadd = ldadd.replace( QRegExp(" \\$\\(LIB_KDEUI\\)"), "" );<br> }<br> QWhatsThis::add(l_kdeui, i18n("KDE user interface"));<br>@@ -354,7 +354,7 @@ void CPrjOptionsDlg::addLinkerPage()<br> l_khtml->setChecked((ldadd.find(" -lkhtml ") != -1) || (ldadd.find(" $(LIB_KHTML) ") != -1));<br> if (l_khtml->isChecked())<br> {<br>- ldadd = ldadd.replace( QRegExp(" -lkhtml"), "" );<br>+ ldadd = ldadd.replace( QRegExp(" -lkhtml "), "" );<br> ldadd = ldadd.replace( QRegExp(" \\$\\(LIB_KHTML\\)"), "" );<br> }<br> QWhatsThis::add(l_khtml, i18n("KDE HTML widget"));<br>@@ -364,7 +364,7 @@ void CPrjOptionsDlg::addLinkerPage()<br> l_kfm->setText("kfm");<br> l_kfm->setChecked(ldadd.find(" -lkfm ") != -1);<br> if (l_kfm->isChecked())<br>- ldadd = ldadd.replace( QRegExp(" -lkfm"), "" );<br>+ ldadd = ldadd.replace( QRegExp(" -lkfm "), "" );<br> // TODO add note that this is KDE-1 only, or remove checkbox if project isn't a KDE-1 project<br>
QWhatsThis::add(l_kfm, i18n("KDE kfm functionality"));<br> <br>@@ -374,7 +374,7 @@ void CPrjOptionsDlg::addLinkerPage()<br> l_kfile->setChecked((ldadd.find(" -lkfile ") != -1) || (ldadd.find(" $(LIB_KFILE) ") != -1));<br> if (l_kfile->isChecked())<br> {<br>- ldadd = ldadd.replace( QRegExp(" -lkfile"), "" );<br>+ ldadd = ldadd.replace( QRegExp(" -lkfile "), "" );<br> ldadd = ldadd.replace( QRegExp(" \\$\\(LIB_KFILE\\)"), "" );<br> }<br> QWhatsThis::add(l_kfile, i18n("KDE file handling"));<br>@@ -385,7 +385,7 @@ void CPrjOptionsDlg::addLinkerPage()<br> l_kparts->setChecked((ldadd.find(" -lkparts ") != -1) || (ldadd.find(" $(LIB_KPARTS) ") != -1));<br> if (l_kparts->isChecked())<br> {<br>- ldadd = ldadd.replace( QRegExp(" -lkparts"), "" );<br>+ ldadd = ldadd.replace( QRegExp(" -lkparts "), "" );<br> ldadd = ldadd.replace( QRegExp(" \\$\\(LIB_KPARTS\\)"), "" );<br> }<br> QWhatsThis::add(l_kparts, i18n("KDE component architect
ure"));<br>@@ -396,7 +396,7 @@ void CPrjOptionsDlg::addLinkerPage()<br> l_kspell->setChecked((ldadd.find(" -lkspell ") != -1) || (ldadd.find(" $(LIB_KSPELL) ") != -1));<br> if (l_kspell->isChecked())<br> {<br>- ldadd = ldadd.replace( QRegExp(" -lkspell"), "" );<br>+ ldadd = ldadd.replace( QRegExp(" -lkspell "), "" );<br> ldadd = ldadd.replace( QRegExp(" \\$\\(LIB_KSPELL\\)"), "" );<br> }<br> QWhatsThis::add(l_kspell, i18n("KDE Spell checking"));<br>@@ -407,7 +407,7 @@ void CPrjOptionsDlg::addLinkerPage()<br> l_kab->setChecked((ldadd.find(" -lkab ") != -1) || (ldadd.find(" $(LIB_KAB) ") != -1));<br> if (l_kab->isChecked())<br> {<br>- ldadd = ldadd.replace( QRegExp(" -lkab"), "" );<br>+ ldadd = ldadd.replace( QRegExp(" -lkab "), "" );<br> ldadd = ldadd.replace( QRegExp(" \\$\\(LIB_KAB\\)"), "" );<br> }<br> QWhatsThis::add(l_kab, i18n("KDE addressbook"));<br>@@ -417,7 +417,7 @@ void CPrjOptionsDlg::addLinkerPage()<br> l_math-
>setText("math");<br> l_math->setChecked(l_khtml->isChecked() || (ldadd.find(" -lm ") != -1));<br> if (l_math->isChecked())<br>- ldadd = ldadd.replace( QRegExp(" -lm"), "" );<br>+ ldadd = ldadd.replace( QRegExp(" -lm "), "" );<br> QWhatsThis::add(l_math, i18n("Math library"));<br> <br> QLabel* addit_ldadd_label;<br></pre>
</blockquote>
<br>
<pre class="moz-signature" cols="$mailwrapcol">--
The KDevelop project: <a class="moz-txt-link-abbreviated" href="mailto:tasin@kdevelop.de">tasin@kdevelop.de</a> [<a class="moz-txt-link-abbreviated" href="http://www.kdevelop.org">www.kdevelop.org</a>]
--
oohhh sveglia.... il mondo e' ammalato, ma x colpa di chi.........
(Zucchero)
:-------W. Tasin, FB 04,FHM-------------------PGP-KeyID:0x7961A645----------:
<Key-Fingerprint: 1610 835F 0080 32F4 6140 6CF7 A7D0 44CD 7961A645>
</pre>
<br>
</body>
</html>