svn commit: r317145 - in stable/11/sys: boot/mips/uboot conf

[Ed Maste]

On 19 April 2017 at 12:24, Rodney W. Grimes
<freebsd at pdx.rh.cn85.dnsmgr.net> wrote:
>>
>>   The linker script CONSTRUCTORS keyword is only meaningful "when linking
>>   object file formats which do not support arbitrary sections, such as
>>   ECOFF and XCOFF"[1] and is ignored for other object file formats.
>>
>>   LLVM's lld does not yet accept (and ignore) CONSTRUCTORS, so just remove
>>   CONSTRUCTORS from the linker script as it has no effect.
>>
>>   [1] https://sourceware.org/binutils/docs/ld/Output-Section-Keywords.html
>
> ^^^^^^^^^^^^ links to external data is probably not a good idea in
> commit messages as it tends to disappear in time.

I understand your point, but think it should be phrased differently:
commit messages should stand alone. Some projects use a commit message
style that consists of little more than "Fixes bug #1723" or
"rdar://90210", and I agree that is insufficient.

In the quoted commit message above the relevant content from the
binutils documentation is quoted inline, and the URL acts as a
reference for the citation. I'd argue that a citation without the URL
is less useful today, and not really any worse in the case that
sourceware.org disappears, given the existence of the Internet Archive
and search engines that could locate the content based on the quoted
content and/or portions of the URL.