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

Issue 4950053: Fix 1821: Write pathes for ly-to-tely to a separate file rather than passing as cmd line args (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 8 months ago by Reinhold
Modified:
12 years, 7 months ago
Reviewers:
wl, Graham Percival (old account), reinhold
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Fix 1821: Write pathes for ly-to-tely to a separate file rather than passing as cmd line args For out-of-source builds, the collated files list has the whole absolute path for each and every file (>1000 for our regtests), which blows up the cmd line length to values that don't work with the shell (<=131000 chars allowed...). This patch takes the list in the makefile and writes the filenames in several patches (300 at a time for now) to a .list file and then passes just this one file to lys-to-tely, which reads in the whole long list of file names from that file. The trick is that while make can handle long variables, we can't pass it to ly-to-tely on the cmd line, but have to use a helper file.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -2 lines) Patch
M make/lysdoc-rules.make View 1 chunk +19 lines, -2 lines 0 comments Download

Messages

Total messages: 3
Graham Percival (old account)
sweet mao that's disgusting. I love it. :) If you can make doc from scratch, ...
12 years, 8 months ago (2011-08-31 04:26:59 UTC) #1
reinhold_kainhofer.com
Am Mittwoch, 31. August 2011, 06:26:59 schrieb percival.music.ca@gmail.com: > sweet mao that's disgusting. I love ...
12 years, 8 months ago (2011-08-31 12:56:59 UTC) #2
wl_gnu.org
12 years, 8 months ago (2011-08-31 17:55:56 UTC) #3
> Actually, with my test setup, I can't even build lilypond, because
> metafont already fails with some long pathes: [...]
> 
> Any idea?

Please report this to the texlive@tug.org


    Werner
Sign in to reply to this message.

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