On Dec 18 11:28, Corinna Vinschen wrote: > On Dec 18 16:38, Takashi Yano wrote: > > Hi Corinna, > > > > On Mon, 15 Dec 2025 16:23:29 +0100 > > Corinna Vinschen wrote: > > > On Dec 5 17:38, Corinna Vinschen wrote: > > > > From: Corinna Vinschen <[email protected]> > > > > > > > > Fix broken code overriding primary group at process tree startup. THis > > > > is fallout frokm the newgrp(1) introduction which showed a problem with > > > > this code. The fix from commit dc7b67316d01 ("Cygwin: uinfo: prefer > > > > token primary group") broke this differently, so here we go, trying to > > > > fix the second problem without breaking the first again. > > > > > > > > Corinna Vinschen (3): > > > > Cygwin: uinfo: correctly check and override primary group > > > > Cygwin: uinfo: allow to override user account as primary group > > > > Cygwin: add release note for primary group override fix > > > > > > Ping? Anybody willing to review? > > > > I would like to review them for you, but I don’t understand what > > the problem is or how these patches are supposed to solve it... > > I actually hoped my commit messages would explain that. Is it > not adequate?
Yes, I think I see where the problem is. I'll send a v2 with improved commit message. Thanks, Corinna
