public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] Fix configure --with-cloog
@ 2011-07-06 10:11 Romain Geissler
  2011-07-06 12:50 ` Romain Geissler
  2011-07-25 20:30 ` Romain Geissler
  0 siblings, 2 replies; 6+ messages in thread
From: Romain Geissler @ 2011-07-06 10:11 UTC (permalink / raw)
  To: gcc-patches

Hello

This patch fix an issue while building with cloog and gmp installed in
a custom separate directories.

How to reproduce :
 - Make sure you've installed cloog and gmp in separate directories
(ie ${WITH-CLOOG-PATH}/lib doesn't contain libgmp)
 - Make sure neither gmp nor cloog is not installed in a directory
searched by default by your linker when looking for libs.
 - Launch configure script with both --with-gmp and --with-cloog
switch properly set

This result in an unexpected error while configuring: error: Unable to
find a usable CLooG.


2011-07-06  Romain Geissler  <romain.geissler@gmail.com>

       * configure: Add $gmplibs to cloog $LDFLAGS

Index: configure
===================================================================
--- configure   (revision 175709)
+++ configure   (working copy)
@@ -5713,7 +5713,7 @@ if test "x$with_cloog" != "xno"; then

     CFLAGS="${CFLAGS} ${clooginc} ${gmpinc}"
   CPPFLAGS="${CPPFLAGS} ${_cloogorginc}"
-  LDFLAGS="${LDFLAGS} ${clooglibs}"
+  LDFLAGS="${LDFLAGS} ${clooglibs} ${gmplibs}"

   case $cloog_backend in
     "ppl-legacy")

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Fix configure --with-cloog
  2011-07-06 10:11 [PATCH] Fix configure --with-cloog Romain Geissler
@ 2011-07-06 12:50 ` Romain Geissler
  2011-07-11  7:52   ` Romain Geissler
  2011-07-25 20:30 ` Romain Geissler
  1 sibling, 1 reply; 6+ messages in thread
From: Romain Geissler @ 2011-07-06 12:50 UTC (permalink / raw)
  To: gcc-patches

> Hello
>
> This patch fix an issue while building with cloog and gmp installed in
> a custom separate directories.
>
> How to reproduce :
>  - Make sure you've installed cloog and gmp in separate directories
> (ie ${WITH-CLOOG-PATH}/lib doesn't contain libgmp)
>  - Make sure neither gmp nor cloog is not installed in a directory
> searched by default by your linker when looking for libs.
>  - Launch configure script with both --with-gmp and --with-cloog
> switch properly set
>
> This result in an unexpected error while configuring: error: Unable to
> find a usable CLooG.
>
>
> 2011-07-06  Romain Geissler  <romain.geissler@gmail.com>
>
>       * configure: Add $gmplibs to cloog $LDFLAGS
>
> Index: configure
> ===================================================================
> --- configure   (revision 175709)
> +++ configure   (working copy)
> @@ -5713,7 +5713,7 @@ if test "x$with_cloog" != "xno"; then
>
>     CFLAGS="${CFLAGS} ${clooginc} ${gmpinc}"
>   CPPFLAGS="${CPPFLAGS} ${_cloogorginc}"
> -  LDFLAGS="${LDFLAGS} ${clooglibs}"
> +  LDFLAGS="${LDFLAGS} ${clooglibs} ${gmplibs}"
>
>   case $cloog_backend in
>     "ppl-legacy")
>

I forgot configure was a generated script. Here is the patch that fix
it at the m4 macro level :


2011-07-06  Romain Geissler  <romain.geissler@gmail.com>

      * config/cloog.m4: Add $gmplibs to cloog $LDFLAGS
      * configure: Regenerate


Index: config/cloog.m4
===================================================================
--- config/cloog.m4     (revision 175907)
+++ config/cloog.m4     (working copy)
@@ -142,7 +142,7 @@ AC_DEFUN([CLOOG_FIND_FLAGS],
   dnl clooglibs & clooginc may have been initialized by CLOOG_INIT_FLAGS.
   CFLAGS="${CFLAGS} ${clooginc} ${gmpinc}"
   CPPFLAGS="${CPPFLAGS} ${_cloogorginc}"
-  LDFLAGS="${LDFLAGS} ${clooglibs}"
+  LDFLAGS="${LDFLAGS} ${clooglibs} ${gmplibs}"

   case $cloog_backend in
     "ppl-legacy")

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Fix configure --with-cloog
  2011-07-06 12:50 ` Romain Geissler
@ 2011-07-11  7:52   ` Romain Geissler
  2011-07-18  9:24     ` Romain Geissler
  0 siblings, 1 reply; 6+ messages in thread
From: Romain Geissler @ 2011-07-11  7:52 UTC (permalink / raw)
  To: gcc-patches; +Cc: bhackett1024

2011/7/6 Romain Geissler <romain.geissler@gmail.com>:
>
> I forgot configure was a generated script. Here is the patch that fix
> it at the m4 macro level :
>
>
> 2011-07-06  Romain Geissler  <romain.geissler@gmail.com>
>
>      * config/cloog.m4: Add $gmplibs to cloog $LDFLAGS
>      * configure: Regenerate
>
>
> Index: config/cloog.m4
> ===================================================================
> --- config/cloog.m4     (revision 175907)
> +++ config/cloog.m4     (working copy)
> @@ -142,7 +142,7 @@ AC_DEFUN([CLOOG_FIND_FLAGS],
>   dnl clooglibs & clooginc may have been initialized by CLOOG_INIT_FLAGS.
>   CFLAGS="${CFLAGS} ${clooginc} ${gmpinc}"
>   CPPFLAGS="${CPPFLAGS} ${_cloogorginc}"
> -  LDFLAGS="${LDFLAGS} ${clooglibs}"
> +  LDFLAGS="${LDFLAGS} ${clooglibs} ${gmplibs}"
>
>   case $cloog_backend in
>     "ppl-legacy")
>

Ping: It seems that little patch has been forgotten.
Is ok for the trunk ?

NB: I don't have write access to the trunk

Romain Geissler

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Fix configure --with-cloog
  2011-07-11  7:52   ` Romain Geissler
@ 2011-07-18  9:24     ` Romain Geissler
  0 siblings, 0 replies; 6+ messages in thread
From: Romain Geissler @ 2011-07-18  9:24 UTC (permalink / raw)
  To: gcc-patches

2011/7/11 Romain Geissler <romain.geissler@gmail.com>:
> 2011/7/6 Romain Geissler <romain.geissler@gmail.com>:
>>
>> I forgot configure was a generated script. Here is the patch that fix
>> it at the m4 macro level :
>>
>>
>> 2011-07-06  Romain Geissler  <romain.geissler@gmail.com>
>>
>>      * config/cloog.m4: Add $gmplibs to cloog $LDFLAGS
>>      * configure: Regenerate
>>
>>
>> Index: config/cloog.m4
>> ===================================================================
>> --- config/cloog.m4     (revision 175907)
>> +++ config/cloog.m4     (working copy)
>> @@ -142,7 +142,7 @@ AC_DEFUN([CLOOG_FIND_FLAGS],
>>   dnl clooglibs & clooginc may have been initialized by CLOOG_INIT_FLAGS.
>>   CFLAGS="${CFLAGS} ${clooginc} ${gmpinc}"
>>   CPPFLAGS="${CPPFLAGS} ${_cloogorginc}"
>> -  LDFLAGS="${LDFLAGS} ${clooglibs}"
>> +  LDFLAGS="${LDFLAGS} ${clooglibs} ${gmplibs}"
>>
>>   case $cloog_backend in
>>     "ppl-legacy")
>>
>
> Ping: It seems that little patch has been forgotten.
> Is ok for the trunk ?
>
> NB: I don't have write access to the trunk
>
> Romain Geissler
>

Ping !

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Fix configure --with-cloog
  2011-07-06 10:11 [PATCH] Fix configure --with-cloog Romain Geissler
  2011-07-06 12:50 ` Romain Geissler
@ 2011-07-25 20:30 ` Romain Geissler
  2011-08-23 15:20   ` Romain Geissler
  1 sibling, 1 reply; 6+ messages in thread
From: Romain Geissler @ 2011-07-25 20:30 UTC (permalink / raw)
  To: Romain Geissler; +Cc: gcc-patches

Le 6 juil. 2011 à 11:04, Romain Geissler a écrit :

> Hello
> 
> This patch fix an issue while building with cloog and gmp installed in
> a custom separate directories.
> 
> How to reproduce :
> - Make sure you've installed cloog and gmp in separate directories
> (ie ${WITH-CLOOG-PATH}/lib doesn't contain libgmp)
> - Make sure neither gmp nor cloog is not installed in a directory
> searched by default by your linker when looking for libs.
> - Launch configure script with both --with-gmp and --with-cloog
> switch properly set
> 
> This result in an unexpected error while configuring: error: Unable to
> find a usable CLooG.
> 
> 
> 2011-07-06  Romain Geissler  <romain.geissler@gmail.com>
> 
>       * configure: Add $gmplibs to cloog $LDFLAGS
> 
> Index: configure
> ===================================================================
> --- configure   (revision 175709)
> +++ configure   (working copy)
> @@ -5713,7 +5713,7 @@ if test "x$with_cloog" != "xno"; then
> 
>     CFLAGS="${CFLAGS} ${clooginc} ${gmpinc}"
>   CPPFLAGS="${CPPFLAGS} ${_cloogorginc}"
> -  LDFLAGS="${LDFLAGS} ${clooglibs}"
> +  LDFLAGS="${LDFLAGS} ${clooglibs} ${gmplibs}"
> 
>   case $cloog_backend in
>     "ppl-legacy")

Ping !

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Fix configure --with-cloog
  2011-07-25 20:30 ` Romain Geissler
@ 2011-08-23 15:20   ` Romain Geissler
  0 siblings, 0 replies; 6+ messages in thread
From: Romain Geissler @ 2011-08-23 15:20 UTC (permalink / raw)
  To: Romain Geissler; +Cc: gcc-patches

Ping

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2011-08-23 14:51 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-07-06 10:11 [PATCH] Fix configure --with-cloog Romain Geissler
2011-07-06 12:50 ` Romain Geissler
2011-07-11  7:52   ` Romain Geissler
2011-07-18  9:24     ` Romain Geissler
2011-07-25 20:30 ` Romain Geissler
2011-08-23 15:20   ` Romain Geissler

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).