On Tue, Oct 28, 2014 at 11:47 PM, Michael Albinus <michael.albi...@gmx.de> wrote:
> Ashi <ashi08...@gmail.com> writes: > > > Hi, Michael, > > Hi Zhongwei, > > > I've found the reason causes the tramp-test22-file-times failed. > > > > I evaluate "(set-file-times > > "/adb:10.164.2.12#5555:data/local/tramp_testdir/tramp-test12242_Hq" ' > > (0 1))", and it returns "t". > > What's the time stamp then of that file on your Android? Check it with > adb itself, not with tramp-adb. > adb returns: -rw-rw-rw- root root 3 -001-12-01 00:00 tramp-test12242_Hq > > > But "(file-attributes > > "/adb:10.164.2.12#5555:data/local/tramp_testdir/tramp-test12242_Hq")" > > returns: (nil 1 0 0 (0 0) (-948638 13211) (0 0) 3 "-rw-rw-rw-" t 1 (-1 > > . 6)) which causes "(should (equal (nth 5 (file-attributes tmp-name1)) > > '(0 1)))" failed. And I've checked that "adb shell" has no perl(1) or > > stat (1). I think there is some problem in "set-file-times". > > '(0 0) means by convention "don't know". There's already some code in > `tramp-test22-file-times' which ought to handle this. Maybe you can > steal some ideas from there? > Do you mean we should also check the 4th item? If yes, I've checked this test can be passed. But it failed at the tramp-test26-process-file test. Following is its output when failed: make -C test \ "EMACS=emacs" "EMACS_INFO=emacs" \ "DESTDIR=" all make[1]: Entering directory `/home/zhongwei/projects/project/tramp/test' emacs -Q -batch -L `pwd`/../lisp -l bytecomp -f batch-byte-compile tramp-tests.el Wrote /home/zhongwei/projects/project/tramp/test/tramp-tests.elc emacs -Q -batch -L `pwd`/../lisp -L `pwd` -l tramp-tests -f ert-run-tests-batch-and-exit Running 35 tests (2014-10-29 19:49:02+0800) Remote directory: `/adb:10.164.2.195#5555:data/local/zw/tramp_testdir' passed 1/35 tramp-test00-availability passed 2/35 tramp-test01-file-name-syntax passed 3/35 tramp-test02-file-name-dissect passed 4/35 tramp-test03-file-name-defaults passed 5/35 tramp-test04-substitute-in-file-name passed 6/35 tramp-test05-expand-file-name passed 7/35 tramp-test06-directory-file-name passed 8/35 tramp-test07-file-exists-p passed 9/35 tramp-test08-file-local-copy passed 10/35 tramp-test09-insert-file-contents passed 11/35 tramp-test10-write-region passed 12/35 tramp-test11-copy-file passed 13/35 tramp-test12-rename-file passed 14/35 tramp-test13-make-directory passed 15/35 tramp-test14-delete-directory passed 16/35 tramp-test15-copy-directory passed 17/35 tramp-test16-directory-files passed 18/35 tramp-test17-insert-directory passed 19/35 tramp-test18-file-attributes passed 20/35 tramp-test19-directory-files-and-attributes skipped 21/35 tramp-test20-file-modes skipped 22/35 tramp-test21-file-links passed 23/35 tramp-test22-file-times passed 24/35 tramp-test23-visited-file-modtime passed 25/35 tramp-test24-file-name-completion passed 26/35 tramp-test25-load Tramp: Opening adb shell connection... Opening adb shell connection... \ Tramp: Opening adb shell connection...done Tramp: Opening adb shell connection... Opening adb shell connection... \ Tramp: Opening adb shell connection...failed Tramp: Moving tmp file `/tmp/tramp.4132qeq' to `/adb:10.164.2.195#5555:/data/local/zw/tramp_testdir/tramp-test4132dUk'... Moving tmp file `/tmp/tramp.4132qeq' to `/adb:10.164.2.195#5555:/data/local/zw/tramp_testdir/tramp-test4132dUk' \ Tramp: Moving tmp file `/tmp/tramp.4132qeq' to `/adb:10.164.2.195#5555:/data/local/zw/tramp_testdir/tramp-test4132dUk'...done Tramp: Opening adb shell connection... Opening adb shell connection... \ Tramp: Opening adb shell connection...failed Test tramp-test26-process-file backtrace: tramp-adb-send-command(["adb" nil "10.164.2.195#5555" "/data" nil] " tramp-adb-send-command-and-check(["adb" nil "10.164.2.195#5555" "/da tramp-adb-get-ls-command(["adb" nil "10.164.2.195#5555" "/data" nil] tramp-adb-handle-file-attributes("/adb:10.164.2.195#5555:/data") apply(tramp-adb-handle-file-attributes "/adb:10.164.2.195#5555:/data tramp-adb-file-name-handler(file-attributes "/adb:10.164.2.195#5555: apply(tramp-adb-file-name-handler file-attributes "/adb:10.164.2.195 byte-code("\300\301\215\207" [suppress (apply foreign operation args byte-code(K\306 \211 <\203\237\n@\307=\203\237\306310\311!\203 tramp-file-name-handler(file-attributes "/adb:10.164.2.195#5555:/dat file-attributes("/adb:10.164.2.195#5555:/data") tramp-adb-handle-file-truename("/adb:10.164.2.195#5555:/data/local/z apply(tramp-adb-handle-file-truename "/adb:10.164.2.195#5555:/data/l tramp-adb-file-name-handler(file-truename "/adb:10.164.2.195#5555:/d apply(tramp-adb-file-name-handler file-truename "/adb:10.164.2.195#5 byte-code("\300\301\215\207" [suppress (apply foreign operation args byte-code(K\306 \211 <\203\237\n@\307=\203\237\306310\311!\203 tramp-file-name-handler(file-truename "/adb:10.164.2.195#5555:/data/ file-truename("/adb:10.164.2.195#5555:/data/local/zw/tramp_testdir/t tramp-adb-handle-file-directory-p("/adb:10.164.2.195#5555:/data/loca apply(tramp-adb-handle-file-directory-p "/adb:10.164.2.195#5555:/dat tramp-adb-file-name-handler(file-directory-p "/adb:10.164.2.195#5555 apply(tramp-adb-file-name-handler file-directory-p "/adb:10.164.2.19 byte-code("\300\301\215\207" [suppress (apply foreign operation args byte-code(K\306 \211 <\203\237\n@\307=\203\237\306310\311!\203 tramp-file-name-handler(file-directory-p "/adb:10.164.2.195#5555:/da delete-file("/adb:10.164.2.195#5555:/data/local/zw/tramp_testdir/tra byte-code("\300\301\302\217\207" [nil (delete-file tmp-name) ((error #[nil "\306\307 \310 \307\211311\312\217\204 \313 !\210)\n\210+\ #[0 "\306\307!r\211q\210\310\311\312\313\314\315 !\316\"\317\320%DC funcall(#[0 "\306\307!r\211q\210\310\311\312\313\314\315 !\316\"\31 ert--run-test-internal([cl-struct-ert--test-execution-info [cl-struc #[0 "r\304 q\210\305 )\306\307\310\311\312\313 !\314\"\315\316%DC\2 funcall(#[0 "r\304 q\210\305 )\306\307\310\311\312\313 !\314\"\315\ ert-run-test([cl-struct-ert-test tramp-test26-process-file "Check `p ert-run-or-rerun-test([cl-struct-ert--stats t [[cl-struct-ert-test t ert-run-tests(t #[385 "\306 \307\"\203D\211\211G\310U\203 \211@\20 ert-run-tests-batch(nil) ert-run-tests-batch-and-exit() command-line-1(("-L" "/home/zhongwei/projects/project/tramp/test/../ command-line() normal-top-level() Test tramp-test26-process-file condition: (error "Process *tramp/adb 10.164.2.195* not running") FAILED 27/35 tramp-test26-process-file skipped 28/35 tramp-test27-start-file-process skipped 29/35 tramp-test28-shell-command skipped 30/35 tramp-test29-vc-registered skipped 31/35 tramp-test30-special-characters passed 32/35 tramp-test31-utf8 skipped 33/35 tramp-test32-asynchronous-requests passed 34/35 tramp-test33-recursive-load failed 35/35 tramp-test34-unload Ran 35 tests, 27 results as expected, 1 unexpected, 7 skipped (2014-10-29 19:49:39+0800) 1 expected failures 1 unexpected results: FAILED tramp-test26-process-file 7 skipped results: SKIPPED tramp-test20-file-modes SKIPPED tramp-test21-file-links SKIPPED tramp-test27-start-file-process SKIPPED tramp-test28-shell-command SKIPPED tramp-test29-vc-registered SKIPPED tramp-test30-special-characters SKIPPED tramp-test32-asynchronous-requests make[1]: *** [check] Error 1 make[1]: Leaving directory `/home/zhongwei/projects/project/tramp/test' make: *** [check] Error 2 But when I remove the kill-buffer-query-functions at line 1249, the test passes. I haven't taken a good look at it because of my test environment broken after system upgrade:( > > > Best regards, > > Zhongwei > > Best regards, Michael. >
_______________________________________________ Tramp-devel mailing list Tramp-devel@gnu.org https://lists.gnu.org/mailman/listinfo/tramp-devel