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

Issue 341050043: makelsr.py: don't delete the output directory (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 1 month ago by Dan Eble
Modified:
1 year, 1 month ago
Reviewers:
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

makelsr.py: don't delete the output directory If the output directory is not a directory, don't just delete it. Abort and let the user decide how to address it. I am pretty confident that these changes are good, but I would be grateful if someone familiar with this script would try running it because I can not test the full script myself. (I have set up my build environment without write access to the source, so updating source files fails. Yes, I have the expertise to change the approach, but I lack the interest.)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -25 lines) Patch
M scripts/auxiliar/makelsr.py View 1 chunk +13 lines, -25 lines 0 comments Download

Messages

Total messages: 1
Dan Eble
1 year, 1 month ago (2018-04-02 18:08:57 UTC) #1

          
Sign in to reply to this message.

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