Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 07 Feb 2025 10:53:04 -0800
From:      Cy Schubert <Cy.Schubert@cschubert.com>
To:        Mark Johnston <markj@FreeBSD.org>
Cc:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   Re: git: d3b1dcb408e2 - main - acpi_iicbus: install the address  space handler by default
Message-ID:  <20250207185304.260B5225@slippy.cwsent.com>
In-Reply-To: <202502071842.517IgxJE051814@gitrepo.freebsd.org>
References:  <202502071842.517IgxJE051814@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
In message <202502071842.517IgxJE051814@gitrepo.freebsd.org>, Mark Johnston 
wri
tes:
> The branch main has been updated by markj:
>
> URL: https://cgit.FreeBSD.org/src/commit/?id=d3b1dcb408e2f63845aee5508fe4c73d
> b1f94b58
>
> commit d3b1dcb408e2f63845aee5508fe4c73db1f94b58
> Author:     Ahmad Khalifa <ahmadkhalifa570@gmail.com>
> AuthorDate: 2025-01-26 18:48:17 +0000
> Commit:     Mark Johnston <markj@FreeBSD.org>
> CommitDate: 2025-02-07 18:20:00 +0000
>
>     acpi_iicbus: install the address space handler by default
>     
>     Signed-off-by: Ahmad Khalifa <ahmadkhalifa570@gmail.com>
>     
>     Reviewed by:    wulf
>     Pull Request:   https://github.com/freebsd/freebsd-src/pull/1583

The question why remains to be answered.

We should try to teach contributors to answer the question why in their 
commit log messages. I think most committers do a pretty good job of that. 
We should expect contributors, who may potentially become committers, to 
answer this question too in their commits.


-- 
Cheers,
Cy Schubert <Cy.Schubert@cschubert.com>
FreeBSD UNIX:  <cy@FreeBSD.org>   Web:  https://FreeBSD.org
NTP:           <cy@nwtime.org>    Web:  https://nwtime.org

			e^(i*pi)+1=0





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