This is the mail archive of the mailing list for the glibc project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PING] [PATCH] sys/ptrace.h: remove obsolete Linux PTRACE_SEIZE_DEVEL constant

On 08/08/2017 12:30 PM, Dmitry V. Levin wrote:
> On Tue, Aug 08, 2017 at 09:20:17AM -0400, Carlos O'Donell wrote:
>> On 08/07/2017 11:33 AM, Dmitry V. Levin wrote:
>>> Hi,
>>> Looks like among those few who care about sys/ptrace.h nobody feels
>>> experienced enough to review this change, so I'll go forward and commit it.
>> Please tread carefully, and give the machine maintainer time to review, or
>> directly TO: the machine maintainers and ask for review.
>> Lack of a response does not mean you can assume consensus. Follow up with
>> machine maintainers, even one ACK from a maintainer goes a long way to
>> knowing there is support for your change.
> JFYI, PTRACE_SEIZE_DEVEL was an architecture-independent constant.

Agreed, but the patch still touches machine-specific headers.
> There is absolutely no hurry, though, so let's give machine maintainers
> more time to review this obvious change.

In this case I think it is perfectly acceptable to track down reviews from
other developers, enough to get consensus that the generic change is OK.
For example if a couple of non-machine maintainers reviewed this an thought
it sensible, then you'd be OK to commit. With machine maintainers perhaps
tweaking this later.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]