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

 
 

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

Submitter:  None
Submitted:  Fri 15 Mar 2013 10:55:08 PM UTC
   
 
Priority:  5 - Normal Status:  Done
Assigned to:  simons Originator Email:  -email is unavailable-
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 29 Mar 2013 05:23:16 PM UTC, comment #4: 

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

Peter Simons <simons>
Group administrator
Tue 26 Mar 2013 06:44:49 PM UTC, comment #3: 

The fix works for my original case. Thanks!

Sebastien Fabbro <sfabbro>
Tue 26 Mar 2013 05:51:05 PM UTC, 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 25 Mar 2013 12:27:31 PM UTC, comment #1: 

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

Peter Simons <simons>
Group administrator
Fri 15 Mar 2013 10:55:08 PM UTC, 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 Files:
   
   
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
  • -email is unavailable- added by sfabbro (Posted a comment)
  • -email is unavailable- added by iskunk (Updated the item)
  • -email is unavailable- added by simons (Posted a comment)
  • -email is unavailable- added by simons
  • -email is unavailable- added by simons
  • -email is unavailable- added by None (Submitted the item)
  •  

    Follow 9 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-04-04 simons StatusReady For Test Done
        Open/ClosedOpen Closed
    2013-03-29 simons StatusNeed Info Ready For Test
    2013-03-26 iskunk Attached File- Added alt-fix-7973.patch, #27697
    2013-03-25 simons StatusNone Need Info
        Assigned toNone simons
        Carbon-Copy- Added steven g. johnson <stevenj@alum.mit.edu>
        Carbon-Copy- Added daniel richard g. <skunk@iskunk.org>
    2013-03-15 None Attached File- Added ax_pthread.m4.patch, #27611

    Back to the top

    Powered by Savane 3.13-ee12.
    Corresponding source code