Menu

#5814 stepmake: use patsubst for finding build-dir

Fixed
2020-03-07
2020-03-01
No

The previous approach used $(subst ) which replaces all occurrences.
This lead to surprising results when the top-build-dir was called
/lilypond, as it is in the Docker builds.

https://codereview.appspot.com/575750043

Discussion

  • Anonymous

    Anonymous - 2020-03-01
    • Description has changed:

    Diff:

    
    
    • Needs: -->
    • Patch: new --> needs_work
     
  • Anonymous

    Anonymous - 2020-03-01

    I cannot get this patch to run past the 'configure' after applying it (after I have done the make/make test-baseline'. I get a stepmake error.

     
  • Han-Wen Nienhuys

     
  • Anonymous

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

    Anonymous - 2020-03-03

    Passes make, make check and a full make doc.

     
  • Anonymous

    Anonymous - 2020-03-05
    • Patch: review --> countdown
     
  • Anonymous

    Anonymous - 2020-03-05

    Patch on countdown for March 7th

     
  • Anonymous

    Anonymous - 2020-03-07
    • Patch: countdown --> push
     
  • Anonymous

    Anonymous - 2020-03-07

    Patch counted down - please push.

     
  • Anonymous

    Anonymous - 2020-03-07
    • labels: --> Fixed 2_21_0
    • status: Started --> Fixed
    • Patch: push -->
     
  • Anonymous

    Anonymous - 2020-03-07
    stepmake: use patsubst for finding build-dir master staging
    author  Han-Wen Nienhuys <hanwen@lilypond.org>  
        Sun, 1 Mar 2020 13:03:36 +0000 (14:03 +0100)
    committer   Han-Wen Nienhuys <hanwen@lilypond.org>  
        Sat, 7 Mar 2020 10:33:05 +0000 (11:33 +0100)
    commit  febe487bb45c97f97377536a5d15da80cce80297