summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2011-01-18 07:20:26 +0100
committerStefan Metzmacher <metze@samba.org>2011-01-18 08:10:06 +0100
commitd5173ca189a080d0bb3a56200203c32a40c4b6e3 (patch)
tree2c549a7de30930f4deb882393498a0b116d52524
parent1667ff79caa91d6bd97df127fb4a3b3d0fa5c3a1 (diff)
downloadsamba-d5173ca189a080d0bb3a56200203c32a40c4b6e3.tar.gz
samba-d5173ca189a080d0bb3a56200203c32a40c4b6e3.tar.bz2
samba-d5173ca189a080d0bb3a56200203c32a40c4b6e3.zip
pidl:wscript: don't warn about pidl gammar file changes for now
We may add some logic that uses git diff HEAD to detect this changes in developer mode later again. metze Autobuild-User: Stefan Metzmacher <metze@samba.org> Autobuild-Date: Tue Jan 18 08:10:06 CET 2011 on sn-devel-104
-rw-r--r--pidl/wscript6
1 files changed, 4 insertions, 2 deletions
diff --git a/pidl/wscript b/pidl/wscript
index 5b3e07c059..e60ca202f5 100644
--- a/pidl/wscript
+++ b/pidl/wscript
@@ -59,11 +59,13 @@ def build(bld):
# Only if the source has changed do we want to re-run yapp
# But we force the developer to use the pidl standalone build
# to regenerate the files.
- need_yapp_build = ('YAPP' in bld.env and (
+ # TODO: only warn in developer mode and if 'git diff HEAD'
+ # shows a difference
+ warn_about_grammar_changes = ('PIDL_BUILD_WARNINGS' in bld.env and (
bld.IS_NEWER('idl.yp', 'lib/Parse/Pidl/IDL.pm') or
bld.IS_NEWER('expr.yp', 'lib/Parse/Pidl/Expr.pm')))
- if need_yapp_build:
+ if warn_about_grammar_changes:
Logs.warn('''
Pidl grammar files have changed. Please use the pidl standalone build
to regenerate them with yapp.