After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 700066 - CC is not passed to g-ir-scanner
CC is not passed to g-ir-scanner
Status: RESOLVED FIXED
Product: gobject-introspection
Classification: Platform
Component: g-ir-scanner
unspecified
Other Windows
: Normal normal
: ---
Assigned To: gobject-introspection Maintainer(s)
gobject-introspection Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-05-10 12:36 UTC by LRN
Modified: 2015-04-18 19:13 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Makefile.introspection: pass *FLAGS to scanner (2.40 KB, patch)
2015-03-30 21:28 UTC, Dieter Verfaillie 		none Details | Review
Makefile.introspection: pass CC to scanner (2.40 KB, patch)
2015-03-30 21:29 UTC, Dieter Verfaillie 		none Details | Review
Makefile.introspection: pass CC to scanner (2.40 KB, patch)
2015-04-18 19:13 UTC, Dieter Verfaillie 		none Details | Review

Description LRN 2013-05-10 12:36:26 UTC 
gobject-introspection\giscanner\sourcescanner.py", line 292 - scanner tries to get CC environmental variable, but it's not defined. Makefiles know the CC, but that isn't an envvar, so it isn't passed to the programs.

Overriding g_ir_scanner in the .pc file (prepending CC="$(CC)" to it) solves the problem for me.
Comment 1 LRN 2013-05-10 19:34:46 UTC 
Another way to work around this is to call make like this:
make INTROSPECTION_SCANNER_ENV=CC="\$(CC)"
Comment 2 Ignacio Casal Quinteiro (nacho) 2014-08-02 08:13:18 UTC 
Can we get a solution here? Maybe having it added to the Makefile.introspection? Having to add this to all the modules of the world is not very nice...
Comment 3 Ignacio Casal Quinteiro (nacho) 2015-02-04 12:20:11 UTC 
ping...
Comment 4 LRN 2015-02-04 14:31:28 UTC 
AFAIK, changes that fix this are slowly seeping into various GI'able pieces of software.

Right now i'm only patching gtk+-2.x (by adding INTROSPECTION_SCANNER_ENV=... to Makefile.am in gdk and gtk).
Comment 5 André Klapper 2015-02-07 17:16:25 UTC 
[Mass-moving gobject-introspection tickets to its own Bugzilla product - see bug 708029. Mass-filter your bugmail for this message: introspection20150207 ]
Comment 6 Garrett Regier 2015-03-24 13:48:25 UTC 
Ping? Another project has the same issue, it would be nice to just fix it once and forall.
Comment 7 Dieter Verfaillie 2015-03-30 21:28:41 UTC 
Created attachment 300628 [details] [review]
Makefile.introspection: pass *FLAGS to scanner

Extends commit 92d9c38 to also pass CC to g-ir-scanner.
Comment 8 Dieter Verfaillie 2015-03-30 21:29:44 UTC 
Created attachment 300629 [details] [review]
Makefile.introspection: pass CC to scanner

Extends commit 92d9c38 to also pass CC to g-ir-scanner.
Comment 9 Ignacio Casal Quinteiro (nacho) 2015-04-17 19:12:51 UTC 
(In reply to Dieter Verfaillie from comment #8)
> Created attachment 300629 [details] [review] [review]
> Makefile.introspection: pass CC to scanner
> 
> Extends commit 92d9c38 to also pass CC to g-ir-scanner.

Any chance to get this in? The patch seems fine to me.
Comment 10 Dieter Verfaillie 2015-04-18 19:12:59 UTC 
The following fix has been pushed:
1f3db78 Makefile.introspection: pass CC to scanner

(In reply to Ignacio Casal Quinteiro (nacho) from comment #9)
> Any chance to get this in? The patch seems fine to me.

Thanks for the review, that's the only reason this patch was on hold.
Comment 11 Dieter Verfaillie 2015-04-18 19:13:04 UTC 
Created attachment 301908 [details] [review]
Makefile.introspection: pass CC to scanner

Extends commit 92d9c38 to also pass CC to g-ir-scanner.