Errors not handled well when doing cat-archive-log

The last major problem that I run into when using DVC is replicable as
follows.

$ tla register-archive -f http://arch.sv.gnu.org/archives/erc/erc
$ tla get erc <at> sv.gnu.org/erc--main--0 erc
Have DVC loaded, and bring up the `erc' directory in dired.
Hit `C-x V L'.

While some log messages are successfully retrieved, there are about 25
instances of the following error.  This locks up Emacs for about 5
minutes: C-g only sometimes stops the madness.

  ssh_exchange_identification: Connection closed by remote host
  Error reading from server

One possible way of dealing with this is to both limit the number of
concurrent connections (say, to 3) and if any connection triggers an
error, prevent the rest from being run.

--

-- 
Michael Olson -- FSF Associate Member #652 -- http://www.mwolson.org/
Interests: Lisp, text markup, protocols -- Jabber: mwolson_at_hcoop.net
  /` |\ | | | Projects: Emacs, Muse, ERC, EMMS, Planner, ErBot, DVC
 |_] | \| |_| Reclaim your digital rights by eliminating DRM.
      See http://www.defectivebydesign.org/what_is_drm for details.

_______________________________________________
Dvc-dev mailing list
Dvc-dev <at> gna.org

Re: Errors not handled well when doing cat-archive-log

Michael Olson <mwolson <at> gnu.org> writes:

> $ tla register-archive -f http://arch.sv.gnu.org/archives/erc/erc
> $ tla get erc <at> sv.gnu.org/erc--main--0 erc
> Have DVC loaded, and bring up the `erc' directory in dired.
> Hit `C-x V L'.
>
> While some log messages are successfully retrieved, there are about 25
> instances of the following error.  This locks up Emacs for about 5
> minutes: C-g only sometimes stops the madness.
>
>   ssh_exchange_identification: Connection closed by remote host
>   Error reading from server

I follewed your instructions but the problem didn't happen.  I use tla
version 1.3.5.

It seems strange to me that an ssh connection is attempted.  In fact,
AFAIK, `C-x V L' should be more or less equivalent to

$ tla logs --full --reverse

which should not trigger any connection to the remote repository.

In addition to that, it is also strange that there is an
`ssh_exchange_identification' error where ssh should not be involved,
because the repository is accessed through http.

Maybe dvc is configured to use baz as backend instead of tla?

Re: Errors not handled well when doing cat-archive-log

Andrea Russo <rastandy <at> inventati.org> writes:

> It seems strange to me that an ssh connection is attempted.  In fact,
> AFAIK, `C-x V L' should be more or less equivalent to
>
> $ tla logs --full --reverse

Not exactly because DVC needs some informations from the log file that
are not in $(tla logs --full --reverse), so it does a cat-archive-log
for each of them (baz 1.5 was adding a --complete-log option to get
all the info in one command). Additionnaly, it keeps a cache of the
logs, so if you have already some of them, cat-archive-log is run only
for the missing ones.

And different calls to tla/baz => different connections ...

--

-- 
Matthieu

Re: Errors not handled well when doing cat-archive-log

Matthieu Moy <Matthieu.Moy <at> imag.fr> writes:

> Andrea Russo <rastandy <at> inventati.org> writes:
>
>> It seems strange to me that an ssh connection is attempted.  In fact,
>> AFAIK, `C-x V L' should be more or less equivalent to
>>
>> $ tla logs --full --reverse
>
> Not exactly because DVC needs some informations from the log file that
> are not in $(tla logs --full --reverse), so it does a cat-archive-log
> for each of them (baz 1.5 was adding a --complete-log option to get
> all the info in one command). Additionnaly, it keeps a cache of the
> logs, so if you have already some of them, cat-archive-log is run only
> for the missing ones.

That is already good enough for my use.
The first time I show the log and get a list of errors.
Then I simply show the log again and then I get usually no more errors.

But Michael is correct that we could have a better implementation.

Stefan.

Re: Errors not handled well when doing cat-archive-log

Andrea Russo <rastandy <at> inventati.org> writes:

> In addition to that, it is also strange that there is an
> `ssh_exchange_identification' error where ssh should not be involved,
> because the repository is accessed through http.

Oh ... I was actually using the developer-only repo instead of the
public access repo for ERC, and that uses SSH.  I guess in order to
replicate the problem, you'd have to have a repository that uses sftp.

--

-- 
Michael Olson -- FSF Associate Member #652 -- http://www.mwolson.org/
Interests: Lisp, text markup, protocols -- Jabber: mwolson_at_hcoop.net
  /` |\ | | | Projects: Emacs, Muse, ERC, EMMS, Planner, ErBot, DVC
 |_] | \| |_| Reclaim your digital rights by eliminating DRM.
      See http://www.defectivebydesign.org/what_is_drm for details.

_______________________________________________
Dvc-dev mailing list
Dvc-dev <at> gna.org
https://mail.gna.org/listinfo/dvc-dev

[commit] rev 136: Applied minor fix for dvc-build.el from Sascha Wilde <wilde <at> sha-bang.de>

Committed revision 136 to http://bzr.xsteve.at/dvc/

Applied minor fix for dvc-build.el from Sascha Wilde <wilde <at> sha-bang.de>

  Set contribdir based on srcdir