svn commit: r242625 - in head/sys: dev/ale dev/ata dev/ata/chipsets dev/ath/ath_hal/ar5212 dev/bge dev/cas dev/dc dev/flash dev/fxp dev/gem dev/lge dev/mii dev/nge dev/pci dev/re dev/sis dev/ste de...

[Peter Jeremy]

On 2012-Nov-05 23:45:00 +0100, Dimitry Andric <dim at FreeBSD.org> wrote:
>On 2012-11-05 23:36, Adrian Chadd wrote:
>> I'm fine with the change, but why didn't you run this by the various
>> driver owners first before doing a drive-by commit?
>
>The change was trivial, and did not cause any binary difference.  I see
>no reason to bother 20 different maintainers with something like this,
>except bureaucratic ones... :)

A bit belated but I think you've missed the most important reason why
you should warn maintainers: You can be creating additional effort for
maintainers.

There is an ongoing push to make head more stable by pushing
experimental development into "private" branches which can then be
merged back to head once the functionality is stable.  Whilst there
may not be a binary difference, you are creating source changes.  If
you've changed something that a maintainer is actively working on,
they need to take the time to work out if the change is still relevant
and whether it impacts the testing they have done.

-- 
Peter Jeremy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 196 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-src-head/attachments/20121117/4429c0c3/attachment.sig>