Re: [PATCH] Cygwin: pipe: Give up to use query_hdl for non-cygwin apps.
Corinna Vinschen writes: > Right you are. We always said that independent Cygwin installations > are supposed to *stay* independent. > > Keep in mind that they don't share the same shared objects in the native > NT namespace and they don't know of each other. It's not only the > process table but also in-use FIFO stuff, pty info, etc. What I was getting at is that a process not showing up in the process list in one Cygwin installation doesn't automatically mean it's a native Windows process, it could be a process started by an independent Cygwin installation. So this way of checking for "native" Windows processes may or may not do what was originally intended. Regards, Achim. -- +<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+ Factory and User Sound Singles for Waldorf Q+, Q and microQ: http://Synth.Stromeko.net/Downloads.html#WaldorfSounds
Re: [PATCH] Cygwin: pipe: Give up to use query_hdl for non-cygwin apps.
Takashi Yano writes: >> After noticing that we enumerate all the processes (which is an expensive >> operation) just to skip all of the non-Cygwin ones anyway, I wonder if it >> wouldn't be smarter to go through the internal list of cygpids and take it >> from there, skipping the `SystemProcessInformation` calls altogether. > > Yeah, that makes sens. I'll submit v2 patch. Keep in mind that there may be different independent Cygwin installations running on the same nachine. Regards, Achim. -- +<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+ Waldorf MIDI Implementation & additional documentation: http://Synth.Stromeko.net/Downloads.html#WaldorfDocs
Re: [PATCH 0/2] Return appropriate handle by _get_osfhandle() and GetStdHandle().
Jon Turney writes: > 'run' is used by the start menu item which starts the X server. > > If that doesn't use it, a visible console window is created for the > bash process it starts (which is the parent of the X server process > and lives for it's lifetime). > > (As a separate issue, I'm not sure all the complex gymnastics run does > to creste the window invisibly are doing anything useful, since we > seem to briefly show the window and then hide it) It may be time to finally retire run and finish the work on run2… Regards, Achim. -- +<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+ SD adaptations for KORG EX-800 and Poly-800MkII V0.9: http://Synth.Stromeko.net/Downloads.html#KorgSDada
Re: [PATCH] winsup/doc/dll.xml: update MinGW/.org to MinGW-w64/.org
Ken Brown via Cygwin-patches writes: > If the unfinished business consists of local commits that haven't yet > been applied upstream, then I typically do the following: > > git fetch # Find out if upstream has changed since my last pull. If so... > git format-patch -n # save n local commits > git reset --hard origin/master > git am 00* # reapply my local commits > > This assumes I've been too lazy to work on a separate branch, which is > often the case for small changes. git branch local # optional git rebase origin/master git branch -D local # once you know you don't need it anymore … does more or less the same thing. I usually skip the safety branch since I can pull it from the reflog if needed and it's unlikely I'll trigger a GC just that moment. Regards, Achim. -- +<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+ SD adaptation for Waldorf microQ V2.22R2: http://Synth.Stromeko.net/Downloads.html#WaldorfSDada
Re: [PATCH] fhandler_proc.cc(format_proc_cpuinfo): fix issues, add fields, flags
Brian Inglis writes: >> It would be easier to review if you would split it up into smaller patches, >> each >> doing one thing, to the extent that this makes sense. For example, the >> simplification achieved by using the ftcprint macro could be done in a >> single >> patch that's separate from the substantive changes. > > Unfortunately, that was added later to make the got it/add it/skip it flag > cross > checks in Linux order more certain vs my own sequential tabular source. I usually use Emacs/magit to split up a bunch of changes into a more comprehensible series of commits post factum, but there are other ways to achieve that same goal. Regards, Achim. -- +<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+ Wavetables for the Waldorf Blofeld: http://Synth.Stromeko.net/Downloads.html#BlofeldUserWavetables
Re: [PATCH] fhandler_proc.cc(format_proc_cpuinfo): fix issues, add fields, flags
Brian Inglis writes: > For informal comparison, attached are Cygwin, WSL, and test release cpuinfo > output, with diffs against the test release output, and the Windows registry > CentralProcessor dump (be careful not to double click on Windows > systems!) The easiest way to prevent that problem would have been to give that file a .txt extension, no? Regards, Achim. -- +<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+ SD adaptation for Waldorf rackAttack V1.04R1: http://Synth.Stromeko.net/Downloads.html#WaldorfSDada