From: Michael Albinus <michael.albinus@gmx.de>
To: Stefan Kangas <stefan@marxist.se>
Cc: emacs-devel@gnu.org
Subject: Re: master 262d0c6: Mark some tests as expensive
Date: Fri, 18 Sep 2020 12:31:44 +0200 [thread overview]
Message-ID: <87363f9y0f.fsf@gmx.de> (raw)
In-Reply-To: <CADwFkmmDcYa0yDPZQdGhst93j5gmqG8D0nU+mg2AwkU-cvsnEA@mail.gmail.com> (Stefan Kangas's message of "Fri, 18 Sep 2020 03:22:54 -0700")
Stefan Kangas <stefan@marxist.se> writes:
Hi Stefan,
> It does seem unfortunate that no tests run for autorevert-mode in the
> default case. I have a patch in the works that will reduce the running
> time for these tests, I will send it here when it's done. (Basically
> the patch would temporarily lower auto-revert-interval, remove sleeps
> and manually set mtime to lower values than the current time.)
Would be helpful. However, there have been discussions (and changes) of
the timeouts in the past, so I'm not sure we can reduce them w/o causing
damage. Please check the git history of autorevert-tests.el for such
changes, hopefully there are bug numbers in the git log you could consult
for the respective discussions. Otherwise, a search on the emacs-devel
and emacs-bugs MLs for the given timeframe of such a change might be helpful.
>> There have been discussions about how long tests shall run. But there
>> hasn't been ever a conclusion to limit them to 1 second. Please discuss
>> this first, before making such changes.
>
> Sure, I just didn't expect this to be controversial. Sorry if this
> change caused any problems.
IIRC, this discussion happened before you've started with work on
Emacs. I'm pretty sure there's something in the emacs-devel ML, but this
was years ago.
> Best regards,
> Stefan Kangas
Best regards, Michael.
next prev parent reply other threads:[~2020-09-18 10:31 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20200910182904.20559.25935@vcs0.savannah.gnu.org>
[not found] ` <20200910182905.F0E4520A2E@vcs0.savannah.gnu.org>
2020-09-11 9:25 ` master 262d0c6: Mark some tests as expensive Michael Albinus
2020-09-11 18:06 ` Stefan Kangas
2020-09-12 10:25 ` Daniel Martín
2020-09-12 10:38 ` Eli Zaretskii
2020-09-12 11:15 ` Lars Ingebrigtsen
2020-09-12 11:24 ` Eli Zaretskii
2020-09-12 12:11 ` Lars Ingebrigtsen
2020-09-12 12:29 ` Eli Zaretskii
2020-09-13 12:30 ` Lars Ingebrigtsen
2020-09-13 15:21 ` Stefan Monnier
2020-09-13 15:30 ` Lars Ingebrigtsen
2020-09-13 17:22 ` Michael Albinus
2020-09-12 16:47 ` Michael Albinus
2020-09-13 12:33 ` Lars Ingebrigtsen
2020-09-13 14:37 ` Eli Zaretskii
2020-09-12 11:27 ` Michael Albinus
2020-09-12 12:15 ` Lars Ingebrigtsen
2020-09-12 12:30 ` Michael Albinus
2020-09-12 12:36 ` Lars Ingebrigtsen
2020-09-12 13:04 ` Dmitry Gutov
2020-09-12 14:23 ` Daniel Martín
2020-09-12 14:49 ` Michael Albinus
2020-09-12 16:47 ` Dmitry Gutov
2020-09-12 14:53 ` Lars Ingebrigtsen
2020-09-12 14:00 ` Daniel Martín
2020-09-12 14:14 ` Eli Zaretskii
2020-09-12 15:16 ` Daniel Martín
2020-09-12 14:43 ` Michael Albinus
2020-09-12 15:02 ` Daniel Martín
2020-09-12 10:52 ` Michael Albinus
2020-09-18 10:22 ` Stefan Kangas
2020-09-18 10:31 ` Michael Albinus [this message]
2020-10-18 18:15 ` Stefan Kangas
2020-10-19 12:40 ` Michael Albinus
2020-10-19 15:34 ` Stefan Kangas
2020-10-19 16:42 ` Michael Albinus
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=87363f9y0f.fsf@gmx.de \
--to=michael.albinus@gmx.de \
--cc=emacs-devel@gnu.org \
--cc=stefan@marxist.se \
/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).