Fix to the reorder_addlibs.sh script (was not handling library names with numbers in them correctly) and the corresponding fix to Makefiles.

Author: danpovey

Diff for: misc/maintenance/reorder_addlibs.sh

@@ -46,7 +46,7 @@ for f in */Makefile; do
         @weird_libs = ();
         %normal_lib_names = {};
         foreach $lib (@addlibs) {
-          if ($lib =~ m|^\.\./([a-z]+)/kaldi-([a-z]+)\.a$| && $1 == $2 && defined $library_name_to_order{$1}) {
+          if ($lib =~ m|^\.\./(.+)/kaldi-(.+)\.a$| && $1 == $2 && defined $library_name_to_order{$1}) {
              $normal_lib_names{$1} = 1;
           } else { push @weird_libs, $lib; }
         }
@@ -57,7 +57,7 @@ for f in */Makefile; do
                push @normalized_addlibs, "../$test_name/kaldi-$test_name.a";
             }
          }
-        if (@weird_libs > 0) {  print STDERR "Unexpected libraries: " . join(":", @weird_libs); }
+        if (@weird_libs > 0) {  print STDERR "Unexpected libraries: " . join(":", @weird_libs) . "\n"; }
         # unexpected libraries that aren not part of the normal list will go last.
         push @normalized_addlibs, @weird_libs;
         @rearranged_lines = ();

Diff for: src/chainbin/Makefile

@@ -20,11 +20,11 @@ cuda-compiled.o: ../kaldi.mk
 
 TESTFILES =
 
-ADDLIBS = ../chain/kaldi-chain.a ../cudamatrix/kaldi-cudamatrix.a \
-          ../decoder/kaldi-decoder.a ../lat/kaldi-lat.a \
-          ../fstext/kaldi-fstext.a ../hmm/kaldi-hmm.a \
+ADDLIBS = ../nnet3/kaldi-nnet3.a ../chain/kaldi-chain.a \
+          ../cudamatrix/kaldi-cudamatrix.a ../decoder/kaldi-decoder.a \
+          ../lat/kaldi-lat.a ../fstext/kaldi-fstext.a ../hmm/kaldi-hmm.a \
           ../transform/kaldi-transform.a ../gmm/kaldi-gmm.a \
           ../tree/kaldi-tree.a ../util/kaldi-util.a ../thread/kaldi-thread.a \
-          ../matrix/kaldi-matrix.a ../base/kaldi-base.a ../nnet3/kaldi-nnet3.a 
+          ../matrix/kaldi-matrix.a ../base/kaldi-base.a 
 
 include ../makefiles/default_rules.mk

Diff for: src/nnet2bin/Makefile

@@ -37,11 +37,11 @@ cuda-compiled.o: ../kaldi.mk
 
 TESTFILES =
 
-ADDLIBS = ../nnet/kaldi-nnet.a ../cudamatrix/kaldi-cudamatrix.a \
-          ../decoder/kaldi-decoder.a ../lat/kaldi-lat.a \
-          ../fstext/kaldi-fstext.a ../hmm/kaldi-hmm.a \
+ADDLIBS = ../nnet2/kaldi-nnet2.a ../nnet/kaldi-nnet.a \
+          ../cudamatrix/kaldi-cudamatrix.a ../decoder/kaldi-decoder.a \
+          ../lat/kaldi-lat.a ../fstext/kaldi-fstext.a ../hmm/kaldi-hmm.a \
           ../transform/kaldi-transform.a ../gmm/kaldi-gmm.a \
           ../tree/kaldi-tree.a ../util/kaldi-util.a ../thread/kaldi-thread.a \
-          ../matrix/kaldi-matrix.a ../base/kaldi-base.a ../nnet2/kaldi-nnet2.a 
+          ../matrix/kaldi-matrix.a ../base/kaldi-base.a 
 
 include ../makefiles/default_rules.mk

Diff for: src/nnet3bin/Makefile

@@ -26,11 +26,11 @@ cuda-compiled.o: ../kaldi.mk
 
 TESTFILES =
 
-ADDLIBS = ../chain/kaldi-chain.a ../cudamatrix/kaldi-cudamatrix.a \
-          ../decoder/kaldi-decoder.a ../lat/kaldi-lat.a \
-          ../fstext/kaldi-fstext.a ../hmm/kaldi-hmm.a \
+ADDLIBS = ../nnet3/kaldi-nnet3.a ../chain/kaldi-chain.a \
+          ../cudamatrix/kaldi-cudamatrix.a ../decoder/kaldi-decoder.a \
+          ../lat/kaldi-lat.a ../fstext/kaldi-fstext.a ../hmm/kaldi-hmm.a \
           ../transform/kaldi-transform.a ../gmm/kaldi-gmm.a \
           ../tree/kaldi-tree.a ../util/kaldi-util.a ../thread/kaldi-thread.a \
-          ../matrix/kaldi-matrix.a ../base/kaldi-base.a ../nnet3/kaldi-nnet3.a 
+          ../matrix/kaldi-matrix.a ../base/kaldi-base.a 
 
 include ../makefiles/default_rules.mk

Diff for: src/online2/Makefile

@@ -13,13 +13,13 @@ OBJFILES = online-gmm-decodable.o online-feature-pipeline.o online-ivector-featu
 
 LIBNAME = kaldi-online2
 
-ADDLIBS = ../ivector/kaldi-ivector.a ../chain/kaldi-chain.a \
+ADDLIBS = ../ivector/kaldi-ivector.a ../nnet3/kaldi-nnet3.a \
+          ../chain/kaldi-chain.a ../nnet2/kaldi-nnet2.a \
           ../cudamatrix/kaldi-cudamatrix.a ../decoder/kaldi-decoder.a \
           ../lat/kaldi-lat.a ../hmm/kaldi-hmm.a ../feat/kaldi-feat.a \
           ../transform/kaldi-transform.a ../gmm/kaldi-gmm.a \
           ../tree/kaldi-tree.a ../util/kaldi-util.a ../thread/kaldi-thread.a \
-          ../matrix/kaldi-matrix.a ../base/kaldi-base.a ../nnet2/kaldi-nnet2.a \
-          ../nnet3/kaldi-nnet3.a 
+          ../matrix/kaldi-matrix.a ../base/kaldi-base.a 
 
 
 

Diff for: src/online2bin/Makefile

@@ -17,12 +17,12 @@ OBJFILES =
 
 TESTFILES =
 
-ADDLIBS = ../ivector/kaldi-ivector.a ../chain/kaldi-chain.a \
+ADDLIBS = ../online2/kaldi-online2.a ../ivector/kaldi-ivector.a \
+          ../nnet3/kaldi-nnet3.a ../chain/kaldi-chain.a ../nnet2/kaldi-nnet2.a \
           ../cudamatrix/kaldi-cudamatrix.a ../decoder/kaldi-decoder.a \
           ../lat/kaldi-lat.a ../fstext/kaldi-fstext.a ../hmm/kaldi-hmm.a \
           ../feat/kaldi-feat.a ../transform/kaldi-transform.a \
           ../gmm/kaldi-gmm.a ../tree/kaldi-tree.a ../util/kaldi-util.a \
           ../thread/kaldi-thread.a ../matrix/kaldi-matrix.a \
-          ../base/kaldi-base.a ../online2/kaldi-online2.a \
-          ../nnet2/kaldi-nnet2.a ../nnet3/kaldi-nnet3.a 
+          ../base/kaldi-base.a 
 include ../makefiles/default_rules.mk

Diff for: src/sgmm2bin/Makefile

@@ -18,9 +18,10 @@ TESTFILES =
 
 
 ADDLIBS = ../decoder/kaldi-decoder.a ../lat/kaldi-lat.a \
-          ../fstext/kaldi-fstext.a ../hmm/kaldi-hmm.a ../feat/kaldi-feat.a \
-          ../transform/kaldi-transform.a ../gmm/kaldi-gmm.a \
-          ../tree/kaldi-tree.a ../util/kaldi-util.a ../thread/kaldi-thread.a \
-          ../matrix/kaldi-matrix.a ../base/kaldi-base.a ../sgmm2/kaldi-sgmm2.a 
+          ../fstext/kaldi-fstext.a ../sgmm2/kaldi-sgmm2.a ../hmm/kaldi-hmm.a \
+          ../feat/kaldi-feat.a ../transform/kaldi-transform.a \
+          ../gmm/kaldi-gmm.a ../tree/kaldi-tree.a ../util/kaldi-util.a \
+          ../thread/kaldi-thread.a ../matrix/kaldi-matrix.a \
+          ../base/kaldi-base.a 
 
 include ../makefiles/default_rules.mk