Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 4 Sep 2020 22:24:31 -0700
From:      Kevin Bowling <kevin.bowling@kev009.com>
To:        Warner Losh <imp@bsdimp.com>
Cc:        Mark Linimon <linimon@lonesome.com>, Andrew Gallatin <gallatin@cs.duke.edu>, Alexey Dokuchaev <danfe@freebsd.org>, Michael Tuexen <tuexen@fh-muenster.de>,  Pedro Giffuni <pfg@freebsd.org>, Mateusz Guzik <mjg@freebsd.org>,  src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org,  svn-src-head@freebsd.org
Subject:   Re: svn commit: r365071 - in head/sys: net net/altq net/route net80211 netgraph netgraph/atm netgraph/atm/ccatm netgraph/atm/sscfu netgraph/atm/sscop netgraph/atm/uni netgraph/bluetooth/common netgraph...
Message-ID:  <CAK7dMtBotsYmHwonsBP4CpFZ0DUF0FOyqd=R5qiz_mHaUKLHrg@mail.gmail.com>
In-Reply-To: <CANCZdfqm90bcVXQzeGis=CQJRwXWf9JQ-0H5tOqdyKk44zNe0Q@mail.gmail.com>
References:  <202009012119.081LJERb018106@repo.freebsd.org> <95844C00-D10A-456D-AD29-DF572043074F@fh-muenster.de> <20200902020507.GA38274@FreeBSD.org> <eba32e79-4b90-ecce-7bbb-455f691d4444@FreeBSD.org> <20200902180626.GA88595@FreeBSD.org> <6124a908-25a5-e023-16da-7963ba229b7f@FreeBSD.org> <08636D5E-AA07-4AE7-B5AC-656B08CF564B@fh-muenster.de> <20200903024226.GA54078@FreeBSD.org> <60ea593f-8258-e30d-b897-f162168b44d3@cs.duke.edu> <20200905010510.GA26297@lonesome.com> <CANCZdfo8km1ahTSajJpLhyFyYyQsPr-ZwMWRZx_jmLGdRpc=WA@mail.gmail.com> <CAK7dMtCAktXP4wMDVfC8FeqnEwtMase7kY6Z=-4MazVQQWdXDQ@mail.gmail.com> <CANCZdfqm90bcVXQzeGis=CQJRwXWf9JQ-0H5tOqdyKk44zNe0Q@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
It's happening right now, and a few times a year at minimum from my
memory.  Any time someone proposes a formatter they are thrown shade,
so the lack of progress there isn't surprising since the current
culture would require a flame proof suit to make progress.  It's kind
of tautological that the status quo doesn't bother long time
contributors due to selection bias, and doesn't mean for instance the
lack of modern tooling is not off putting to younger developers that
learn new tools and wonder why we remain in the stone age.  An example
we are finally overcoming is the git migration.  Must we drag our feet
every opportunity given to modernize ourselves from the other popular
open source OS, or can we make obvious decisions to get ahead of them?
 I think if you ask anyone under the age of 30 you will get a pretty
unanimous desire for automatic formatting.

On Fri, Sep 4, 2020 at 8:48 PM Warner Losh <imp@bsdimp.com> wrote:
>
>
>
> On Fri, Sep 4, 2020, 9:11 PM Kevin Bowling <kevin.bowling@kev009.com> wro=
te:
>>
>> I disagree that the problem is intractable.  It's just a decision and
>> it has a one time cost with long term benefits like paying off a high
>> interest loan.  The intractability opinion seemed justifiable for a
>> long time but it's been proven false by other communities,
>> particularly Go and Rust and there is nothing syntactically special
>> about these languages that enable this; it's just a decision to make
>> the style fit an extant formatter.  An arbitrary formater may leave a
>> little bit of annoyance to each person's taste, but that is a tiny
>> drop in the bucket compared to never having to discuss and especially
>> correct (which may /seem/ helpful but is pretty offputting to
>> newcomers).  A tool does it, and it takes the wind out of any passive
>> aggressive bike shed opportunities from either maintainer or
>> contributor.  It sucks that downstreams have to fall in line, but that
>> doesn't stop progress on any other major changes in FreeBSD.
>
>
> How often are there really such bikesheds these days? I've seen no eviden=
ce of them in the hundreds of phab reviews I've seen. It is the ghost of th=
e past when 10 or 15 years ago it was a big deal. Why bother creating yet a=
nother barrier to commits because we used to suck, but now have barely a ru=
mble of bad behavior around it...
>
> Warner
>
>> On Fri, Sep 4, 2020 at 7:57 PM Warner Losh <imp@bsdimp.com> wrote:
>> >
>> > On Fri, Sep 4, 2020, 7:05 PM Mark Linimon <linimon@lonesome.com> wrote=
:
>> >
>> > > On Fri, Sep 04, 2020 at 02:15:04PM -0400, Andrew Gallatin wrote:
>> > > > and I also anticipate it will cause problems with MFCs
>> > >
>
>
>> > > And existing PRs and DRs.
>> > >
>> >
>> > Or we could just not bother we these changes at all. It's a pipe dream=
 we
>> > will ever be style(9) compliant in all our code, or that we can magica=
lly
>> > have a tool to enforce in new commits. We have better things to worry
>> > about. We should continue to ignore this non problem and for new users
>> > point them at the 95% correct format thing to run their submitted patc=
hes
>> > if they submit something too far out of whack.
>> >
>> > The last sweep deleted a boatload of blank lines that were in there on
>> > purpose. Not worth adding them back, but still annoying to no real ben=
efit.
>> >
>> > I just don't see the benefits at all of doing anything here. The few
>> > reviews that I've seen mention it seem to be the right level of effort=
.
>> >
>> > Warner
>> >
>> > >
>> > _______________________________________________
>> > svn-src-head@freebsd.org mailing list
>> > https://lists.freebsd.org/mailman/listinfo/svn-src-head
>> > To unsubscribe, send any mail to "svn-src-head-unsubscribe@freebsd.org=
"



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAK7dMtBotsYmHwonsBP4CpFZ0DUF0FOyqd=R5qiz_mHaUKLHrg>