Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1750)

Issue 553080043: Issue 5572: Compile modified C++ files automatically before testing (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 6 months ago by Dan Eble
Modified:
4 years, 6 months ago
Reviewers:
lemzwerg
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

https://sourceforge.net/p/testlilyissues/issues/5572/ Issue 5572/1: Compile modified C++ files automatically before testing The main change is to make sure lilypond and scripts are up to date before running tests. Build fonts too, though this is only sufficient to handle builds following a `make clean`. Other changes: * Make a single baseline copy of fonts instead of replicating them in every subdirectory with tests. * If the developer has forgotten to create a baseline, try to catch it quickly. * Fix warnings about multiple sets of rules for local-test by renaming one of the targets lysdoc-test. Issue 5572/2: Contributor's Guide updates

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+105 lines, -67 lines) Patch
M Documentation/contributor/introduction.itexi View 1 chunk +5 lines, -4 lines 0 comments Download
M Documentation/contributor/programming-work.itexi View 3 chunks +24 lines, -25 lines 0 comments Download
M Documentation/contributor/regressions.itexi View 4 chunks +21 lines, -23 lines 0 comments Download
M GNUmakefile.in View 1 chunk +34 lines, -12 lines 0 comments Download
M make/lysdoc-targets.make View 2 chunks +11 lines, -3 lines 0 comments Download
M stepmake/stepmake/generic-vars.make View 1 chunk +9 lines, -0 lines 0 comments Download
M stepmake/stepmake/texinfo-rules.make View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2
Dan Eble
4 years, 6 months ago (2019-10-16 01:40:45 UTC) #1
lemzwerg
4 years, 6 months ago (2019-10-16 09:54:34 UTC) #2
Without testing yet, this looks very nice, thanks!
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b