svn commit: r1149401 - /subversion/trunk/build/generator/swig/header_wrappers.py

[da...@apache.org]

Author: danielsh
Date: Thu Jul 21 23:00:44 2011
New Revision: 1149401

URL: http://svn.apache.org/viewvc?rev=1149401&view=rev
Log:
Cause 'make swig-pl' to trigger the 'SWIG disabled at configure time'
error, like 'swig-py' and 'swig-rb' already do.

* build/generator/swig/header_wrappers.py
  (write_makefile_rules): Make 'autogen-swig-%s' depend on '.swig_checked'.
    Add comment with Arfrever's explanation as to why that is only necessary
    for the swig-pl bindings, but not for swig-py and swig-rb.

Modified:
    subversion/trunk/build/generator/swig/header_wrappers.py

Modified: subversion/trunk/build/generator/swig/header_wrappers.py
URL: http://svn.apache.org/viewvc/subversion/trunk/build/generator/swig/header_wrappers.py?rev=1149401&r1=1149400&r2=1149401&view=diff
==============================================================================
--- subversion/trunk/build/generator/swig/header_wrappers.py (original)
+++ subversion/trunk/build/generator/swig/header_wrappers.py Thu Jul 21 23:00:44 2011
@@ -65,7 +65,13 @@ class Generator(generator.swig.Generator
       )
     makefile.write('SWIG_WRAPPERS = %s\n\n' % ' '.join(wrapper_fnames))
     for short_name in self.short.values():
-      makefile.write('autogen-swig-%s: $(SWIG_WRAPPERS)\n' % short_name)
+      # swig-pl needs the '.swig_checked' target here; swig-rb and swig-py
+      # already reach it via a different dependency chain:
+      #
+      #    In build-outputs.mk, swig-py and swig-rb targets depend on *.la
+      #    targets, which depend on *.lo targets, which depend on *.c targets,
+      #    which depend on .swig_checked target.
+      makefile.write('autogen-swig-%s: .swig_checked $(SWIG_WRAPPERS)\n' % short_name)
     makefile.write('\n\n')
 
   def proxy_filename(self, include_filename):