patchGNU Autoconf Archive - Patches: patch #7973, AX_PTHREAD do not define...

 
 

patch #7973: AX_PTHREAD do not define PTHREAD_CC on non aix

Submitted by:  None
Submitted on:  Fri Mar 15 22:55:08 2013  
 
Priority: 5 - NormalStatus: Done
Assigned to: Peter Simons <simons>Originator Email: -unavailable-
Open/Closed: Closed

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri Mar 29 17:23:16 2013, comment #4:

Pushed in 386b290576d78b33da477bc2c4620f81ea730eea. Thank you very much for the patch.

Peter Simons <simons>
Project AdministratorIn charge of this item.
Tue Mar 26 18:44:49 2013, comment #3:

The fix works for my original case. Thanks!

Sebastien Fabbro <sfabbro>
Tue Mar 26 17:51:05 2013, comment #2:

Good catch. But the macro already has too many PTHREAD_CC="$CC"s in there as it is; the bug is the result of trying to catch all the corner cases individually, and failing.

I'm attaching a somewhat heavier fix (alt-fix-7973.patch) that addresses the problem by simplifying the code. Could the original submitter give it a try?

(file #27697)

Daniel Richard G. <iskunk>
Mon Mar 25 12:27:31 2013, comment #1:

Steven, Daniel, does this patch look fine to you?

Peter Simons <simons>
Project AdministratorIn charge of this item.
Fri Mar 15 22:55:08 2013, original submission:

Latest revision on ax_pthread.m4 is missing one test case: GCC is defined, and a non AIX host.

Here is a simple patch to overcome it.

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #27697:  alt-fix-7973.patch added by iskunk (1015B - text/x-patch)
file #27611:  ax_pthread.m4.patch added by None (283B - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by sfabbro (Posted a comment)
  • -unavailable- added by iskunk (Updated the item)
  • -unavailable- added by simons (Posted a comment)
  • -unavailable- added by simons
  • -unavailable- added by simons
  • -unavailable- added by None (Submitted the item)
  •  

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 9 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu Apr 4 10:11:12 2013simonsStatusReady For Test=>Done
      Open/ClosedOpen=>Closed
    Fri Mar 29 17:23:16 2013simonsStatusNeed Info=>Ready For Test
    Tue Mar 26 17:51:05 2013iskunkAttached File-=>Added alt-fix-7973.patch, #27697
    Mon Mar 25 12:27:31 2013simonsStatusNone=>Need Info
      Assigned toNone=>simons
      Carbon-Copy-=>Added steven g. johnson
      Carbon-Copy-=>Added daniel richard g.
    Fri Mar 15 22:55:09 2013NoneAttached File-=>Added ax_pthread.m4.patch, #27611

    Back to the top


    Powered by Savane 3.1-cleanup