Menu

#5783 Use $(MAKE) instead of 'make' in lilypond-book regtests

Started
None
abandoned
2020-02-23
2020-02-22
No

This fixes parallel build of lilypond-book tests

https://codereview.appspot.com/569400043

Discussion

  • Han-Wen Nienhuys

     
  • Anonymous

    Anonymous - 2020-02-23
    • Description has changed:

    Diff:

    
    
    • Needs: -->
    • Patch: new --> review
    • Type: -->
     
  • Anonymous

    Anonymous - 2020-02-23

    Passes make, make test-baseline and a full make doc. Cannot do a proper test though (i.e. with make check).

    When I try to test this in the 'normal' way it fails even if I use the config.status --recheck && touch config.hh workaround for 'build' files.

    So I am assuming that as this seem sto be changing dir paths that this is going to be expected?

     
    • David Kastrup

      David Kastrup - 2020-02-23

      As this is a change in the build system, it is a humongous red flag if if fails after running configure. How does it fail?

       
  • Han-Wen Nienhuys

    • Patch: review --> abandoned