Merge lp:~robert-ancell/unity-greeter/no-warnings into lp:unity-greeter

Proposed by Robert Ancell
Status: Merged
Approved by: Michael Terry
Approved revision: 988
Merged at revision: 992
Proposed branch: lp:~robert-ancell/unity-greeter/no-warnings
Merge into: lp:unity-greeter
Diff against target: 25 lines (+2/-2)
2 files modified
src/Makefile.am (+1/-1)
tests/Makefile.am (+1/-1)
To merge this branch: bzr merge lp:~robert-ancell/unity-greeter/no-warnings
Reviewer Review Type Date Requested Status
Michael Terry (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+191732@code.launchpad.net

Commit message

Disable C warnings - Vala generates warnings that can't be fixed and generally aren't a problem

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michael Terry (mterry) wrote :

Yup, makes sense. I do the same thing in deja-dup.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/Makefile.am'
2--- src/Makefile.am 2013-02-13 20:46:08 +0000
3+++ src/Makefile.am 2013-10-17 22:10:37 +0000
4@@ -34,7 +34,7 @@
5
6 unity_greeter_CFLAGS = \
7 $(UNITY_GREETER_CFLAGS) \
8- $(WARN_CFLAGS) \
9+ -w \
10 -DGETTEXT_PACKAGE=\"$(GETTEXT_PACKAGE)\" \
11 -DLOCALEDIR=\""$(localedir)"\" \
12 -DVERSION=\"$(VERSION)\" \
13
14=== modified file 'tests/Makefile.am'
15--- tests/Makefile.am 2013-03-15 18:49:28 +0000
16+++ tests/Makefile.am 2013-10-17 22:10:37 +0000
17@@ -39,7 +39,7 @@
18
19 unity_greeter_test_CFLAGS = \
20 $(UNITY_GREETER_CFLAGS) \
21- $(WARN_CFLAGS) \
22+ -w \
23 -DGETTEXT_PACKAGE=\"$(GETTEXT_PACKAGE)\" \
24 -DLOCALEDIR=\""$(localedir)"\" \
25 -DVERSION=\"$(VERSION)\" \

Subscribers

People subscribed via source and target branches