[Bug 22006] Multiple applications require support for thread and process security descriptors (Continuum 0.40, Acrobat Reader DC 2015)

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=22006

Zebediah Figura <z.figura12 at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |z.figura12 at gmail.com

--- Comment #13 from Zebediah Figura <z.figura12 at gmail.com> ---
(In reply to Anastasius Focht from comment #12)
> It's unfortunate that the removal of Wine-Staging patchset
> https://github.com/wine-staging/wine-staging/tree/v3.16/patches/server-
> CreateProcess_ACLs patchset was hidden behind a rebase commit
> https://github.com/wine-staging/wine-staging/commit/
> 3f082c2d0ad0ad46037daac23d785ca159c0dbab which did more things than just
> that.

Without trying to sound defensive, I'm not sure I follow; I see that as part of
the process of rebasing, which means essentially "make all of the Staging patch
sets apply against the latest upstream commit". That is, the
server-CreateProcess_ACLs patch set applied against 93994dfc0, and did not
apply against 3f082c2d0, and was removed because it was no longer necessary.
Splitting up the parts of a rebase would be pointless (the intermediate
versions would not apply against anything, at all) and take extra time.

> Whenever a patchset is dropped from Wine-Staging, the description file shall
> be checked for any WineHQ Bugzilla bug references to update the ticket.

I tend to assume that a commit author will take action to resolve (or ask for a
retest on, etc.) the bug they fix. In future I'll try to be more diligent, if
this is unreliable.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.