headers
Matt McClure | 6 Aug 2012 04:05
Picon
Gravatar

bug#12145: 24.1.50; enable-remote-dir-locals breaks find-name-dired in TRAMP buffers

When `enable-remote-dir-locals` is `t`, `find-name-dired` in TRAMP buffers gives

    ...
    error in process filter: Wrong type argument: markerp, nil
    error in process filter: accept-process-output: Wrong type
argument: markerp, nil
    error in process filter: Wrong type argument: markerp, nil
    nil: exited abnormally with code 1.
    find-dired: Wrong type argument: processp, nil

To reproduce on Mac OS X:

    open -n -a Emacs --args -Q --no-site-file --eval '(progn (setq
enable-remote-dir-locals t) (find-name-dired "/user <at> host:/etc"
"*.conf"))'

I'm not able to reproduce the defect with the similar invocation:

    open -n -a Emacs --args -Q --no-site-file --eval '(progn
(find-name-dired "/user <at> host:/etc" "*.conf"))'

In GNU Emacs 24.1.50.1 (x86_64-apple-darwin, NS apple-appkit-1038.36)
 of 2012-07-14 on bob.porkrind.org
Bzr revision: 109087 cyd <at> gnu.org-20120714053223-jxkxt958pqg8tisb
Windowing system distributor `Apple', version 10.3.1138
Configured using:
 `configure '--host=x86_64-apple-darwin' '--build=i686-apple-darwin'
 '--with-ns' 'build_alias=i686-apple-darwin'
 'host_alias=x86_64-apple-darwin' 'CC=gcc -mmacosx-version-min=10.5''
(Continue reading)

Permalink | Reply |
headers
Michael Albinus | 6 Aug 2012 14:18
Picon
Picon
Gravatar

bug#12145: 24.1.50; enable-remote-dir-locals breaks find-name-dired in TRAMP buffers

Matt McClure <matthewlmcclure <at> gmail.com> writes:

> When `enable-remote-dir-locals` is `t`, `find-name-dired` in TRAMP buffers gives
>
>     error in process filter: Wrong type argument: markerp, nil

I can reproduce the bug locally. It happens in `comint-output-filter',
which is enabled by `tramp-handle-shell-command'.

> I'm not able to reproduce the defect with the similar invocation:
>
>     open -n -a Emacs --args -Q --no-site-file --eval '(progn
> (find-name-dired "/user <at> host:/etc" "*.conf"))'

Same here. It is a side-effect of an additional call to `file-exists-p'
for the directory local file; it is not related to directory local
settings as such.

Two Tramp processes do interact badly, when reading processes' output in
parallel.

I will continue to debug, it isn't such obvious what happens.

Best regards, Michael.
Permalink | Reply |
headers
Michael Albinus | 20 Aug 2012 16:07
Picon
Picon
Gravatar

bug#12145: 24.1.50; enable-remote-dir-locals breaks find-name-dired in TRAMP buffers

Michael Albinus <michael.albinus <at> gmx.de> writes:

> Matt McClure <matthewlmcclure <at> gmail.com> writes:
>
>> When `enable-remote-dir-locals` is `t`, `find-name-dired` in TRAMP
>> buffers gives
>>
>>     error in process filter: Wrong type argument: markerp, nil
>
> I can reproduce the bug locally. It happens in `comint-output-filter',
> which is enabled by `tramp-handle-shell-command'.
>
> Two Tramp processes do interact badly, when reading processes' output in
> parallel.
>
> I will continue to debug, it isn't such obvious what happens.

I've committed a fix for that. You might test it.

Best regards, Michael.
Permalink | Reply |
headers
Matt McClure | 28 Aug 2012 16:12
Picon
Gravatar

bug#12145: 24.1.50; enable-remote-dir-locals breaks find-name-dired in TRAMP buffers

Hi Michael,

On Mon, Aug 20, 2012 at 10:07 AM, Michael Albinus
<michael.albinus <at> gmx.de> wrote:
> I've committed a fix for that. You might test it.

The fix works for me. Thanks!

--

-- 
Matt McClure
http://www.matthewlmcclure.com
http://www.mapmyfitness.com/profile/matthewlmcclure
Permalink | Reply |
headers
Michael Albinus | 28 Aug 2012 16:42
Picon
Picon
Gravatar

bug#12145: 24.1.50; enable-remote-dir-locals breaks find-name-dired in TRAMP buffers

Version: 24.3

Matt McClure <matthewlmcclure <at> gmail.com> writes:

> Hi Michael,

Hi Matt,

> On Mon, Aug 20, 2012 at 10:07 AM, Michael Albinus
> <michael.albinus <at> gmx.de> wrote:
>> I've committed a fix for that. You might test it.
>
> The fix works for me. Thanks!

Thanks for testing. I'm closing the bug.

Best regards, Michael.
Permalink | Reply |

Gmane