[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote pro
From: |
Philipp Stephani |
Subject: |
bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process |
Date: |
Wed, 6 May 2020 12:50:18 +0200 |
Am Mi., 6. Mai 2020 um 12:38 Uhr schrieb Philipp Stephani
<p.stephani2@gmail.com>:
>
> Am Mi., 6. Mai 2020 um 10:38 Uhr schrieb Michael Albinus
> <michael.albinus@gmx.de>:
> >
> > Michael Albinus <michael.albinus@gmx.de> writes:
> >
> > Hi Philipp,
> >
> > >> It looks like `tramp-send-command-and-check' should not just search
> > >> for the exit code, but also return it.
> > >
> > > No, most use cases a simple nil or t is appropriate. Otherwise, you would
> > > need to check always (zerop (tramp-send-command-and-check ...)). But in
> > > tramp-sh-handle-process-file, it shall return the exit code (indicated
> > > by an optional argument).
> > >
> > > Same for tramp-adb-send-command-and-check, called in
> > > tramp-adb-handle-process-file.
> >
> > I've fixed this in master, could you pls check?
>
> Looks like it's working fine, thanks for the quick fix!
As a second step, consider also translating signals: if the exit code
is > 128, subtract 128 and return an appropriate string, like
call-process does.
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Philipp Stephani, 2020/05/05
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Philipp Stephani, 2020/05/05
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Michael Albinus, 2020/05/05
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Michael Albinus, 2020/05/06
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Philipp Stephani, 2020/05/06
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process,
Philipp Stephani <=
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Michael Albinus, 2020/05/06
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Michael Albinus, 2020/05/06
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Philipp Stephani, 2020/05/06
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Michael Albinus, 2020/05/06
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Philipp Stephani, 2020/05/06
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Philipp Stephani, 2020/05/06
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Michael Albinus, 2020/05/06
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Michael Albinus, 2020/05/07
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Philipp Stephani, 2020/05/09
- bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process, Noam Postavsky, 2020/05/13