unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Kangas <stefan@marxist.se>
To: Jean-Christophe Helary <jean.christophe.helary@traduction-libre.org>
Cc: 39187@debbugs.gnu.org
Subject: bug#39187: Fwd: "Package refresh is already in progress, please wait..."
Date: Thu, 23 Jan 2020 19:18:28 +0100	[thread overview]
Message-ID: <87iml2f34b.fsf@marxist.se> (raw)
In-Reply-To: <87muakus4m.fsf@marxist.se> (Stefan Kangas's message of "Sun, 19 Jan 2020 02:55:37 +0100")

>> > It looks like emacs does not properly handle package refreshes when
>> > the internet connection has been killed in the middle of an
>> > refresh.
>> > 
>> > I often get this message in that case and I can't call package
>> > until I actually kill emacs and restart it.
[...]
> I think the fix here is simply to make sure that these processes times
> out.

Unless I'm missing something, it turns out that this is not that easy
to do.  At least I couldn't figure out how to do it.  I found the
following probably relevant bug:

 bug#34607: url-retrieve-synchronously: TIMEOUT option value takes no effect

For now at least, I'm giving up on finding a solution where url does
the right thing and times out. 

-----

I see two ways forward:

1. Remove the warning and accept that we might launch several
   processes doing the same update one after the other.  I would like
   to avoid that on master, but it might be the least intrusive change
   for emacs-27.
  
2. When we run a refresh, save the current time.  If the user tries to
   run another refresh, check if more than X seconds has elapsed since
   the old one was started.  If it has been more than X seconds,
   prompt the user to kill the old refresh and re-launch it.  It won't
   be perfect, but it will be better than what we had before at least.

Does anyone else have any ideas or preferences here?  Otherwise, I'll
revert the change that exposed this bug on the emacs-27 branch, and
implement the second change on master.

Best regards,
Stefan Kangas





  parent reply	other threads:[~2020-01-23 18:18 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CADwFkmmNJYecn62nN8rxfFqMD9dU75z_BfGebf89M4XmnmZZWA@mail.gmail.com>
2020-01-19  1:28 ` bug#39187: Fwd: "Package refresh is already in progress, please wait..." Jean-Christophe Helary
2020-01-19  1:55   ` Stefan Kangas
2020-01-19  9:09     ` Michael Albinus
2020-01-19 15:20       ` Eli Zaretskii
2020-01-20 12:36         ` Michael Albinus
2020-01-20 13:29           ` Stefan Kangas
2020-01-20 13:52             ` Michael Albinus
2020-01-23 18:18     ` Stefan Kangas [this message]
2020-02-09 20:33       ` Stefan Kangas
2020-02-09 21:41         ` Jean-Christophe Helary
2020-02-10 15:30           ` Stefan Kangas
2020-02-11  2:17             ` Jean-Christophe Helary
2020-02-11  5:55             ` Jean-Christophe Helary
2020-02-11  6:25               ` Stefan Kangas
2020-02-11  6:38                 ` Jean-Christophe Helary
2020-03-22 11:40       ` Stefan Kangas
2020-04-08  1:22         ` Stefan Kangas
2020-04-08  1:50           ` Jean-Christophe Helary

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87iml2f34b.fsf@marxist.se \
    --to=stefan@marxist.se \
    --cc=39187@debbugs.gnu.org \
    --cc=jean.christophe.helary@traduction-libre.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).