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

 
 

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

Submitted by:  Galen Rhodes <galenrhodes>
Submitted on:  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

Add a New Comment (Rich Markup)
   

You are not logged in

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

 

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

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

Richard Frith-Macdonald <CaS>
Project 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)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    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.7