patchGNU Autoconf Archive - Patches: patch #7911, AX_PTHREAD handles AIX incorrectly

 
 

patch #7911: AX_PTHREAD handles AIX incorrectly

Submitter:  None
Submitted:  Thu 03 Jan 2013 10:32:07 AM 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
   

Sat 02 Feb 2013 10:04:35 AM UTC, comment #3: 

Pushed in e40cfd6e5038b4e42c2ae9fb3c567b8deeee921b. Thank you very much, guys!

Peter Simons <simons>
Group administrator
Fri 04 Jan 2013 09:16:45 PM UTC, comment #2: 

I'd make a few changes; see attached.

  • Keep the not-GCC check
  • Furthermore, do this only on AIX
  • Quoting tweaks
  • Sorted the big case pattern and fixed a couple typos
  • Use AS_EXECUTABLE_P() instead of "test -x"


Thomas's patch also addresses /usr/vac/bin/ not being in the user's PATH, which is in fact the case in my employer's production AIX environment.

(file #27202)

Daniel Richard G. <iskunk>
Fri 04 Jan 2013 04:59:08 PM UTC, comment #1: 

Hi Steven, hi Daniel,

do you have an opinion with regard to this patch? Is it okay for you to apply it?

Peter Simons <simons>
Group administrator
Thu 03 Jan 2013 10:32:07 AM UTC, original submission:  

The PTHREAD_CC compiler is currently unconditionally replaced with xlc_r on AIX unless set by the user.

This is erroneous because most compiler wrappers on AIX have a specific feature set not necessarily matched by xlc_r.

I have therefore improved the corresponding shell code to match the wrapper actually used to its corresponding thread-safe variant and would like to see the change made publically available.

Legalese: you are free to distribute and/or use the corresponding submission under the following license:

Copying and distribution of this file, with or without modification,
are permitted in any medium without royalty provided the copyright
notice and this notice are preserved.  This file is offered as-is,
without any warranty.

Regards,
Thomas Jahns

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #27190:  0001-Improve-pthread-compiler-elicitation-for-AIX.patch added by None (1KiB - text/x-diff - created with git format-patch vs. cbc823484f repository main-line)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -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-02-02 simons StatusReady For Test Done
        Open/ClosedOpen Closed
    2013-02-02 simons StatusIn Progress Ready For Test
    2013-01-04 iskunk Attached File- Added 0001-Improve-pthread-compiler-elicitation-for-AIX.v2.patch, #27202
    2013-01-04 simons StatusNone In Progress
        Assigned toNone simons
        Carbon-Copy- Added steven g. johnson <stevenj@alum.mit.edu>
        Carbon-Copy- Added daniel richard g. <skunk@iskunk.org>
    2013-01-03 None Attached File- Added 0001-Improve-pthread-compiler-elicitation-for-AIX.patch, #27190

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code