Re: [hatari-devel] HDC code update patch |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/hatari-devel Archives
]
- To: hatari-devel@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [hatari-devel] HDC code update patch
- From: Uwe Seimet <Uwe.Seimet@xxxxxxxxx>
- Date: Mon, 14 Mar 2022 20:14:37 +0100
- Authentication-results: strato.com; dkim=none
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1647285277; s=strato-dkim-0002; d=seimet.de; h=In-Reply-To:References:Message-ID:Subject:To:From:Date:Cc:Date:From: Subject:Sender; bh=pzl1/k4py+OfMhPgfAsWcqjh7vfZ+xlmvcBjY3gCDL0=; b=RW1KQhCtpiSVdO4TKp/I12jlPQNZvPokXSTtnslivVBHLJmokA43pDODjkhEz7GhEM PTKhsIHWv4cQqc6sLrn8IpoN4oebPiQ/mxDVpSvCVmAbWZ0twYxfcvQVVIDK3GahVSku df2HUzTpdW4BPcoKbOQSTCetR/9RzEv7jF/pDny7V5Eoc1ViVKbd3HtLhZdbiDY67v+V U4BtBTfTjSP9OT41T/i3UULXGpDR5x9QTSaf4630j5E2CZUVtfUjkV0joPU2gqxhY5y0 f4QOdVA6W6Z2nxK0pDcFR5869706gpzrOjCk0Wlxsb8eY1sgq76ONdEWPPkEmhWEOkO+ W96Q==
Hi again,
> Main reason for this is that I'd like your commit message to tell reason
> for the functional changes you did not mention:
> - else if (ctr->opcode == 0x05 || (ctr->opcode >= 0x20 && ctr->opcode <=
> 0x7d)) {
> + else if (ctr->opcode >= 0x20 && ctr->opcode <= 0x7d) {
> ...
> - else if (ctr->opcode >= 0x60)
> + else if (ctr->opcode >= 0x60 && ctr->opcode != HD_REPORT_LUNS)
>
> (If it e.g. fixes your earlier commit, it would be nice to give ID for
> the fixed commit in the commit message.)
Don't get me wrong, but let me just say that the recommended length of git
commit messages is 70-75 characters. https://i.stack.imgur.com/uzUqh.png
shows that for the Linux kernel the contributors mostly manage to stay
within this range.
If you mention the commit ID you refer to in your comment there is hardly
any space left for other information. I don't see this practice followed in
hardly any recent commit messages, by the way.
Again, don't get me wrong, but if what you pointed out is the Hatari commit
message policy, it is IMHO not very developer-friendly.
The way github handles this is quite good, because you can edit the
consolidated commit message in the web frontend before merging a PR, usually
with commit squashing enabled. Of course, one does not need github for this,
because squashing commits is a git feature.
I would like to hear other opinions on that.
Best regards
Uwe