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 692055 - Update .gitignore and autoconf
Update .gitignore and autoconf
Status: RESOLVED FIXED
Product: gnome-terminal
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: GNOME Terminal Maintainers
GNOME Terminal Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-01-19 05:10 UTC by Jasper St. Pierre (not reading bugmail)
Modified: 2013-01-20 17:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
src: Make sure to include git.mk (570 bytes, patch)
2013-01-19 05:10 UTC, Jasper St. Pierre (not reading bugmail)
none Details | Review
Update copy of git.mk (6.66 KB, patch)
2013-01-19 05:11 UTC, Jasper St. Pierre (not reading bugmail)
none Details | Review
configure.ac: Use AM_PATH_GLIB_2_0 instead of custom PATH_PROG goo (1.42 KB, patch)
2013-01-19 05:11 UTC, Jasper St. Pierre (not reading bugmail)
none Details | Review
configure.ac: Replace custom warnings goo with GNOME_COMPILE_WARNINGS (1.32 KB, patch)
2013-01-19 05:11 UTC, Jasper St. Pierre (not reading bugmail)
none Details | Review

Description Jasper St. Pierre (not reading bugmail) 2013-01-19 05:10:56 UTC
See patches. I couldn't get rid of omf.make and xmldocs.make,
I assume those are pulled in by gconf or something else.
Comment 1 Jasper St. Pierre (not reading bugmail) 2013-01-19 05:10:58 UTC
Created attachment 233841 [details] [review]
src: Make sure to include git.mk
Comment 2 Jasper St. Pierre (not reading bugmail) 2013-01-19 05:11:00 UTC
Created attachment 233842 [details] [review]
Update copy of git.mk
Comment 3 Jasper St. Pierre (not reading bugmail) 2013-01-19 05:11:02 UTC
Created attachment 233843 [details] [review]
configure.ac: Use AM_PATH_GLIB_2_0 instead of custom PATH_PROG goo
Comment 4 Jasper St. Pierre (not reading bugmail) 2013-01-19 05:11:05 UTC
Created attachment 233844 [details] [review]
configure.ac: Replace custom warnings goo with GNOME_COMPILE_WARNINGS
Comment 5 Christian Persch 2013-01-20 13:32:44 UTC
I committed the first two patches. I'm not going to use AM_PATH_GLIB_2_0 since IMHO that macro should disappear instead of being used *more* widely. And since we use a longer list of compile warnings anyway, I just updated the list instead of using more gnome-common macros (since also IMHO those macros should disappear).
Comment 6 Jasper St. Pierre (not reading bugmail) 2013-01-20 14:51:50 UTC
(In reply to comment #5)
> I committed the first two patches. I'm not going to use AM_PATH_GLIB_2_0 since
> IMHO that macro should disappear instead of being used *more* widely. And since
> we use a longer list of compile warnings anyway, I just updated the list
> instead of using more gnome-common macros (since also IMHO those macros should
> disappear).

I'm curious why. AM_PATH_GLIB_2_0 is one line that does most everything that the current script does, and GNOME_COMPILE_WARNINGS just got a large update to the set of compiler warnings it has.
Comment 7 Christian Persch 2013-01-20 17:05:00 UTC
I can't even count the time lost over the years to the stupid 'pkg-config said version x, but it's really version y' check in that macro.

Unless I'm missing something, GNOME_COMPILE_WARNINGS just has a very minimal set of flags, much less than the list g-t uses currently ?