[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Various updates to reduce make doc output (issue 5727055)
From: |
Julien Rioux |
Subject: |
Re: Various updates to reduce make doc output (issue 5727055) |
Date: |
Wed, 14 Mar 2012 11:37:37 +0100 |
On Wed, Mar 14, 2012 at 9:56 AM, Phil Holmes <address@hidden> wrote:
> ----- Original Message ----- From: "Julien Rioux" <address@hidden>
> To: "Phil Holmes" <address@hidden>
>
>> >
>> > It seems just not worth it. We _never_ want to check warnings as part of
>> > make doc. That's what regression tests are for.
>> >
>> > --
>> > Phil Holmes
>>
>> I disagree, make -s doc is useful to identify warning messages that
>> need fixing, and the log files are also useful for this. I think that
>> the progress messages are what you should focus on silencing. The
>> warning messages should be either fixed at the source or left in place
>> so that someone eventually decides to fix them at the source. In this
>> particular case, it might be that nobody will ever work to improve
>> midi2ly, but that's not for us to say.
>>
>> Regards,
>> Julien
>
>
> Sorry - I expressed myself badly. I meant that we shouldn't use make doc to
> check that warnings that we expect to appear do, in fact, continue to
> appear. We should use make test to check that the "correct" warnings
> continue to be output. I agree 100% that make doc _should_ make it easy to
> find new warnings we were unaware of.
>
> The problem with this one is that Lilypond (like Sibelius...) only provides
> 4 voices to allow notes to avoid colliding on a stave. I haven't delved
> deeply into midi2ly, but my assumption is that it maps midi channels on a
> single stave to different voices. The "offending" midi file has more that 4
> channels on a stave and therefore the mapping to 4 voices is never going to
> work properly - and so midi2ly warns the user. But we don't want to
> continue to see those warnings on the screen, hence the suppression with -q.
> I can't see why an interactive user would use -q, so it won't give them a
> problem. I also don't believe that having another logfile solely to contain
> the message "warning: found more than 5 voices on a staff, expect bad
> output" makes much sense. Hence the way I went.
>
> --
> Phil Holmes
>
>
I'm fine with either
1) Leave the warning there (it's only one line of `make doc' output), or
2) Document in `midi2ly --help' that --quiet also silences warning messages.
Cheers,
Julien
- Re: Various updates to reduce make doc output (issue 5727055), (continued)
- Re: Various updates to reduce make doc output (issue 5727055), graham, 2012/03/04
- Re: Various updates to reduce make doc output (issue 5727055), PhilEHolmes, 2012/03/11
- Re: Various updates to reduce make doc output (issue 5727055), PhilEHolmes, 2012/03/12
- Re: Various updates to reduce make doc output (issue 5727055), julien . rioux, 2012/03/12
- Re: Various updates to reduce make doc output (issue 5727055), Phil Holmes, 2012/03/12
- Re: Various updates to reduce make doc output (issue 5727055), Graham Percival, 2012/03/12
- Re: Various updates to reduce make doc output (issue 5727055), Phil Holmes, 2012/03/12
- Re: Various updates to reduce make doc output (issue 5727055), Julien Rioux, 2012/03/13
- Re: Various updates to reduce make doc output (issue 5727055), Phil Holmes, 2012/03/14
- Re: Various updates to reduce make doc output (issue 5727055),
Julien Rioux <=
- Re: Various updates to reduce make doc output (issue 5727055), Trevor Daniels, 2012/03/14
- Message not available
- Message not available
- Re: Various updates to reduce make doc output (issue 5727055), Trevor Daniels, 2012/03/14
- Re: Various updates to reduce make doc output (issue 5727055), Phil Holmes, 2012/03/14
- Re: Various updates to reduce make doc output (issue 5727055), Carl Sorensen, 2012/03/14
- Re: Various updates to reduce make doc output (issue 5727055), Trevor Daniels, 2012/03/14
- Re: Various updates to reduce make doc output (issue 5727055), Phil Holmes, 2012/03/15
- Re: Various updates to reduce make doc output (issue 5727055), Trevor Daniels, 2012/03/15
- Re: Various updates to reduce make doc output (issue 5727055), Graham Percival, 2012/03/15
- Re: Various updates to reduce make doc output (issue 5727055), Graham Percival, 2012/03/15
- Re: Various updates to reduce make doc output (issue 5727055), Phil Holmes, 2012/03/15