darcs

Issue 1029 truncated progress reports can be confusing

Title truncated progress reports can be confusing
Priority bug Status given-up
Milestone Resolved in
Superseder Nosy List darcs-devel, dmitry.kurochkin, eivuokko, jaredj, kirby, kowey, simonpj, twb, wglozer
Assigned To kirby
Topics ProbablyEasy, UI, Windows

Created on 2008-08-22.12:08:07 by simonpj, last changed 2017-07-31.00:43:20 by gh.

Files
File name Uploaded Type Edit Remove
unnamed simonpj, 2008-08-22.12:08:06 text/html
Messages
msg5636 (view) Author: simonpj Date: 2008-08-22.12:08:06
More dards2 output on Windows

Notice the line in red is truncated.  Is there any need to truncate the message?  I did have a moment when I wondered what this "b" package was...

Simon

Identifying repository simonpj@darcs.haskell.org:/home/darcs/packages/base
scp: /home/darcs/packages/base/_darcs/format: No such file or directory
Identifying repository simonpj@darcs.haskell.org:/home/darcs/packages/base
scp: /home/darcs/packages/base/_darcs/prefs/sources: No such file or directory
Reading inventory of repository simonpj@darcs.haskell.org:/home/darcs/packages/b
scp: /home/darcs/packages/base/_darcs/prefs/motd: No such file or directory
Reading inventory of repository c:/simonpj/darcs/validate-HEAD/libraries/base
Pulling from "simonpj@darcs.haskell.org:/home/darcs/packages/base"...
Attachments
msg5639 (view) Author: kowey Date: 2008-08-22.12:18:11
Simon's red line is:
Reading inventory of repository simonpj@darcs.haskell.org:/home/darcs/packages/b

By the way Simon, I don't see the red lines, not on gmail anyway, and certainly
not on the bugtracker!

If anybody wants to look into this, you'd probably want to work out why we
truncate the messages.
msg7808 (view) Author: twb Date: 2009-05-13.04:41:42
On Fri, Aug 22, 2008 at 02:08:06AM +0000, Simon Peyton-Jones wrote:
> More darcs2 output on Windows.  Notice the line in red is truncated.
> Is there any need to truncate the message?  I did have a moment when
> I wondered what this "b" package was...
>
> Identifying repository simonpj@darcs.haskell.org:/home/darcs/packages/base
> scp: /home/darcs/packages/base/_darcs/format: No such file or directory
> Identifying repository simonpj@darcs.haskell.org:/home/darcs/packages/base
> scp: /home/darcs/packages/base/_darcs/prefs/sources: No such file or directory
> Reading inventory of repository simonpj@darcs.haskell.org:/home/darcs/packages/b

This is the only occurrence of "Reading inventory of repository", in
src/Darcs/Repository/DarcsRepo.lhs.  I don't see where the truncation
is happening.  toPath just returns the string part of the object, and
ioAbsoluteOrRemote is just converting the string to a remote path
(because isdir "simon...base" fails).

  read_repo :: RepoPatch p => [DarcsFlag] -> String -> IO (SealedPatchSet p)
  read_repo opts d = do
    realdir <- toPath `fmap` ioAbsoluteOrRemote d
    let k = "Reading inventory of repository "++d
    beginTedious k
    read_repo_private k opts realdir "inventory" `catch`
                          (\e -> do hPutStrLn stderr ("Invalid repository:  " ++ realdir)
                                    ioError e)

Can anyone reproduce this with a recent version of darcs (i.e. 2.2)?
The report doesn't contain instructions on how to generate this error,
and I don't know how.

Is it possible that the output was simply truncated by the terminal
emulator in which it was running, or something along those lines?
msg7813 (view) Author: kowey Date: 2009-05-15.15:45:45
On Wed, May 13, 2009 at 04:41:46 -0000, Trent Buck wrote:
> This is the only occurrence of "Reading inventory of repository", in
> src/Darcs/Repository/DarcsRepo.lhs.  I don't see where the truncation
> is happening.  toPath just returns the string part of the object, and
> ioAbsoluteOrRemote is just converting the string to a remote path
> 
>   read_repo :: RepoPatch p => [DarcsFlag] -> String -> IO (SealedPatchSet p)
>   read_repo opts d = do
>     realdir <- toPath `fmap` ioAbsoluteOrRemote d
>     let k = "Reading inventory of repository "++d
>     beginTedious k

Furthermore, this beginTedious function does does an hPutStrLn stderr
of the string via debugMessage.

What we could do maybe is make a small demonstrator that prints some
very long strings with the progress reporting mechanism or maybe
just debugMessage.
msg10574 (view) Author: kowey Date: 2010-03-30.10:30:17
We still need somebody to confirm that this truncation happens.  Perhaps
Salvatore?  It shouldn't take a very long time...
msg15868 (view) Author: lukeworth Date: 2012-07-14.21:35:32
I cannot reproduce this on my mac: very long pathnames are shown in full
by --debug-verbose. Also, read_repo no longer exists in the source code,
seemingly replaced by readOldRepo in Old.hs.
History
Date User Action Args
2008-08-22 12:08:07simonpjcreate
2008-08-22 12:18:17koweysetstatus: unread -> unknown
topic: + ProbablyEasy, UI
title: Darcs messsages -> truncated progress reports can be confusing
nosy: + jaredj
messages: + msg5639
priority: bug
2009-05-13 04:41:46twbsetnosy: + twb, dmitry.kurochkin, simon, thorkilnaur
messages: + msg7808
2009-05-15 15:45:48koweysetnosy: kowey, dagit, simonpj, simon, twb, thorkilnaur, jaredj, dmitry.kurochkin
messages: + msg7813
2009-08-10 23:43:38adminsetnosy: - dagit
2009-08-17 17:27:04koweysetstatus: unknown -> needs-reproduction
nosy: kowey, simonpj, simon, twb, thorkilnaur, jaredj, dmitry.kurochkin
2009-08-25 17:45:00adminsetnosy: + darcs-devel, - simon
2009-08-27 14:25:33adminsetnosy: kowey, darcs-devel, simonpj, twb, thorkilnaur, jaredj, dmitry.kurochkin
2010-03-30 10:30:21koweysettopic: + Windows
nosy: + wglozer, eivuokko, kirby, - thorkilnaur
messages: + msg10574
assignedto: kirby
2012-07-14 21:35:33lukeworthsetmessages: + msg15868
2017-07-31 00:43:20ghsetstatus: needs-reproduction -> given-up