Mail Archives: cygwin/2003/07/02/10:24:07
On Wed, 2 Jul 2003, Brian Dessent wrote:
> [snip]
> There's a couple of problems with it still, in the backspaces/quotes
> department. Your .reg file installs the command:
>
> "%CYGROOT%\\bin\\bash -c \"echo -n `/bin/cygpath -u
> '%l'`>/dev/clipboard\"" + NewLine
>
> When I run the command I get an error. The proper quoting is
>
> "%CYGROOT%\bin\bash" -c "echo -n `/bin/cygpath -u '%l'`>/dev/clipboard"
>
> You don't want to escape the double-quotes because they are there to
> tell the windows shell to make all that stuff a single arg, after -c.
> You need double quotes around the exe image in the off chance there's a
> space in $CYGROOT. And there's the issue of the raw binary newline at
> the end. The hexified version of that is
> hex(2):22,25,43,59,47,52,4f,4f,54,25,5c,62,69,6e,5c,62,61,73,68,22,20,2d,63,20,22,65,63,68,6f,20,2d,6e,20,60,2f,62,69,6e,2f,63,79,67,70,61,74,68,20,2d,75,20,27,25,6c,27,60,3e,2f,64,65,76,2f,63,6c,69,70,62,6f,61,72,64,22,00
>
> ----
>
> Anyway, the Right Way (IMHO) to do this would be something like the
> following:
>
> ----- copy_cygpath.c -----
> #include <sys/cygwin.h>
> #include <windows.h>
>
> int main(int argc, char **argv)
> {
> HGLOBAL hglbBuffer;
> LPTSTR lptstrBuffer;
>
> if(argc != 2) {
> // usage: copy_cygpath [win32 path]
> return 1;
> }
>
> hglbBuffer = GlobalAlloc(GMEM_MOVEABLE, (MAX_PATH + 1)*sizeof(TCHAR));
> if (hglbBuffer == NULL) {
> return 1;
> }
>
> lptstrBuffer = GlobalLock(hglbBuffer);
> cygwin_conv_to_full_posix_path(argv[1], lptstrBuffer);
> GlobalUnlock(hglbBuffer);
>
> if(OpenClipboard(NULL) == 0) {
> // failure!
> GlobalFree(hglbBuffer);
> return 1;
> }
>
> EmptyClipboard();
> SetClipboardData(CF_TEXT, hglbBuffer);
> CloseClipboard();
>
> return 0;
> }
> ------
Hmm, again, why not do the following?
----- copy_cygpath.c -----
#include <sys/cygwin.h>
#include <windows.h>
#include <stdio.h>
int main(int argc, char **argv)
{
char res[MAX_PATH + 1];
FILE *clipboard;
if(argc != 2) {
// usage: copy_cygpath [win32 path]
return 1;
}
cygwin_conv_to_full_posix_path(argv[1], res);
if ((clipboard = fopen("/dev/clipboard", "w")) == NULL) {
// failure!
return 1;
}
fprintf(clipboard, "%s", res);
fclose(clipboard);
return 0;
}
------
> $ gcc copy_cygpath.c -o copy_cygpath.exe -mwindows -luser32
> $ mv copy_cygpath.exe /bin
>
> Now your registry entry is just:
>
> "%CYGROOT%\bin\copy_cygpath.exe" "%1"
>
> or
>
> ------
> REGEDIT4
>
> [HKEY_CLASSES_ROOT\Directory\shell\CygPath]
> @="&Copy LFN Cygwin Path"
>
> [HKEY_CLASSES_ROOT\Directory\shell\CygPathLFN\Command]
> @=hex(2):22,25,43,59,47,52,4f,4f,54,25,5c,62,69,6e,5c,63,6f,70,79,5f,63,79,67,70,61,74,68,2e,65,78,65,22,20,22,25,31,22,00
> ------
>
> This has the advantage of loading a single process, rather than bash,
> echo, and cygpath. You also eliminate the silly console window that
> flashes open and then closes.
FYI, "echo" is a bash builtin, so the only 2 processes loaded are "bash"
and "cygpath". Also, you could use "run" to eliminate the console window
(with one more process to load, of course).
Igor
> One might also want to change the C code to backslash escape spaces and
> other non-[\w\d] characters. That way you could still work with the
> long filenames at the command prompt. Alternatively you could have it
> paste the path as '/path/with a/space' (with the quotes.)
> Brian
--
http://cs.nyu.edu/~pechtcha/
|\ _,,,---,,_ pechtcha AT cs DOT nyu DOT edu
ZZZzz /,`.-'`' -. ;-;;,_ igor AT watson DOT ibm DOT com
|,4- ) )-,_. ,\ ( `'-' Igor Pechtchanski, Ph.D.
'---''(_/--' `-'\_) fL a.k.a JaguaR-R-R-r-r-r-.-.-. Meow!
"I have since come to realize that being between your mentor and his route
to the bathroom is a major career booster." -- Patrick Naughton
--
Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple
Problem reports: http://cygwin.com/problems.html
Documentation: http://cygwin.com/docs.html
FAQ: http://cygwin.com/faq/
- Raw text -