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

Issue 13333048: Build everything on the first pass of make. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 7 months ago by Julien Rioux
Modified:
10 years, 7 months ago
Reviewers:
John Mandereau, lemzwerg, mail
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Build everything on the first pass of make. Everything should be built and up-to-date after a one-shot `make' call. Currently, this is not the case: fonts are being rebuilt on the secnod pass. This required a dependency fix for emmentaler fonts. The dependencies are ordered so that fonts aren't rebuilt on a subsequent call to make. For one-shot `make doc', we also have issues, and things are a bit more difficult to track down. Dependencies for web.texi in translated languages are fixed.

Patch Set 1 #

Patch Set 2 : Only sed through an input file if it exists. #

Patch Set 3 : mf/GNUmakefile: Remove unused macros and update a comment. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -11 lines) Patch
M make/doc-i18n-root-rules.make View 1 chunk +1 line, -1 line 0 comments Download
M mf/GNUmakefile View 1 2 3 chunks +6 lines, -9 lines 0 comments Download
M stepmake/stepmake/texinfo-vars.make View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5
Julien Rioux
Please review.
10 years, 7 months ago (2013-09-11 10:04:54 UTC) #1
mail_philholmes.net
----- Original Message ----- From: <julien.rioux@gmail.com> To: <john.mandereau@gmail.com>; <lemzwerg@googlemail.com> Cc: <reply@codereview-hr.appspotmail.com>; <lilypond-devel@gnu.org> Sent: Wednesday, September ...
10 years, 7 months ago (2013-09-12 09:47:24 UTC) #2
Julien Rioux
On Thu, Sep 12, 2013 at 5:47 AM, Phil Holmes <mail@philholmes.net> wrote: > I don't ...
10 years, 7 months ago (2013-09-12 10:25:30 UTC) #3
Julien Rioux
Only sed through an input file if it exists.
10 years, 7 months ago (2013-09-12 16:16:15 UTC) #4
Julien Rioux
10 years, 7 months ago (2013-09-12 16:17:11 UTC) #5
mf/GNUmakefile: Remove unused macros and update a comment.
Sign in to reply to this message.

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