LGTM, with nits. https://codereview.appspot.com/577700045/diff/547810068/flower/GNUmakefile File flower/GNUmakefile (right): https://codereview.appspot.com/577700045/diff/547810068/flower/GNUmakefile#ne... flower/GNUmakefile:17: TEST_LOADLIBES = $(LIBRARY) $(CXXABI_LIBS) I suggest s/TEST_LOADLIBES/TEST_LOADLIBS/. https://codereview.appspot.com/577700045/diff/547810068/flower/GNUmakefile#ne... flower/GNUmakefile:33: AR=ar s/AR=ar/AR = ar/ for consistency. Irrespective of that I wonder why setting `AR` is necessary at all since the configure script sets this variable (via the `STEPMAKE_LIB` macro) – it seems to make that it won't work with cross-compiling.
commit 719b9352fd8f3fb0a904ba7cfde402d8352e0eac Author: Han-Wen Nienhuys <hanwen@lilypond.org> Date: Sat Mar 21 23:37:34 2020 +0100 Cleanup flower/ makefile