GNUstep - Patches: patch #7822, [GNUstep Make] Bad code generator...
You are not allowed to post comments on this tracker with your current authentication level.
patch #7822: [GNUstep Make] Bad code generator in Makefiles/Instance/framework.make, causes compiler errors
Submitter: | Stanislav Yaglo <stam> | ||
Submitted: | Thu 26 Jul 2012 08:17:55 PM UTC | ||
Category: | None | Priority: | 5 - Normal |
Status: | Ready For Test | Privacy: | Public |
Assigned to: | thebeing | Open/Closed: | Closed |
Thu 16 Aug 2012 08:01:07 PM UTC, comment #3: |
Niels Grewe <thebeing> |
Wed 15 Aug 2012 06:52:01 PM UTC, comment #2: It seems that the new clang considers definition of a new root class an advanced feature by default, since base classes now should have _attribute_((objc_root_class)) before @interface.
|
Stanislav Yaglo <stam> |
Wed 15 Aug 2012 02:46:47 PM UTC, comment #1: Please note that clang doesn't strictly emit an error here, but a warning which is then escalated using -Werror, so gnustep-make is not generating invalid code here (there are perfectly sensible reasons for implementing root classes, after all). But there seems to be no valid reason why the framework dummy-classes should be root-classes, so I went ahead applied your patch. |
Niels Grewe <thebeing> |
Thu 26 Jul 2012 08:17:55 PM UTC, original submission:
Hi, when building an Etoile framework I stumbled upon a compiler error with the new clang (version 3.2 (trunk 160797)). It doesn't like @interface without a base class.
|
Stanislav Yaglo <stam> |
Depends on the following items: None found
Items that depend on this one: None found
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.
Follow 4 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2013-12-20 | FredKiefer | Assigned to | None | thebeing | |
Open/Closed | Open | Closed | |||
2012-08-15 | thebeing | Status | None | Ready For Test | |
2012-07-26 | stam | Attached File | - | Added framework.make.patch, #26266 |
Powered by Savane 3.13-f8d8.
Corresponding source code
Hi Stanislav,
Thanks alot for the pointer! I've updated gnustep-base to use the attribute for its root classes (when compiling with clang, that is).
Cheers,
Niels