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

Issue 565170043: Issue 5583: www_post.py: don't mirror regtest baseline (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:
dak, dan, pkx166h
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

https://sourceforge.net/p/testlilyissues/issues/5583/ I ran into a problem running "make ... doc" today, where www_post was trying unsuccessfully to mirror a symlink in an out-test-baseline subdirectory. I looked around for information that would justify mirroring the baseline and found none, so I've solved the problem by excluding it. If you know better, please speak up.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M scripts/build/www_post.py View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 5
Dan Eble
4 years, 6 months ago (2019-10-23 13:39:24 UTC) #1
pkx166h_runbox.com
Hello, On Wed, 23 Oct 2019 06:39:24 -0700, nine.fierce.ballads@gmail.com wrote: > Reviewers: , > > ...
4 years, 6 months ago (2019-10-23 16:12:06 UTC) #2
dak
"James Lowe" <pkx166h@runbox.com> writes: > Hello, > > On Wed, 23 Oct 2019 06:39:24 -0700, ...
4 years, 6 months ago (2019-10-23 16:36:41 UTC) #3
dan_faithful.be
> On Oct 23, 2019, at 12:36, David Kastrup <dak@gnu.org> wrote: > > "James Lowe" ...
4 years, 6 months ago (2019-10-23 16:46:01 UTC) #4
dak
4 years, 6 months ago (2019-10-23 16:51:52 UTC) #5
Dan Eble <dan@faithful.be> writes:

>> On Oct 23, 2019, at 12:36, David Kastrup <dak@gnu.org> wrote:
>> 
>> "James Lowe" <pkx166h@runbox.com> writes:
>> 
>> Fixes should always be to staging rather than master but I guess that's
>> what you meant.  It depends on just how old the problem is whether one
>> tries to hotfix it.
>
> Baselines created after this commit (a few back from the current tip
> of master) will create out-test-baseline/share as a symlink.
>
>     0d7744aad0 Issue 5572/1: Compile modified C++ files automatically before
testing
> — 
> Dan

I've no qualified opinion about this one.  If you think it's messing up
current testing, feel free to push the fix to staging without waiting
for count-down etc (I doubt there will be much further feedback) if you
are reasonably confident that this will address it.

-- 
David Kastrup
Sign in to reply to this message.

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