bugGNUstep - Bugs: bug #50751, non-fragile abi compiler flag was...

Group
 
 

bug #50751: non-fragile abi compiler flag was incorrectly removed from common.make

Submitter:  Galen Rhodes <galenrhodes>
Submitted:  Fri 07 Apr 2017 03:45:56 PM UTC
   
 
Category:  Makefiles Severity:  3 - Normal
Item Group:  Bug Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 09 Apr 2017 06:32:26 AM UTC, comment #1: 

Thanks very much for the report ... corrected in svn.

Richard Frith-Macdonald <CaS>
Group Member
Fri 07 Apr 2017 03:45:56 PM UTC, original submission:  

On May 19, 2016 a commit was made with the comment "remove unused flag" that remove the "-fobjc-nonfragile-abi" from the INTERNAL_OBJCFLAGS variable if the user requested the use of non fragile abi option.  My guess is that the intention was to remove it from the INTERNAL_LDFLAGS variable (the next line in the makefile) which would have made more sense given the comment.

But this change pretty much broke the GUI and BACKEND packages for anyone trying to build the GNUstep libraries for use with clang and ARC.  Attempting to use the -fobjc-arc flag while compiling a GUI application results in an executable that won't load because of a mismatched ABI configuration.

The only way to fix this is to add the "-fobjc-nonfragile-abi" flag back to the INTERNAL_OBJCFLAGS variable in common.make and then rebuild all of the GNUstep libraries.

See attached screen shots for the details of the commit that broke this.

Galen Rhodes <galenrhodes>

 

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

Attach Files:
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by CaS (Posted a comment)
  • -email is unavailable- added by galenrhodes (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-04-09 CaS StatusNone Fixed
        Open/ClosedOpen Closed
    2017-04-07 galenrhodes Attached File- Added GitboxScreenSnapz001.png, #40318
        Attached File- Added KaleidoscopeScreenSnapz001.png, #40319

    Back to the top

    Powered by Savane 3.13-3230.
    Corresponding source code