unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: master d89d5e0: Fix handling of folder "+/" in MH-E
       [not found] ` <20211116060540.0F2A020CB3@vcs0.savannah.gnu.org>
@ 2021-11-16  8:06   ` Lars Ingebrigtsen
  2021-11-16 15:49     ` Mike Kupfer
  2021-11-18  5:07     ` Mike Kupfer
  0 siblings, 2 replies; 4+ messages in thread
From: Lars Ingebrigtsen @ 2021-11-16  8:06 UTC (permalink / raw)
  To: emacs-devel; +Cc: Mike Kupfer

stepheng+savannah@gildea.com (Stephen Gildea) writes:

>     Fix handling of folder "+/" in MH-E

This leads to:

2 unexpected results:
   FAILED  mh-folder-completion-function-08-plus-slash
   FAILED  mh-folder-completion-function-09-plus-slash-tmp

on Debian/bullseye (with nativecomp).  Here's more of the error data:

Test mh-folder-completion-function-08-plus-slash condition:
    (ert-test-failed
     ((should
       (equal "+/"
	      (mh-folder-completion-function "+/" nil nil)))
      :form
      (equal "+/" nil)
      :value nil :explanation
      (different-types "+/" nil)))
   FAILED   7/16  mh-folder-completion-function-08-plus-slash (0.000249 sec)
call-process mock unexpected arglist (folders -noheader -norecurse -nototal +/)
Test mh-folder-completion-function-09-plus-slash-tmp backtrace:
  signal(ert-test-failed (((should (equal "+/tmp/" (mh-folder-completi
  ert-fail(((should (equal "+/tmp/" (mh-folder-completion-function "+/
  (if (unwind-protect (setq value-387 (apply fn-385 args-386)) (setq f
  (let (form-description-389) (if (unwind-protect (setq value-387 (app
  (let ((value-387 'ert-form-evaluation-aborted-388)) (let (form-descr
  (let* ((fn-385 #'equal) (args-386 (condition-case err (let ((signal-
  (if (and (not (consp "+/tmp/")) (consp (mh-folder-completion-functio
  (progn (setq temp-home-dir (mh-test-utils-setup)) (if (and (not (con


-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: master d89d5e0: Fix handling of folder "+/" in MH-E
  2021-11-16  8:06   ` master d89d5e0: Fix handling of folder "+/" in MH-E Lars Ingebrigtsen
@ 2021-11-16 15:49     ` Mike Kupfer
  2021-11-17  7:13       ` Stefan Kangas
  2021-11-18  5:07     ` Mike Kupfer
  1 sibling, 1 reply; 4+ messages in thread
From: Mike Kupfer @ 2021-11-16 15:49 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: emacs-devel

Lars Ingebrigtsen wrote:

> stepheng+savannah@gildea.com (Stephen Gildea) writes:
> 
> >     Fix handling of folder "+/" in MH-E
> 
> This leads to:
> 
> 2 unexpected results:
>    FAILED  mh-folder-completion-function-08-plus-slash
>    FAILED  mh-folder-completion-function-09-plus-slash-tmp
> 
> on Debian/bullseye (with nativecomp).

Thanks.  I'll take a look, though it might take a few days before I get
anywhere.  I wonder if this is another conflict between nativecomp and
the mock environment that the MH-E tests set up.

mike



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: master d89d5e0: Fix handling of folder "+/" in MH-E
  2021-11-16 15:49     ` Mike Kupfer
@ 2021-11-17  7:13       ` Stefan Kangas
  0 siblings, 0 replies; 4+ messages in thread
From: Stefan Kangas @ 2021-11-17  7:13 UTC (permalink / raw)
  To: Mike Kupfer, Lars Ingebrigtsen; +Cc: emacs-devel

Mike Kupfer <mkupfer@alum.berkeley.edu> writes:

>> 2 unexpected results:
>>    FAILED  mh-folder-completion-function-08-plus-slash
>>    FAILED  mh-folder-completion-function-09-plus-slash-tmp
>>
>> on Debian/bullseye (with nativecomp).
>
> Thanks.  I'll take a look, though it might take a few days before I get
> anywhere.  I wonder if this is another conflict between nativecomp and
> the mock environment that the MH-E tests set up.

I've marked these tests as unstable for now (commit fc8c976298), please
remove that tag when you fix them.  Thanks in advance.



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: master d89d5e0: Fix handling of folder "+/" in MH-E
  2021-11-16  8:06   ` master d89d5e0: Fix handling of folder "+/" in MH-E Lars Ingebrigtsen
  2021-11-16 15:49     ` Mike Kupfer
@ 2021-11-18  5:07     ` Mike Kupfer
  1 sibling, 0 replies; 4+ messages in thread
From: Mike Kupfer @ 2021-11-18  5:07 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: emacs-devel

Lars Ingebrigtsen wrote:

> stepheng+savannah@gildea.com (Stephen Gildea) writes:
> 
> >     Fix handling of folder "+/" in MH-E
> 
> This leads to:
> 
> 2 unexpected results:
>    FAILED  mh-folder-completion-function-08-plus-slash
>    FAILED  mh-folder-completion-function-09-plus-slash-tmp

This should now be fixed by b48cbaf5c7.

mike



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-11-18  5:07 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20211116060538.5412.31404@vcs0.savannah.gnu.org>
     [not found] ` <20211116060540.0F2A020CB3@vcs0.savannah.gnu.org>
2021-11-16  8:06   ` master d89d5e0: Fix handling of folder "+/" in MH-E Lars Ingebrigtsen
2021-11-16 15:49     ` Mike Kupfer
2021-11-17  7:13       ` Stefan Kangas
2021-11-18  5:07     ` Mike Kupfer

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).