[email protected] (Pierre Téchoueyres) writes: > Hello everybody,
Hi Pierre, > After digging into the traces (thanks to `tramp-verbose'), I've found > `tramp-do-file-attributes-with-stat' and > `tramp-do-directory-files-and-attributes-with-stat'. > On each of theses functions a replace of // is done. > > But this is done to workaround a bug in pdks on Opsware and I can't test > it. > > The workaround I propose is to replace the //%N// of the stat command by > **%N**. I thing it's impossible to create a link with stars (*) in name. Oh, it's possible: # touch /tmp/\*\*tmp\*\* # ln -s /tmp/\*\*tmp\*\* /tmp/\*\*\*tmp\*\*\* # ls -al /tmp/*tmp* lrwxrwxrwx 1 albinus albinus 12 Aug 20 10:00 /tmp/***tmp*** -> /tmp/**tmp** -rw-rw-r-- 1 albinus albinus 0 Aug 20 10:00 /tmp/**tmp** And your patch makes the Tramp test suite fail. Try # make check The problem is, that in function `tramp--test-special-characters' the file "*foo*bar*baz*" is checked. With your mask "**", the stat command temporarily returns "***foo*bar*baz***", Tramp cannot decide safely, which of the "**" characters has to be replaced in sed. So we would need another mask, which won't appear most likely in the file name. Maybe you play with generating an arbitrary string, as `tramp-end-of-heredoc' does? Best regards, Michael. _______________________________________________ Tramp-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/tramp-devel
