Jürgen Hötzel <[email protected]> writes:

> Anyway: This could also occur on a GNU/Linux system, when using 32bit
> UIDs: 
>
> "sudo chown 4294967294 test"

For sure. Fortunately, this didn't happen in the wild, otherwise I would
have been plagued with bug reports for years :-)

> The use of floats instead of 29bit integers fixes the issue. Thanks

Hmm. But I was to hasty, there is a second place which must be
changed. The whole patch reads now

--8<---------------cut here---------------start------------->8---
*** /home/albinus/src/tramp-2-1-stable/lisp/tramp.el.~2.814.2.18~       
2012-05-08 17:32:37.635812395 +0200
--- /home/albinus/src/tramp-2-1-stable/lisp/tramp.el    2012-05-08 
17:27:05.043185491 +0200
***************
*** 2946,2953 ****
      (tramp-get-test-command vec)
      (tramp-shell-quote-argument localname)
      (tramp-get-remote-stat vec)
!     (if (eq id-format 'integer) "%u" "\"%U\"")
!     (if (eq id-format 'integer) "%g" "\"%G\"")
      (tramp-shell-quote-argument localname))))
  
  (defun tramp-handle-set-visited-file-modtime (&optional time-list)
--- 2946,2953 ----
      (tramp-get-test-command vec)
      (tramp-shell-quote-argument localname)
      (tramp-get-remote-stat vec)
!     (if (eq id-format 'integer) "%ue0" "\"%U\"")
!     (if (eq id-format 'integer) "%ge0" "\"%G\"")
      (tramp-shell-quote-argument localname))))
  
  (defun tramp-handle-set-visited-file-modtime (&optional time-list)
***************
*** 3401,3408 ****
      (tramp-shell-quote-argument localname)
      (tramp-get-ls-command vec)
      (tramp-get-remote-stat vec)
!     (if (eq id-format 'integer) "%u" "\"%U\"")
!     (if (eq id-format 'integer) "%g" "\"%G\""))))
  
  ;; This function should return "foo/" for directories and "bar" for
  ;; files.
--- 3401,3408 ----
      (tramp-shell-quote-argument localname)
      (tramp-get-ls-command vec)
      (tramp-get-remote-stat vec)
!     (if (eq id-format 'integer) "%ue0" "\"%U\"")
!     (if (eq id-format 'integer) "%ge0" "\"%G\""))))
  
  ;; This function should return "foo/" for directories and "bar" for
  ;; files.
--8<---------------cut here---------------end--------------->8---

Committed to Tramp's repository.

> Jürgen 

Best regards, Michael.

_______________________________________________
Tramp-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/tramp-devel

Reply via email to