python: Fix "make distcheck" error on version.py.
authorBen Pfaff <blp@nicira.com>
Mon, 12 Mar 2012 17:34:22 +0000 (10:34 -0700)
committerBen Pfaff <blp@nicira.com>
Mon, 19 Mar 2012 18:04:02 +0000 (11:04 -0700)
The generated version.py has to go in the srcdir and has to be regenerated
based on config.status, which breaks "make distcheck" because it
write-protects the srcdir.  However, the contents of version.py only change
when the version number changes, so we can just "touch" it when it doesn't
really need to change.

The same pattern is used elsewhere in the tree for other files in the same
situation, e.g. the various RPM spec files.

Reported-by: Chris Wright <chrisw@sous-sol.org>
Signed-off-by: Ben Pfaff <blp@nicira.com>
python/automake.mk

index f9d2c57..db0683f 100644 (file)
@@ -62,7 +62,7 @@ ovs-uninstall-local:
 
 ALL_LOCAL += $(srcdir)/python/ovs/version.py
 $(srcdir)/python/ovs/version.py: config.status
-       $(ro_shell) > $@.tmp
-       echo 'VERSION = "$(VERSION)"' >> $@.tmp
-       echo 'BUILDNR = "$(BUILDNR)"' >> $@.tmp
-       mv $@.tmp $@
+       $(ro_shell) > $(@F).tmp
+       echo 'VERSION = "$(VERSION)"' >> $(@F).tmp
+       echo 'BUILDNR = "$(BUILDNR)"' >> $(@F).tmp
+       if cmp -s $(@F).tmp $@; then touch $@; rm $(@F).tmp; else mv $(@F).tmp $@; fi