--liOOAslEiF7prFVr
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

Hi,

I'm compiling kdelibs in trunk with gcc-4.0 and noticed the following
isssue:

configure is testing "if Qt is patched for -fvisibility" by looking for
a non-empty Q_EXPORT. The corresponding patch for qt3 defines Q_EXPORT:
# define Q_EXPORT __attribute__((visibility("default")))

however in qt4 the corresponding macro has a different name:
# define Q_DECL_EXPORT __attribute__((visibility("default")))

Is the test still necessary or do all qt4 versions support this?
Anyway, here is a patch for acinclude.m4.in to update to the new macro.

michael


--liOOAslEiF7prFVr
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename="acinclude.m4.in.diff"

Index: acinclude.m4.in
================================================== =================
--- acinclude.m4.in (revision 443829)
+++ acinclude.m4.in (working copy)
@@ -2938,9 +2938,9 @@
AC_TRY_COMPILE(
[
#include
-#if Q_EXPORT - 0 != 0
-/* if this compiles, then Q_EXPORT is undefined */
-/* if Q_EXPORT is nonempty, this will break compilation */
+#if Q_DECL_EXPORT - 0 != 0
+/* if this compiles, then Q_DECL_EXPORT is undefined */
+/* if Q_DECL_EXPORT is nonempty, this will break compilation */
#endif
], [/* elvis is alive */],
kde_cv_val_qt_gcc_visibility_patched=no, kde_cv_val_qt_gcc_visibility_patched=yes)

--liOOAslEiF7prFVr
Content-Type: text/plain; charset="iso-8859-1"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

=

>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscrib=

e <<

--liOOAslEiF7prFVr--