Re: [Github-comments] [geany/geany] Add .editorconfig (#1193)

2016-10-13 Thread Colomban Wendling
Yeah why not, as said I don't really mind, and if it helps sure. -- 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/pull/1193#issuecomment-253436837

Re: [Github-comments] [geany/geany] Add .editorconfig (#1193)

2016-10-13 Thread Colomban Wendling
b4n commented on this pull request. > @@ -0,0 +1,11 @@ +root = true + +[*] +indent_size = 4 +end_of_line = lf + +[*.{c,h,cxx,am,m4}] +indent_style = tab couldn't that apply to everything, and only special-case some other stuff? Or is the goal here not to risk overriding an automatic detection

Re: [Github-comments] [geany/geany] Add .editorconfig (#1193)

2016-10-13 Thread elextr
elextr commented on this pull request. > @@ -0,0 +1,11 @@ +root = true + +[*] +indent_size = 4 +end_of_line = lf + +[*.{c,h,cxx,am,m4}] +indent_style = tab it should be gobal. When we edit Geany in Geany all filetypes should get indent 4 and the idea is to make github look the same. -- You

Re: [Github-comments] [geany/geany] Add .editorconfig (#1193)

2016-10-13 Thread Colomban Wendling
b4n commented on this pull request. > @@ -0,0 +1,11 @@ +root = true + +[*] +indent_size = 4 +end_of_line = lf + +[*.{c,h,cxx,am,m4}] +indent_style = tab Indentation width seems global, just not style (tab or space) -- You are receiving this because you are subscribed to this thread. Reply to

Re: [Github-comments] [geany/geany] Add .editorconfig (#1193)

2016-10-13 Thread elextr
elextr commented on this pull request. > @@ -0,0 +1,11 @@ +root = true + +[*] +indent_size = 4 +end_of_line = lf + +[*.{c,h,cxx,am,m4}] +indent_style = tab Sorry, should have added, my googling seems to suggest that the indent_width only works where indent_style = tab. -- You are receiving t

Re: [Github-comments] [geany/geany] Add .editorconfig (#1193)

2016-10-13 Thread Colomban Wendling
b4n commented on this pull request. > @@ -0,0 +1,11 @@ +root = true + +[*] +indent_size = 4 +end_of_line = lf + +[*.{c,h,cxx,am,m4}] +indent_style = tab Seems its spelled `indent_size` :) And I'd be surprised if there was no way to change the indentation size when using spaces. Though, there

Re: [Github-comments] [geany/geany] Add .editorconfig (#1193)

2016-10-13 Thread elextr
elextr commented on this pull request. > @@ -0,0 +1,11 @@ +root = true + +[*] +indent_size = 4 +end_of_line = lf + +[*.{c,h,cxx,am,m4}] +indent_style = tab width/size whatever :) the point is that from what I could find by googling (there is no official github documentation) is that github onl

Re: [Github-comments] [geany/geany] Add .editorconfig (#1193)

2016-10-13 Thread Colomban Wendling
b4n commented on this pull request. > @@ -0,0 +1,11 @@ +root = true + +[*] +indent_size = 4 +end_of_line = lf + +[*.{c,h,cxx,am,m4}] +indent_style = tab Oh you mean for display. Yeah sure then width doesn't matter for spaces. And to check GitHub it's fairly easy, just browse the file as change

Re: [Github-comments] [geany/geany] Add .editorconfig (#1193)

2016-10-13 Thread elextr
elextr commented on this pull request. > @@ -0,0 +1,11 @@ +root = true + +[*] +indent_size = 4 +end_of_line = lf + +[*.{c,h,cxx,am,m4}] +indent_style = tab Yeah, thats what I mean. But, I guess limiting the indent_type=tab to only the filetypes that need it is probably useful when used for ed

Re: [Github-comments] [geany/geany-plugins] [spellchecker] Apostrophes (`'`, ascii 39) at string boundary are spell-checked (#484)

2016-10-13 Thread Enrico Tröger
It depends what OS you use. On Linux compiling from source is quite easy: ```sh git clone git://github.com/geany/geany cd geany ./autogen.sh --prefix=/tmp/geany_test make make install cd .. git clone git://github.com/geany/geany-plugins cd geany-plugins git checkout -b spellcheck_strip_quotes_issu

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

[Github-comments] [geany/geany-plugins] GeanyVC git push (#488)

2016-10-13 Thread systemdarena
Hi, just installed GeanyVC, which mostly looks to be what I want, but why does it lack the push command? I see there is "Update", but that only does a pull. atm i am resorting to just adding git push to geany's build commands, which I guess is about the same thing... -- You are receiving this

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 it

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

2016-10-13 Thread Matthew Brush
@elextr "filename encoding" on Windows is [always UTF-8 with GLib](https://developer.gnome.org/glib/stable/glib-Character-Set-Conversion.html#g-filename-from-utf8). -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://git

Re: [Github-comments] [geany/geany-plugins] [spellchecker] Apostrophes (`'`, ascii 39) at string boundary are spell-checked (#484)

2016-10-13 Thread Andrea Stacchiotti
I brutally replaced `spellcheck.so` in `/usr/lib` and tested it a bit. It seems to work perfectly well. Thanks for the 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-plugins/issues/484#is

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: https://github.com/geany/geany

Re: [Github-comments] [geany/geany] Provide utf-8 command line arguments on Windows (#1258)

2016-10-13 Thread Stefan Oberhumer
I'm not sure if it's necessary but have you checked you don't have to handle argv[0] the same way ? (Eg geany is in an directory containing unicode characters) -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.c

Re: [Github-comments] [geany/geany-plugins] GeanyVC git push (#488)

2016-10-13 Thread Frank Lanitz
push is missing on purpose as it differs - between VCS - on git there are branches and remote resources Both would need to be implemented making of course Geany a full graphical frontend but still a lot of work to do. -- You are receiving this because you are subscribed to this thread. Repl

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: https://github.com/geany/gea

Re: [Github-comments] [geany/geany] Proxy Plugins Filename Pattern Matching (#1236)

2016-10-13 Thread Thomas Martitz
> Read the comments in the linked #1233. A better question is why do you want > to match file extensions at all? They aren't unique enough to decide whether > a file is for a given proxy and you have to check them further in the probe() > function anyway. I couldn't find an answer, other than y