[macports-ports] branch master updated: nmap: fixes for 32-bit and pre-C++11 platforms
Joshua Root
jmr at macports.org
Wed Nov 1 22:29:17 UTC 2023
An understandable presumption in itself, however there was another force
push after dluke's last comment, so the changes that ended up being
merged had not been reviewed by the maintainer.
This was pretty clearly just a misunderstanding, but maybe we should
clarify the policies to only allow merging by others (before timeout)
when there is a non-stale approving review from the maintainer.
- Josh
On 2/11/2023 09:12, Perry E. Metzger wrote:
> Your communication in the thread was:
>
> > ideally we'll get confirmation from @barracuda156
> <https://github.com/barracuda156> that we've got a working solution
> before we merge as well.
>
> After he verified that it was working for him, I presumed you were okay
> with it being committed.
>
> Perry
>
> On 11/1/23 17:22, Daniel J. Luke wrote:
>> Perry - I had not yet signed off on this PR.
>>
>> The port is not openmaintainer.
>>
>> Please refrain from making changes to non-openmaintainer ports without the maintainer's approval.
>>
>>> On Nov 1, 2023, at 12:48 PM, Christopher Chavez via macports-changes<macports-changes at lists.macports.org> wrote:
>>> Perry E. Metzger (pmetzger) pushed a commit to branch master
>>> in repository macports-ports.
More information about the macports-dev
mailing list