Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-11-13 Thread Enrico Tröger
Closed #1259. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1259#event-857057248

Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-11-13 Thread Enrico Tröger
This should be fixed now in Geany 1.29 (released today). We merged #1095 which fixes locale issues when spawning external commands. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-10-24 Thread Mikhail
Yes, with attached here version the bug is gone. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1259#issuecomment-255931637

Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-10-24 Thread Enrico Tröger
@NTMan this might be related to #1076 and a possible fix (or a step to a fix) might be in #1095. It'd be cool if you could try: http://download.geany.org/snapshots/old/geany-1.28nightly20160703_pr1095_setup.exe (and

Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-10-13 Thread elextr
@NTMan you are welcome to provide patches, but (AFAIK) we currently have no regular windows contributor, so it may be slow if you don't I'm afraid. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-10-13 Thread elextr
Yes, forgot that, but now we don't use glib to spawn stuff on windows any more so we need the right encoding to pass to spawn unless it converts? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-10-13 Thread elextr
@b4n [WTF](https://github.com/geany/geany/blob/master/src/utils.c#L1238) that doesn't look like it handles locale on windows, and [build](https://github.com/geany/geany/blob/master/src/build.c#L804) tests the result immediately after return, using the still UTF-8 directory name, so no wonder

Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-10-13 Thread Mikhail
And this is possible fix? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1259#issuecomment-253509543

Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-10-07 Thread elextr
If it only fails with non-ASCII characters in the filename it looks like a disagreement about the locale. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1259#issuecomment-252388770

Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-10-07 Thread Mikhail
> The directory where eslint is found must be in the %PATH% environment > variable. ![08 10 2016 02-13](https://cloud.githubusercontent.com/assets/200750/19205653/36c26674-8cfd-11e6-9cd1-2485f3a7bc95.png) Of course I know about it. This is Windows specific problem not reproduced on Linux. >

Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-10-07 Thread Colomban Wendling
What is the error displayed in the statusbar (between the parentheses)? I unfortunately don't read Russian so I can't tell, but I guess it must be something like "no such file or directory", "failed to change directory" or something like that? And as @codebrainz suggests, I'd guess it's

Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-10-07 Thread Matthew Brush
> Geany able launch eslint only when full path specified. This is incorrect. The directory where eslint is found must be in the `%PATH%` environment variable. Geany will need to be restarted after changing the environment variables. > if checked file stored with long path name. Maybe it's the

Re: [Github-comments] [geany/geany] not working Lint in Geany on Windows (#1259)

2016-10-07 Thread Mikhail
Geany able launch eslint only when full path specified. This is uncorrect. Second problem: Geany couldn't again launch eslint (even if full path specified for eslint) if checked file stored with long path name. Demonstration: https://youtu.be/IO31rX8JfNs -- You are receiving this because you