[coreboot] links in commit messages: don't do it.

ron minnich rminnich at gmail.com
Tue Oct 15 19:52:30 CEST 2013


I see a commit went in with this text:

"... have tested on Ubuntu 13.04 with AMD Catalyst 13.4 Proprietary
Linux Display Driver[1].

[1]. http://support.amd.com/us/gpudownload/linux/Pages/radeon_linux.aspx"

OK, if you click on that link you find it contains very little useful
information: it's a web form with four fields. It doesn't actually
point to the driver and it's not clear on first looking at it just how
I *would* get that driver. And, in time, it will become another dead
link in our growing pool of dead links.

So what information was added by this link? None. The text was more
than sufficient.

Please don't put links in commit messages. I would -2 this CL but it
already went in.

ron



More information about the coreboot mailing list