Artifact [8debe85449]
Not logged in

Artifact 8debe854495dba9acf9e3ace02eeb4f39471ea13:

Ticket change [8debe85449] - New ticket [995a864c5a] Editor command fails with spaces. by mjanssen 2011-02-02 14:43:05.
D 2011-02-02T14:43:05.079
J comment The\sfix\sfor\sbug\s[8d073be8808beaa7b614b6d489e729dfb51ac114]\sonly\sseems\sto\shave\sfixed\sthe\sissue\sfor\sthe\sgdiff\scommand.\sIf\seditor\scontains\sspaces,\sexecuting\sit\sstill\sfails.\r\n\r\n<verbatim>\r\nC:\\Docs\\Projects\\01.\sLean\sSDMC>fossil\scommit\r\nc:/Program\sFiles/Vim/vim73/gvim.exe\s"C:/<snipped>/ci-comment-5\r\n260D688DBC8.txt"\r\n'c:/Program'\sis\snot\srecognized\sas\san\sinternal\sor\sexternal\scommand,\r\noperable\sprogram\sor\sbatch\sfile.\r\nc:\\Docs\\Bin\\fossil.exe:\seditor\saborted\r\n</verbatim>\r\n\r\nDiff\sbelow\sfixes\sthis,\sbut\sa\sbetter\ssolution\swould\sprobably\sbe\sto\sextend\sthe\sportable_system\scommand\sto\scover\sall\suses\sof\sfossil_system\r\n\r\n---\ssrc/checkin.c\r\n+++\ssrc/checkin.c\r\n@@\s-442,11\s+442,11\s@@\r\n\s#if\sdefined(_WIN32)\r\n\s\s\sblob_add_cr(&text);\r\n\s#endif\r\n\s\s\sblob_write_to_file(&text,\szFile);\r\n\s\s\sif(\szEditor\s){\r\n-\s\s\s\szCmd\s=\smprintf("%s\s\\"%s\\"",\szEditor,\szFile);\r\n+\s\s\s\szCmd\s=\smprintf("\\"%s\\"\s\\"%s\\"",\szEditor,\szFile);\r\n\s\s\s\s\sprintf("%s\\n",\szCmd);\r\n\s\s\s\s\sif(\sfossil_system(zCmd)\s){\r\n\s\s\s\s\s\s\sfossil_panic("editor\saborted");\r\n\s\s\s\s\s}\r\n\s\s\s\s\sblob_reset(&text);
J foundin b6a4e8592d58ead7878f4a072fc6c4da71c7ea97
J private_contact 9e3254e6e3c2643cc148bab2d55f46bb4c0446f4
J severity Important
J status Open
J title Editor\scommand\sfails\swith\sspaces
J type Code_Defect
K 995a864c5adf79456808d8f2bcd44c4959f46136
U mjanssen
Z 751d22182fca9442c3a669a5db5f89c9