{"thread":{"id":"15675","subject":"[PATCH] git-gui: Help identify aspell version on Windows too","startedAt":"2008-09-25T21:31:22Z","lastAt":"2008-09-26T15:06:53Z","messageCount":7,"participants":["Gustaf Hendeby","Johannes Sixt","Shawn O. Pearce"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"91635","messageId":"1222378282-21757-1-git-send-email-hendeby@isy.liu.se","threadId":"15675","inReplyTo":null,"subject":"[PATCH] git-gui: Help identify aspell version on Windows too","fromName":"Gustaf Hendeby","fromEmail":"hendeby@isy.liu.se","sentAt":"2008-09-25T21:31:22Z","receivedAt":"2008-09-25T21:31:22Z","isPatch":true,"sender":{"key":"hendeby@isy.liu.se","avatar":"https://avatars.githubusercontent.com/u/730316?v=4"},"body":"On windows, git gui fails to correctly extract the aspell version\n(experienced with aspell version 0.50.3) due to scilent white space at\nthe end of the version string.  Trim the obtained version string to\nwork around this.\n\nSigned-off-by: Gustaf Hendeby <hendeby@isy.liu.se>\n---\n lib/spellcheck.tcl |    2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/lib/spellcheck.tcl b/lib/spellcheck.tcl\nindex 78f344f..c8cc030 100644\n--- a/lib/spellcheck.tcl\n+++ b/lib/spellcheck.tcl\n@@ -80,7 +80,7 @@ method _connect {pipe_fd} {\n \t\terror_popup [strcat [mc \"Unrecognized spell checker\"] \":\\n\\n$s_version\"]\n \t\treturn\n \t}\n-\tset s_version [string range $s_version 5 end]\n+\t\t\tset s_version [string range [string trim $s_version] 5 end]\n \tregexp \\\n \t\t{International Ispell Version .* \\(but really (Aspell .*?)\\)$} \\\n \t\t$s_version _junk s_version\n-- \n1.6.0.2.509.g4def\n"},{"id":"91660","messageId":"48DC815C.6030401@viscovery.net","threadId":"15675","inReplyTo":"1222378282-21757-1-git-send-email-hendeby@isy.liu.se","subject":"Re: [PATCH] git-gui: Help identify aspell version on Windows too","fromName":"Johannes Sixt","fromEmail":"j.sixt@viscovery.net","sentAt":"2008-09-26T06:29:48Z","receivedAt":"2008-09-26T06:29:48Z","isPatch":true,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"Gustaf Hendeby schrieb:\n> On windows, git gui fails to correctly extract the aspell version\n> (experienced with aspell version 0.50.3) due to scilent white space at\n> the end of the version string.  Trim the obtained version string to\n> work around this.\n\nWith this I've partial success here: Aspell 0.50.3 alpha (downloaded from\nhttp://aspell.net/win32/) is recognized, but\n\n- it only ever checks the first line of the commit message;\n\n- after I change the catalog in the Options, I can't \"Save\" (it does\nnothing), until I change to some other catalog. I haven't discovered the\npattern, yet, when the \"Save\" succeeds.\n\n- At startup the correct catalog is used. But if I change the catalog\nduring a session (if I succeed, see above), then the second-last picked\ncatalog is used.\n\nAny idea how to debug this?\n\n> @@ -80,7 +80,7 @@ method _connect {pipe_fd} {\n>  \t\terror_popup [strcat [mc \"Unrecognized spell checker\"] \":\\n\\n$s_version\"]\n>  \t\treturn\n>  \t}\n> -\tset s_version [string range $s_version 5 end]\n> +\t\t\tset s_version [string range [string trim $s_version] 5 end]\n\nA strange indentation you have here.\n\n>  \tregexp \\\n>  \t\t{International Ispell Version .* \\(but really (Aspell .*?)\\)$} \\\n>  \t\t$s_version _junk s_version\n\n-- Hannes\n"},{"id":"91669","messageId":"48DCA5F3.4090009@isy.liu.se","threadId":"15675","inReplyTo":"48DC815C.6030401@viscovery.net","subject":"Re: [PATCH] git-gui: Help identify aspell version on Windows too","fromName":"Gustaf Hendeby","fromEmail":"hendeby@isy.liu.se","sentAt":"2008-09-26T09:05:55Z","receivedAt":"2008-09-26T09:05:55Z","isPatch":true,"sender":{"key":"hendeby@isy.liu.se","avatar":"https://avatars.githubusercontent.com/u/730316?v=4"},"body":"On 09/26/2008 08:29 AM, Johannes Sixt wrote:\n> Gustaf Hendeby schrieb:\n>> On windows, git gui fails to correctly extract the aspell version\n>> (experienced with aspell version 0.50.3) due to scilent white space at\n>> the end of the version string.  Trim the obtained version string to\n>> work around this.\n> \n> With this I've partial success here: Aspell 0.50.3 alpha (downloaded from\n> http://aspell.net/win32/) is recognized, but\n> \n> - it only ever checks the first line of the commit message;\n\nHmm, strange, I just had it mark my whole first like red, I think it\ntook an enter at the end of the line to get everything started though.\nDoes it check the rest of the lines for you?  I know spell checking is\ndisabled for too early versions of Aspell, due to some change in the\ninterface I think.\n\n> \n> - after I change the catalog in the Options, I can't \"Save\" (it does\n> nothing), until I change to some other catalog. I haven't discovered the\n> pattern, yet, when the \"Save\" succeeds.\n> \n> - At startup the correct catalog is used. But if I change the catalog\n> during a session (if I succeed, see above), then the second-last picked\n> catalog is used.\n> \n> Any idea how to debug this?\n\nI have experienced some saving related issues I think, but I'm not sure.\n Unfortunately I have very little understanding on what is going on in\nthe code, basically I don't know tcl/tk at all.  I was just lucky to\nfind a solution to my problem that seemed to work for me.\n\n> \n>> @@ -80,7 +80,7 @@ method _connect {pipe_fd} {\n>>  \t\terror_popup [strcat [mc \"Unrecognized spell checker\"] \":\\n\\n$s_version\"]\n>>  \t\treturn\n>>  \t}\n>> -\tset s_version [string range $s_version 5 end]\n>> +\t\t\tset s_version [string range [string trim $s_version] 5 end]\n> \n> A strange indentation you have here.\n\nYep, sorry about that, seems I had a nonstandard tab-width setting.  Too\nbad not all projects can decide on the same tab width. :(  Will resend\nif there are no other comments to this, and at the same time fix the\nspelling in the commit message...\n\n/Gustaf\n"},{"id":"91671","messageId":"48DCAC08.9060004@viscovery.net","threadId":"15675","inReplyTo":"48DCA5F3.4090009@isy.liu.se","subject":"Re: [PATCH] git-gui: Help identify aspell version on Windows too","fromName":"Johannes Sixt","fromEmail":"j.sixt@viscovery.net","sentAt":"2008-09-26T09:31:52Z","receivedAt":"2008-09-26T09:31:52Z","isPatch":true,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"Gustaf Hendeby schrieb:\n> On 09/26/2008 08:29 AM, Johannes Sixt wrote:\n>> Gustaf Hendeby schrieb:\n>>> On windows, git gui fails to correctly extract the aspell version\n>>> (experienced with aspell version 0.50.3) due to scilent white space at\n>>> the end of the version string.  Trim the obtained version string to\n>>> work around this.\n>> With this I've partial success here: Aspell 0.50.3 alpha (downloaded from\n>> http://aspell.net/win32/) is recognized, but\n>>\n>> - it only ever checks the first line of the commit message;\n> \n> Hmm, strange, I just had it mark my whole first like red, I think it\n> took an enter at the end of the line to get everything started though.\n> Does it check the rest of the lines for you?  I know spell checking is\n> disabled for too early versions of Aspell, due to some change in the\n> interface I think.\n\nMy situation is this: I fire up git-gui, and because I have some stale\n.git/GITGUI_MSG file, the commit message box is not empty. Aspell begins\nits work, and git-gui correctly marks spelling mistakes in the first line.\nEven if I type new text, delete old text, or change the whole text by\nclicking \"Amend\", no new spelling mistakes are marked for the whole session.\n\n-- Hannes\n"},{"id":"91682","messageId":"20080926142818.GW3669@spearce.org","threadId":"15675","inReplyTo":"48DCA5F3.4090009@isy.liu.se","subject":"Re: [PATCH] git-gui: Help identify aspell version on Windows too","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2008-09-26T14:28:18Z","receivedAt":"2008-09-26T14:28:18Z","isPatch":true,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Gustaf Hendeby <hendeby@isy.liu.se> wrote:\n> >>  \t}\n> >> -\tset s_version [string range $s_version 5 end]\n> >> +\t\t\tset s_version [string range [string trim $s_version] 5 end]\n> > \n> > A strange indentation you have here.\n> \n> Yep, sorry about that, seems I had a nonstandard tab-width setting.  Too\n> bad not all projects can decide on the same tab width. :(  Will resend\n> if there are no other comments to this, and at the same time fix the\n> spelling in the commit message...\n\nI fixed the indentation, but didn't notice the spelling error in\nthe commit message.  Anyway, its too late, I have already pushed\nthis fix out to my maint and master branches in git-gui.  I'm not\ngoing to rewind 'em.\n\n-- \nShawn.\n"},{"id":"91684","messageId":"20080926143119.GX3669@spearce.org","threadId":"15675","inReplyTo":"48DCAC08.9060004@viscovery.net","subject":"Re: [PATCH] git-gui: Help identify aspell version on Windows too","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2008-09-26T14:31:19Z","receivedAt":"2008-09-26T14:31:19Z","isPatch":true,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Johannes Sixt <j.sixt@viscovery.net> wrote:\n> \n> My situation is this: I fire up git-gui, and because I have some stale\n> .git/GITGUI_MSG file, the commit message box is not empty. Aspell begins\n> its work, and git-gui correctly marks spelling mistakes in the first line.\n> Even if I type new text, delete old text, or change the whole text by\n> clicking \"Amend\", no new spelling mistakes are marked for the whole session.\n\nHmm.  Sounds to me like aspell crashes and dies after a while.\n\nWhen aspell terminates on its own git-gui doesn't try to restart it.\nI've never seen it abort on my Mac, or my Linux systems.  Or even\non Windows when I was using the Cygwin version of aspell.\n\n-- \nShawn.\n"},{"id":"91693","messageId":"48DCFA8D.7080905@viscovery.net","threadId":"15675","inReplyTo":"20080926143119.GX3669@spearce.org","subject":"Re: [PATCH] git-gui: Help identify aspell version on Windows too","fromName":"Johannes Sixt","fromEmail":"j.sixt@viscovery.net","sentAt":"2008-09-26T15:06:53Z","receivedAt":"2008-09-26T15:06:53Z","isPatch":true,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"Shawn O. Pearce schrieb:\n> Johannes Sixt <j.sixt@viscovery.net> wrote:\n>> My situation is this: I fire up git-gui, and because I have some stale\n>> .git/GITGUI_MSG file, the commit message box is not empty. Aspell begins\n>> its work, and git-gui correctly marks spelling mistakes in the first line.\n>> Even if I type new text, delete old text, or change the whole text by\n>> clicking \"Amend\", no new spelling mistakes are marked for the whole session.\n> \n> Hmm.  Sounds to me like aspell crashes and dies after a while.\n> \n> When aspell terminates on its own git-gui doesn't try to restart it.\n> I've never seen it abort on my Mac, or my Linux systems.  Or even\n> on Windows when I was using the Cygwin version of aspell.\n\nNo, it does not abort. It's still running, but sitting there idly.\n\n-- Hannes\n"}]}