Opened 11 years ago

Closed 11 months ago

#40307 closed defect (wontfix)

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 (13)

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

Cc: manuel.deljesuspe@… added

Cc Me!

comment:2 Changed 11 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 11 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

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

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

comment:5 in reply to:  4 Changed 11 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 11 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 11 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 11 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 11 years ago by mojca (Mojca Miklavec) (previous) (diff)

comment:9 Changed 11 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 10 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 10 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.

comment:12 Changed 19 months ago by nilason (Nicklas Larsson)

May I suggest to close this ticket. The reported errors are issued by ctypesgen's limitation of parsing certain (secondary and below included) system files. Even now, with this port having reached 8.2.1, similar warnings are being issued, but are all absolutely harmless in regards to the functioning of GRASS (the ctypesgen generated python interface for GRASS API is properly being created).

comment:13 Changed 11 months ago by Schamschula (Marius Schamschula)

Resolution: wontfix
Status: newclosed
Note: See TracTickets for help on using tickets.