[patch gcc]: Add executable-extension for exported-symbol-test in configure.ac

[patch gcc]: Add executable-extension for exported-symbol-test in configure.ac

[Kai Tietz]

Hi,

this patch adds exeext to -rdynamic and exported symbol-check within
configure.ac.  By this this test will be preformed as intended on
targets with executable-extensions (nevertheless -rdynamics will still
fail for pe-coff targets).

ChangeLog

2013-11-10  Karlson2k  <k2k@narod.ru>
        Kai Tietz  <ktietz@redhat.com>

    PR plugins/52872
    * configure.ac: Adding for exported symbols check
    and for rdynamic-check executable-extension.
    * configure: Regenerated.

Tested for i686-w64-mingw32, x86_64-w64-mingw32, and i686-pc-cygwin.
I will apply this patch soon, if there are no objections.

Regards,
Kai

Index: configure.ac
===================================================================
--- configure.ac    (Revision 204461)
+++ configure.ac    (Arbeitskopie)
@@ -5334,15 +5334,15 @@
   AC_MSG_CHECKING([for exported symbols])
   if test "x$export_sym_check" != x; then
     echo "int main() {return 0;} int foobar() {return 0;}" > conftest.c
-    ${CC} ${CFLAGS} ${LDFLAGS} conftest.c -o conftest > /dev/null 2>&1
-    if $export_sym_check conftest | grep foobar > /dev/null; then
+    ${CC} ${CFLAGS} ${LDFLAGS} conftest.c -o conftest$ac_exeext >
/dev/null 2>&1
+    if $export_sym_check conftest$ac_exeext | grep foobar > /dev/null; then
       : # No need to use a flag
       AC_MSG_RESULT([yes])
     else
       AC_MSG_RESULT([yes])
       AC_MSG_CHECKING([for -rdynamic])
-      ${CC} ${CFLAGS} ${LDFLAGS} -rdynamic conftest.c -o conftest >
/dev/null 2>&1
-      if $export_sym_check conftest | grep foobar > /dev/null; then
+      ${CC} ${CFLAGS} ${LDFLAGS} -rdynamic conftest.c -o
conftest$ac_exeext > /dev/null 2>&1
+      if $export_sym_check conftest$ac_exeext | grep foobar > /dev/null; then
         plugin_rdynamic=yes
         pluginlibs="-rdynamic"
       else

Re: [patch gcc]: Add executable-extension for exported-symbol-test in configure.ac

[Bernhard Reutner-Fischer]

On 10 November 2013 13:49:37 Kai Tietz <ktietz70@googlemail.com> wrote:
> Hi,
>
> this patch adds exeext to -rdynamic and exported symbol-check within
> configure.ac.  By this this test will be preformed as intended on
> targets with executable-extensions (nevertheless -rdynamics will still
> fail for pe-coff targets).
>
> ChangeLog
>
> 2013-11-10  Karlson2k  <k2k@narod.ru>
>         Kai Tietz  <ktietz@redhat.com>
>
>     PR plugins/52872
>     * configure.ac: Adding for exported symbols check
>     and for rdynamic-check executable-extension.
>     * configure: Regenerated.
>
> Tested for i686-w64-mingw32, x86_64-w64-mingw32, and i686-pc-cygwin.
> I will apply this patch soon, if there are no objections.
>
> Regards,
> Kai
>
> Index: configure.ac
> ===================================================================
> --- configure.ac    (Revision 204461)
> +++ configure.ac    (Arbeitskopie)
> @@ -5334,15 +5334,15 @@
>    AC_MSG_CHECKING([for exported symbols])
>    if test "x$export_sym_check" != x; then
>      echo "int main() {return 0;} int foobar() {return 0;}" > conftest.c
> -    ${CC} ${CFLAGS} ${LDFLAGS} conftest.c -o conftest > /dev/null 2>&1
> -    if $export_sym_check conftest | grep foobar > /dev/null; then
> +    ${CC} ${CFLAGS} ${LDFLAGS} conftest.c -o conftest$ac_exeext >
> /dev/null 2>&1
> +    if $export_sym_check conftest$ac_exeext | grep foobar > /dev/null; then

grep -q

>        : # No need to use a flag
>        AC_MSG_RESULT([yes])
>      else
>        AC_MSG_RESULT([yes])
>        AC_MSG_CHECKING([for -rdynamic])
> -      ${CC} ${CFLAGS} ${LDFLAGS} -rdynamic conftest.c -o conftest >
> /dev/null 2>&1
> -      if $export_sym_check conftest | grep foobar > /dev/null; then
> +      ${CC} ${CFLAGS} ${LDFLAGS} -rdynamic conftest.c -o
> conftest$ac_exeext > /dev/null 2>&1
> +      if $export_sym_check conftest$ac_exeext | grep foobar > /dev/null; then

Ditto.
Thanks,
>          plugin_rdynamic=yes
>          pluginlibs="-rdynamic"
>        else



Sent with AquaMail for Android
http://www.aqua-mail.com