]> git.ipfire.org Git - thirdparty/automake.git/commitdiff
[ng] refactor: move hack for libtool installs from automake to install.am
authorStefano Lattarini <stefano.lattarini@gmail.com>
Mon, 30 Jul 2012 14:58:40 +0000 (16:58 +0200)
committerStefano Lattarini <stefano.lattarini@gmail.com>
Mon, 30 Jul 2012 15:08:17 +0000 (17:08 +0200)
* automake.in (generate_makefile): Move the hack necessary to make the
installation of libtool libraries and binaries dependent on them work
"on degenerate systems even with make -j" (according to the comments of
the original authors :-) from here ...
* lib/am/install.am: ... to here, with improved comments.

For the origin of the hack we are moving around here, refer to commit
bd4a1d5 of 2000-10-19, "* automake.in (handle_merge_targets): Allow
parallel install with forced relink".

Signed-off-by: Stefano Lattarini <stefano.lattarini@gmail.com>
automake.in
lib/am/install.am

index 10d03ca3847087377338678534a114e28139ad41..bbdd130e41efc5d4705455c46be458e48ebea481 100644 (file)
@@ -6895,16 +6895,6 @@ sub generate_makefile ($$)
   # Special targets "all" and "check".
   handle_all_and_check;
 
-  # FIXME: Gross!
-  if (var ('lib_LTLIBRARIES') && var ('bin_PROGRAMS'))
-    {
-      $output_rules .= "install-binPROGRAMS: install-libLTLIBRARIES\n\n";
-    }
-  if (var ('nobase_lib_LTLIBRARIES') && var ('bin_PROGRAMS'))
-    {
-      $output_rules .= "install-binPROGRAMS: install-nobase_libLTLIBRARIES\n\n";
-    }
-
   almost_verbatim ('install');
   handle_clean ($makefile);
   handle_factored_dependencies;
index 197290547946931d4c18513344ca45c2d89b34fb..a87e172059281365b31361a6cfb397d0a302d081 100644 (file)
@@ -94,3 +94,17 @@ install-strip:
                INSTALL_STRIP_FLAG=-s \
                $(if $(STRIP),"INSTALL_PROGRAM_ENV=STRIPPROG='$(STRIP)'") \
                install
+
+# Allow parallel install with forced relink.  See commit Automake bd4a1d5
+# of 2000-10-19 for a little more background.
+# FIXME: this is gross, and is debatable how useful and/or needed this
+# workaround still is today.  This is something that should be eventually
+# discussed with the Libtool guys.
+ifdef bin_PROGRAMS
+  ifdef lib_LTLIBRARIES
+    install-binPROGRAMS: install-libLTLIBRARIES
+  endif
+  ifdef nobase_lib_LTLIBRARIES
+    install-binPROGRAMS: install-nobase_libLTLIBRARIES
+  endif
+endif