Thank you)))<div>So, Alexander adviced me to make a pull request, he was busy when I asked him to look at the changes)</div><div>So, windows does not support VTE, instead of that the separated console window is opened.</div>
<div>About build system, as build system I used was MinGW/MSys, I will do supporting for WAF too.</div><div>Anyway, will do more explanatory commit message, waf supporting & some small code change))</div><div><br><br>
<div class="gmail_quote">On Thu, Feb 7, 2013 at 4:09 AM, Matthew Brush <span dir="ltr"><<a href="mailto:mbrush@codebrainz.ca" target="_blank">mbrush@codebrainz.ca</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On 13-02-06 02:31 PM, vadim kochan wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
Can someone please take a look at this?<br>
<a href="https://github.com/geany/geany-plugins/pull/83" target="_blank">https://github.com/geany/<u></u>geany-plugins/pull/83</a><br>
<br>
</blockquote>
<br></div></div>
Hi,<br>
<br>
I looked when it was submitted, didn't see any commit message details, 10 files and 900+ lines changed, Windows-specific, Win32 API-using and that it's not one of the plugins I maintain, and ran away screaming like a little girl :)<br>
<br>
Did you get in touch with Alexander Petukhov (cesspit) the plugin maintainer at all? He's probably gonna be your best bet for getting these changes in (or at least find out if he's inclined/has time to review/test/merge and/or minds if someone else does). Until word from him, probably nobody is going to get too involved, especially with whispers of a new release coming in the quite near future.<br>
<br>
>From a quick glance at the pull request (not code review):<br>
<br>
- Needs more commit message details (lots more)<br>
- Maybe it would be better as a few smaller commits, maybe<br>
- What replaces the guarded-out VTE/pty/etc code on Windows?<br>
- Why another build system instead of Waf and/or Autotools?<br>
* I don't think other plugins does this and it misses one<br>
of the benefits of Geany-Plugins project (shared build/release<br>
infrastructure). Better would probably be to making it<br>
build with Waf on Windows so that it gets released/distrubuted<br>
on Windows platform.<br>
- Why?<br>
* <a href="https://github.com/vkochan/geany-plugins/commit/fda45f31d11fcc73ddefc06ca3e2d38eace5d802#L6R41" target="_blank">https://github.com/vkochan/<u></u>geany-plugins/commit/<u></u>fda45f31d11fcc73ddefc06ca3e2d3<u></u>8eace5d802#L6R41</a><br>
* <a href="https://github.com/vkochan/geany-plugins/commit/fda45f31d11fcc73ddefc06ca3e2d38eace5d802#L5L474" target="_blank">https://github.com/vkochan/<u></u>geany-plugins/commit/<u></u>fda45f31d11fcc73ddefc06ca3e2d3<u></u>8eace5d802#L5L474</a><br>
* <a href="https://github.com/vkochan/geany-plugins/commit/fda45f31d11fcc73ddefc06ca3e2d38eace5d802#L1R45" target="_blank">https://github.com/vkochan/<u></u>geany-plugins/commit/<u></u>fda45f31d11fcc73ddefc06ca3e2d3<u></u>8eace5d802#L1R45</a><br>
<br>
Cheers,<br>
Matthew Brush<br>
<br>
______________________________<u></u>_________________<br>
Devel mailing list<br>
<a href="mailto:Devel@lists.geany.org" target="_blank">Devel@lists.geany.org</a><br>
<a href="https://lists.geany.org/cgi-bin/mailman/listinfo/devel" target="_blank">https://lists.geany.org/cgi-<u></u>bin/mailman/listinfo/devel</a><br>
</blockquote></div><br></div>