On Thu, Nov 29, 2018 at 11:00:18PM +0000, Stuart Henderson wrote:
> On 2018/11/29 17:36, George Koehler wrote:
> > On Thu, 29 Nov 2018 09:38:31 +0100
> > Olivier Taïbi <[email protected]> wrote:
> > 
> > > 2) remove the check
> > >     if (job_pid == getpgid(job_pid))
> > >  from vim's mch_signal_job() in src/os_unix.c so that the whole group
> > >  always gets a signal.
> > 
> > Thank you for finding the failing getpgid().  I suggest to remove the
> > check, like in the below diff.  The process group id should equal the
> > process id after vim called setsid(2) or setpgid(2).  I had helped to
> > remove a similar getpgid() call from meson:
> > https://github.com/mesonbuild/meson/commit/64906b0
> 
> Thanks for digging into this. I definitely would prefer to handle this
> via upstream rather than as a patch in ports..

I just sent this bug report upstream:
https://github.com/vim/vim/issues/3656

Reply via email to