Opened 7 years ago

Last modified 7 years ago

#40307 new defect

grass @6.4.2: Unsupported compiler detected

Reported by: mojca (Mojca Miklavec) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: manuel.deljesuspe@…, cooljeanius (Eric Gallager)
Port: grass

Description

The following log is from the buildbot for 10.8:

GISRC=/opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/demolocation/.grassrc64 GISBASE=/opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0 PATH="/opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/bin:$PATH" PYTHONPATH="" DYLD_LIBRARY_PATH="/opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/bin:/opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/lib::" LC_ALL=C ./ctypesgen.py --cpp "/usr/bin/llvm-gcc-4.2 -E -I/opt/local/include     -DPACKAGE=\""grasslibs"\" -DPACKAGE=\""grasslibs"\"  -I/opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/include" -lgrass_g3d.6.4.2   /opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/include/grass/G3d.h -o g3d.py
Status: Preprocessing /opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/.tmp/tmp1UOQVF.h
Status: /usr/bin/llvm-gcc-4.2 -E -I/opt/local/include     -DPACKAGE="grasslibs" -DPACKAGE="grasslibs"  -I/opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/include -U __BLOCKS__ -U __GNUC__ -dD "-Dinline=" "-D__inline__=" "-D__extension__=" "-D_Bool=uint8_t" "-D__const=const" "-D__asm__(x)=" "-D__asm(x)=" "-DCTYPESGEN=1" "-D__uint16_t=uint16_t" "-D__uint32_t=uint32_t" "-D__uint64_t=uint64_t" /opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/.tmp/tmp1UOQVF.h
Error: /usr/bin/llvm-gcc-4.2 -E -I/opt/local/include     -DPACKAGE="grasslibs" -DPACKAGE="grasslibs"  -I/opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/include: In file included from /usr/include/stdio.h:64,
Error: /usr/bin/llvm-gcc-4.2 -E -I/opt/local/include     -DPACKAGE="grasslibs" -DPACKAGE="grasslibs"  -I/opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/include:                  from /opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/include/grass/gis.h:24,
Error: /usr/bin/llvm-gcc-4.2 -E -I/opt/local/include     -DPACKAGE="grasslibs" -DPACKAGE="grasslibs"  -I/opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/include:                  from /opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/include/grass/stats.h:5,
Error: /usr/bin/llvm-gcc-4.2 -E -I/opt/local/include     -DPACKAGE="grasslibs" -DPACKAGE="grasslibs"  -I/opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/dist.i386-apple-darwin12.3.0/include:                  from /opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/.tmp/tmp1UOQVF.h:1:
Error: /usr/bin/llvm-gcc-4.2 -E -I/opt/local/include     -DPACKAGE="grasslibs" -DPACKAGE="grasslibs"  -I/opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/grass-6.4.2/
dist.i386-apple-darwin12.3.0/include: /usr/include/sys/cdefs.h:81:2: warning: #warning "Unsupported compiler detected"
Status: Parsing /opt/local/var/macports/build/_opt_mports_dports_gis_grass/grass/work/.tmp/tmp1UOQVF.h
Error: /usr/include/stdio.h:252: Syntax error at '__attribute__'
Error: /usr/include/stdio.h:252: Syntax error at 'i2'
Error: /usr/include/stdio.h:258: Syntax error at '__attribute__'
Error: /usr/include/stdio.h:258: Syntax error at 'i2'
Error: /usr/include/stdio.h:267: Syntax error at '__attribute__'
Error: /usr/include/stdio.h:267: Syntax error at 'i1'
Error: /usr/include/stdio.h:274: Syntax error at '__attribute__'
Error: /usr/include/stdio.h:274: Syntax error at 'i1'
Error: /usr/include/stdio.h:277: Syntax error at '__attribute__'
Error: /usr/include/stdio.h:277: Syntax error at 'i2'

See also #32797 and #32790.

(Full log is here.)

Change History (11)

comment:1 Changed 7 years ago by manuel.deljesuspe@…

Cc: manuel.deljesuspe@… added

Cc Me!

comment:2 Changed 7 years ago by cooljeanius (Eric Gallager)

See also #32797 and #32790.

Especially the first one of those. Making sure that grass is UsingTheRightCompiler definitely seems like it should fix this.

comment:3 Changed 7 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:4 Changed 7 years ago by mojca (Mojca Miklavec)

Are you willing to investigate and come up with a patch?

comment:5 in reply to:  4 Changed 7 years ago by cooljeanius (Eric Gallager)

Replying to mojca@…:

Are you willing to investigate and come up with a patch?

Well, I suppose I could copy over the port and mess around with it in my local portfile repo... I'm on 10.6, though, so I won't be able to solve this exact error, because I don't have access to 10.8... I might be able to deal with the more general issue of UsingTheRightCompiler though...

comment:6 Changed 7 years ago by mojca (Mojca Miklavec)

It might indeed be the case that solving that one would also solve this problem. So yes, please go forward.

Btw: would you be interested in becoming the maintainer of grass?

comment:7 Changed 7 years ago by cooljeanius (Eric Gallager)

OK, forked it, and now I'm running into #40315 (or something similar to it)... idk about becoming a maintainer, as there are already a bunch of tickets open against it that I'm not sure I'd be able to solve...

comment:8 Changed 7 years ago by mojca (Mojca Miklavec)

There are currently six tickets open. If you managed to solve the right compiler (#32797), this might close two tickets (including this one).

Another two tickets (#36904/#40333) are again the same issue that is in my opinion just a matter of discussion, making some decision and following that decision in all wxPython-2.8 dependencies.

Then you are only left with:

  • #36594 that should eventually be solved upstream and would probably be avoided if wxWidgets 2.8 was used (the other two tickets); your only task could be complaining about the problem upstream
  • #40315 which could be partially avoided if wxWidgets was made non-optional (complete solution could also involve requesting an upstream fix)

Becoming the maintainer would also be a great chance to get some additional experience. Being a maintainer doesn't mean that you need to be able to solve every single problem on your own.

Last edited 7 years ago by mojca (Mojca Miklavec) (previous) (diff)

comment:9 Changed 7 years ago by mojca (Mojca Miklavec)

I now made some changes in SVN and version 6.4.3 seems to build OK on all four buildbots:

Now, it's true that I was the one making the report, but I don't have a 10.8 machine and it seems to build fine on the buildbot now. It's true that I switched to Python 2.6 and wxWidgets 2.8 and that both are now required (not optional).

I would close this ticket now until someone else bumps into it or unless someone else figures out what I'm missing.

comment:10 in reply to:  8 Changed 7 years ago by cooljeanius (Eric Gallager)

Replying to mojca@…:

Becoming the maintainer would also be a great chance to get some additional experience. Being a maintainer doesn't mean that you need to be able to solve every single problem on your own.

Well, maybe once I have commit access, or if someone who does already have commit access agrees to co-maintain it with me (and openmaintainer)... To be honest though I haven't touched my copy of the grass Portfile in like 6 months, so I'm not sure how high a priority I'd be able to give it... I will try to integrate some of your changes from #44056 (i.e. r121097, r121101, r121102, and r121115) into my copy though, and see how that goes...

comment:11 Changed 7 years ago by mojca (Mojca Miklavec)

Please note that Grass 7 seems to be a completely different beast when it comes to the build process. But at this point it probably makes more sense to concentrate on version 7 than to spend time on 6.4.

I feel guilty for not taking more time to finish discussing patches with you.

Note: See TracTickets for help on using tickets.