Daniel Shahaf wrote:
> I'm a bit hesitant about disabling notifications _entirely_ in cat-cmd.c
> and diff-cmd.c. Disabling all notifications (as opposed to only
> hydration-related notifications which we focus on right now) seems like
> it could easily have unintended consequences. Do we do that
Karl Fogel wrote on Wed, Mar 30, 2022 at 17:58:55 -0500:
> On 30 Mar 2022, Julian Foad wrote:
> > Karl Fogel wrote:
> > > I think printing these messages to stderr makes the most sense.
> > > There are plenty of programs out there that parse the stdout of
> > > 'svn'; we don't want to interfere wit
On 30 Mar 2022, Julian Foad wrote:
Karl Fogel wrote:
I think printing these messages to stderr makes the most sense.
There are plenty of programs out there that parse the stdout of
'svn'; we don't want to interfere with them.
As you point out, it's especially important for 'svn diff' and
'sv
On Wed, Mar 30, 2022 at 9:02 AM Julian Foad wrote:
>
> Karl Fogel wrote:
> > I think printing these messages to stderr makes the most sense.
> > There are plenty of programs out there that parse the stdout of
> > 'svn'; we don't want to interfere with them.
> >
> > As you point out, it's especiall
Karl Fogel wrote:
> I think printing these messages to stderr makes the most sense.
> There are plenty of programs out there that parse the stdout of
> 'svn'; we don't want to interfere with them.
>
> As you point out, it's especially important for 'svn diff' and
> 'svn cat' that stdout remain
On 24 Mar 2022, Julian Foad wrote:
For 'svn diff' especially, if we don't print the notifications,
then we
miss out on informing the user during one of the times when it
could be
particularly valuable to them. (They are waiting for diff output,
which
previously in svn used to come quickly.)
I
Finished and committed: https://svn.apache.org/r1899173 .
This looks and feels much better to me now.
Example:
$ svn revert -R subversion/
Fetching text bases ..done
Reverted 'subversion/svn/cat-cmd.c'
Reverted 'subversion/svn/diff-cmd.c'
Reverted [...]
- Julian
Updated and cleaned-up patch 'hydrating-notifications-2.patch' attached,
for interest. Still TODO: update test expectations.
hydrating-notifications-2.patch
Description: Binary data
Daniel Sahlberg wrote:
> How does svn cat handle any other informative or warning messages?
I don't think 'svn cat' prints any other informative or warning messages.
It occurs to me now that anything on stderr is generally assumed to
indicate an error, in tests in the test suite. We need to updat
Den tors 24 mars 2022 kl 11:34 skrev Julian Foad :
> For 'svn diff' especially, if we don't print the notifications, then we
> miss out on informing the user during one of the times when it could be
> particularly valuable to them. (They are waiting for diff output, which
> previously in svn used
For 'svn diff' especially, if we don't print the notifications, then we
miss out on informing the user during one of the times when it could be
particularly valuable to them. (They are waiting for diff output, which
previously in svn used to come quickly.)
It would be ugly to print these notificat
Den tors 24 mars 2022 kl 09:53 skrev Julian Foad :
> Daniel Sahlberg wrote:
> >> I'll put it on my todo list, but I can't promise when I find time to
> >> to that.
>
> I only meant to ask you to clarify what you meant: whether you are
> reporting for TSVN 1.14 or a TSVN 1.15-dev build (based on th
I made a mistake in:
> 'svn cat' already suppresses these notifications.
CORRECTION:
'svn cat' currently prints these notifications.
'svn diff' currently prints these notifications.
--> TODO: stop printing them here.
It seems the only change needed here is to make both 'svn diff' AND 'svn
cat'
Daniel Sahlberg wrote:
>> I'll put it on my todo list, but I can't promise when I find time to
>> to that.
I only meant to ask you to clarify what you meant: whether you are
reporting for TSVN 1.14 or a TSVN 1.15-dev build (based on the
pristines-on-demand branch or otherwise; I don't know if anyo
Den tors 24 mars 2022 kl 09:04 skrev Julian Foad :
> Daniel, can you confirm if you mean you are using a 1.14-based TSVN, on
> a 1.15 working copy, and it is showing bytes-transferred feedback during
> the hydrating? I had assumed it would need to be rebuilt and modified to
> do that. If that part
Marl Phippard wrote:
> Yes, and I agree --quiet should suppress. [...TSVN] hooked into our
> notifications to be providing an update on bytes transferred. [...] I
> just assumed they would work ... maybe they still do?
Daniel Sahlberg wrote:
>> Yes, I like this very much. The more feedback [...] t
Den ons 23 mars 2022 kl 23:58 skrev Mark Phippard :
> On Wed, Mar 23, 2022 at 5:51 PM Julian Foad wrote:
>
> > I thought maybe we would like to show this detail by default, and
> > suppress it when '--quiet' is passed. (Not implemented in this demo
> patch.)
> >
> > I was also mildly surprised to
On Wed, Mar 23, 2022 at 5:51 PM Julian Foad wrote:
> I thought maybe we would like to show this detail by default, and
> suppress it when '--quiet' is passed. (Not implemented in this demo patch.)
>
> I was also mildly surprised to see that the fetches are not necessarily
> all grouped together a
The notifications are printed for any command. Let's please pretend I
didn't use 'update' as an example. I don't want us to mix up this
discussion with the discussion about how 'update' shouldn't hydrate.
Let's pretend I used 'diff' or 'revert' as the example, like this:
[[[
$ svn revert -R 1991/*
19 matches
Mail list logo
