Issue 5625053: Directs regtest output for last 2 files to stderr

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Issue 5625053: Directs regtest output for last 2 files to stderr (Closed)

Can't Edit
Can't Publish+Mail
Start Review

Created:
12 years, 3 months ago by PhilEHolmes

Modified:
12 years, 3 months ago

Reviewers:
Graham Percival, dak

CC:
lilypond-devel_gnu.org

Visibility:
Public.

Description

2 further regtests that send output to stdout as opposed to stderr. Updated to send the output to stderr. This is the last of this series of patches.

Patch Set 1 #

Created: 12 years, 3 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+20 lines, -20 lines)			Patch
	M	input/regression/loglevels.ly	View		1 chunk	+11 lines, -11 lines	0 comments	Download
	M	input/regression/scheme-engraver.ly	View		2 chunks	+9 lines, -9 lines	0 comments	Download

Messages

Total messages: 2

Expand All Messages | Collapse All Messages

dak

12 years, 3 months ago (2012-02-04 16:46:12 UTC) #2

On 2012/02/04 16:39:00, PhilEHolmes wrote:
> Please review.

I am not actually sure that we should not for the regtests divert
current-output-port generally to current-error-port and be done without all
those fixups.  Of course, this will still need to cater for those parts of the
code that choose to go to stdout directly, bypassing Scheme.

Other than that general objection, looks like the way to go.

Expand All Messages | Collapse All Messages