unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: master 453ffe5: Fix minor Tramp oddities
       [not found] ` <20200520090953.514B820AEB@vcs0.savannah.gnu.org>
@ 2020-05-20 12:08   ` Stefan Monnier
  2020-05-20 16:02     ` Michael Albinus
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Monnier @ 2020-05-20 12:08 UTC (permalink / raw)
  To: Michael Albinus; +Cc: emacs-devel

> +	     (archive (tramp-archive-file-name-archive filename))
> +	     ;; Sometimes, it fails with "Variable binding depth exceeds
> +	     ;; max-specpdl-size".
> +	     (max-specpdl-size (* 2 max-specpdl-size)))

This is an ugly hack to work around a problem.  Emacs doesn't shy away
from ugly hacks to work around problem, but we should make every effort
to document the problem so that if someone has the time&energy to try
and find a better fix they know where to look.

IOW, could you make a bug-report of the problem (ideally with
a reliably reproducible recipe and/or a backtrace) and link to it in the
above comment?


        Stefan




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

* Re: master 453ffe5: Fix minor Tramp oddities
  2020-05-20 12:08   ` master 453ffe5: Fix minor Tramp oddities Stefan Monnier
@ 2020-05-20 16:02     ` Michael Albinus
  2020-05-20 18:02       ` andres.ramirez
  2020-05-20 23:17       ` Stefan Monnier
  0 siblings, 2 replies; 4+ messages in thread
From: Michael Albinus @ 2020-05-20 16:02 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

Stefan Monnier <monnier@iro.umontreal.ca> writes:

Hi Stefan,

>> +	     (archive (tramp-archive-file-name-archive filename))
>> +	     ;; Sometimes, it fails with "Variable binding depth exceeds
>> +	     ;; max-specpdl-size".
>> +	     (max-specpdl-size (* 2 max-specpdl-size)))
>
> This is an ugly hack to work around a problem.  Emacs doesn't shy away
> from ugly hacks to work around problem, but we should make every effort
> to document the problem so that if someone has the time&energy to try
> and find a better fix they know where to look.

The problem happens sporadically only. And IIRC, it happened only with
Emacs <=26. max-specpdl-size was increased in Emacs 27.
>
> IOW, could you make a bug-report of the problem (ideally with
> a reliably reproducible recipe and/or a backtrace) and link to it in the
> above comment?

A bug report would be useless. I've added this information to the comment.

>         Stefan

Best regards, Michael.



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

* Re: master 453ffe5: Fix minor Tramp oddities
  2020-05-20 16:02     ` Michael Albinus
@ 2020-05-20 18:02       ` andres.ramirez
  2020-05-20 23:17       ` Stefan Monnier
  1 sibling, 0 replies; 4+ messages in thread
From: andres.ramirez @ 2020-05-20 18:02 UTC (permalink / raw)
  To: Michael Albinus; +Cc: Stefan Monnier, emacs-devel

Hi Michael.
>>>>> "Michael" == Michael Albinus <michael.albinus@gmx.de> writes:


[...]


    Michael> The problem happens sporadically only. And IIRC, it
    Michael> happened only with Emacs <=26. max-specpdl-size was
    Michael> increased in Emacs 27.

Just a little bit of more info. This happens to me just on
emacs23. But. Perhaps are other opinions about this.


Best Regards



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

* Re: master 453ffe5: Fix minor Tramp oddities
  2020-05-20 16:02     ` Michael Albinus
  2020-05-20 18:02       ` andres.ramirez
@ 2020-05-20 23:17       ` Stefan Monnier
  1 sibling, 0 replies; 4+ messages in thread
From: Stefan Monnier @ 2020-05-20 23:17 UTC (permalink / raw)
  To: Michael Albinus; +Cc: emacs-devel

> The problem happens sporadically only. And IIRC, it happened only with
> Emacs <=26. max-specpdl-size was increased in Emacs 27.
[...]
> A bug report would be useless. I've added this information to the comment.

Good, thanks,


        Stefan




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

end of thread, other threads:[~2020-05-20 23:17 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20200520090951.21321.65869@vcs0.savannah.gnu.org>
     [not found] ` <20200520090953.514B820AEB@vcs0.savannah.gnu.org>
2020-05-20 12:08   ` master 453ffe5: Fix minor Tramp oddities Stefan Monnier
2020-05-20 16:02     ` Michael Albinus
2020-05-20 18:02       ` andres.ramirez
2020-05-20 23:17       ` Stefan Monnier

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