Thread overview
[Issue 22] Build fails with macro expansion failures for TARGET_ANDROID
Oct 27, 2012
Johannes Pfau
Oct 27, 2012
Johannes Pfau
Oct 27, 2012
Iain Buclaw
Nov 06, 2012
Johannes Pfau
October 27, 2012
http://gdcproject.org/bugzilla/show_bug.cgi?id=22

Johannes Pfau <johannespfau@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |johannespfau@gmail.com
         AssignedTo|ibuclaw@gdcproject.org      |johannespfau@gmail.com

--- Comment #1 from Johannes Pfau <johannespfau@gmail.com> 2012-10-27 18:16:33 UTC ---
A recent change in gcc broke this. Workaround: Use an older gcc snapshot. The problem: GCC now defines a TARGET_ANDROID itself which clashes with our TARGET_ANDROID:

-------------------------------------
    * opth-gen.awk (TARGET_* generation): Always generate TARGET_<xxx>
    for Mask options, whether they use Var(...) or not.
-------------------------------------

We can probably use that TARGET_ANDROID macro and throw away our
implementation. Can you test if whether it works if you put the calls to
TARGET_ANDROID in parens? So in d-spec.c:59:
#if TARGET_WINDOS || TARGET_OSX || (TARGET_ANDROID)

and d-spec.c:488:22:
#if TARGET_LINUX && !(TARGET_ANDROID)

If this does not fix it, it's probably a bug in upstream gcc as the error is in the TARGET_ANDROID macro definition which is part of gcc.

-- 
Configure issuemail: http://gdcproject.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all issue changes.
October 27, 2012
http://gdcproject.org/bugzilla/show_bug.cgi?id=22

--- Comment #2 from Johannes Pfau <johannespfau@gmail.com> 2012-10-27 18:21:03 UTC ---
Ermm, maybe the gcc TARGET_ANDROID macro expands to a runtime expression. This needs some more work then.

-- 
Configure issuemail: http://gdcproject.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all issue changes.
October 27, 2012
http://gdcproject.org/bugzilla/show_bug.cgi?id=22

Iain Buclaw <ibuclaw@gdcproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ibuclaw@gdcproject.org

--- Comment #3 from Iain Buclaw <ibuclaw@gdcproject.org> 2012-10-27 19:16:20 UTC ---
(In reply to comment #2)
> Ermm, maybe the gcc TARGET_ANDROID macro expands to a runtime expression. This needs some more work then.

That it does. Best thing to do is just give it another name...

-- 
Configure issuemail: http://gdcproject.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all issue changes.
November 06, 2012
http://gdcproject.org/bugzilla/show_bug.cgi?id=22

Johannes Pfau <johannespfau@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #4 from Johannes Pfau <johannespfau@gmail.com> 2012-11-06 15:46:41 UTC ---
FIXED: https://github.com/D-Programming-GDC/GDC/commit/102ab270fdf9d01fce5dd2c2c30ab308e6be9f37

-- 
Configure issuemail: http://gdcproject.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all issue changes.