Date: Tue, 23 Oct 2018 17:33:14 -0700 (PDT) From: "Rodney W. Grimes" <freebsd-rwg@pdx.rh.CN85.dnsmgr.net> To: Warner Losh <imp@bsdimp.com> Cc: Brooks Davis <brooks@freebsd.org>, FreeBSD-STABLE Mailing List <freebsd-stable@freebsd.org>, Ian Lepore <ian@freebsd.org>, FreeBSD Net <freebsd-net@freebsd.org>, "Julian H. Stacey" <jhs@berklix.com>, Michelle Sullivan <michelle@sorbs.net>, "freebsd-arch@freebsd.org" <freebsd-arch@freebsd.org>, freebsd-fcp@freebsd.org Subject: Re: FCP-0101: Deprecating most 10/100 Ethernet drivers Message-ID: <201810240033.w9O0XEep025273@pdx.rh.CN85.dnsmgr.net> In-Reply-To: <CANCZdforDVmD-yBZP5Xm2=CbeOU0PyDAMUrZK95_yGXBgWoKhw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Tue, Oct 23, 2018 at 6:02 PM Rodney W. Grimes < > freebsd-rwg@pdx.rh.cn85.dnsmgr.net> wrote: ... deleted un relavent text ... > > > > > > Brooks, > > > > > > Is there a public revised version of FCP-0101 that > > reflects the > > > > > > feedback which is what core is voting on? > > > > > > > > > > > > > > > > Its on github, just like it's been the whole time for anybody to see, > > > > > submit pull requests against and track: > > > > > > > > I have no gh account, desires no gh account, so have no way to > > > > submit a change request other than through direct email to > > > > brooks or another gh user. This is fundementally flawed. > > > > > > > > > https://github.com/freebsd/fcp/blob/master/fcp-0101.md > > > > > > > > Thank you for the link, I had looked at it before MeetBSD, > > > > which did not have most of the recent changes done "a day ago". > > > > > > > > Isnt this document now in a frozen state while core reviews/votes? > > > > > > I sent it to a vote at c224c67557297d7cba909ad008542cb60980cc6b only > > > to notice a bug in table rendering. I submitted a pull request fix > > > that and a missing word which was merged since neither was material. I > > > suppose they could have waited or been skipped, but there's no value in > > > the FCP process being bound by the sort of pointless rigidity that led > > > to -DPOSIX_MISTAKE in every libc compile line. > > > > The FCP process itself is unclear on this point, > > I think this should be clarified. > > > > It is much more clear on post approval: > > Changes after acceptance > > > > FCPs may need revision after they have been moved into the > > accepted state. In such cases, the author SHOULD update the > > FCP to reflect the final conclusions. > > If the changes are major, the FCP SHOULD be withdrawn > > and restarted. > > > > Good point Rod. While common sense suggests that trivial changes during the > voting should be allowed for editorial purposes (eg fix grammar mistakes, > table rendering etc), it's better to spell that out so there's no confusion. Agreed. > > diff --git a/fcp-0000.md b/fcp-0000.md > index b4fe0f3..c8cc6f7 100644 > --- a/fcp-0000.md > +++ b/fcp-0000.md > @@ -144,7 +144,10 @@ When the discussion of a change has come to a suitable > and acceptable close it > SHOULD be updated to the `vote` state. > > At this time the FreeBSD Core Team will vote on the subject of the FCP. The > -result of vote moves the FCP into the `accepted` or `rejected` state. > +result of vote moves the FCP into the `accepted` or `rejected` state. The > +core team MAY make minor edits to the FCP to correct minor mistakes. Core > +MAY return the proposal to the submitter if there are major problems that > +need to be addressed. This allows and clarifies that core may modify it, it does not address if the author/submitter can modify it. > > FCPs in the `accepted` state can be updated and corrected. > See the "Changes after acceptance" section for more information. > > Normally I'd submit that as a pull request, but since I know you don't use > github, I thought I'd present it here to see if this answers your concerns > before doing so. I thank you very much for that consideration, -- Rod Grimes rgrimes@freebsd.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201810240033.w9O0XEep025273>