Re: [hatari-devel] SCSI Driver logging improvement |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/hatari-devel Archives
]
- To: hatari-devel@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [hatari-devel] SCSI Driver logging improvement
- From: Uwe Seimet <Uwe.Seimet@xxxxxxxxx>
- Date: Fri, 29 Nov 2024 23:12:02 +0100
- Arc-authentication-results: i=1; strato.com; arc=none; dkim=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; t=1732918339; s=strato-dkim-0002; d=strato.com; h=In-Reply-To:References:Message-ID:Subject:To:From:Date:Cc:Date:From: Subject:Sender; bh=Cy+YXGGOju3JhUgTY87zQbFbvUN/KLyKGi3DjMhXfmE=; b=Y4Z9zNUI/BaWnaQKb9ms4oefO4RGxBGm4DBzC3S6qWmvlQO1ptJBgnZRkqw7Z9dFdh gbfFvSrLiEiId93Pt8w5VbN6s3QNm7dBxSaxwuC6ciDT5n6Dd5bKFA3B6oEEIo3y0/Me kDY1XJKwMTOwIgquPZp4l+6p7W21dVuF+BHYI5HY1R4VnXhSEa3+IYtTUOJtl0fh8Q+C CGuq3WblMG04MO69t5BSrknTIrL2xaQE++P/sEsppp1G3c0z2onFLC6z6Rs973q3b0WR PJSVuRdl94ShEfxxnJ6h+pHVaBNXJivI0r/UjkhHK50Ud5NoPLmI27+nAtivq5YR8b1X o7qQ==
- Arc-seal: i=1; a=rsa-sha256; t=1732918339; cv=none; d=strato.com; s=strato-dkim-0002; b=jlQJZTpO8zz8iwNANO4AcAjDxOjuL5YreJNOfMvJ2d59kVl7XiYySAb/t6hS+U/W0b G9jCXf2S5XgL5aMFUY6n74WA4Hrbnd7j6Y5TqjLT3rUKlZ+S5fkRljCXkk6YjZyBezPg jVqvVHgrq78O5GEJjLljnPXlH7xztazAE7SHc7JPpDQGcAi90P2GFozSdFtb300rf1iV jpoaJ9oL+/hfE8OKJ68i9Nm0SABxG3gN8+OydJH4paQHgD1Awd6LdnmSR8ykdPWju6Gg Zab1s1COu8UoABKAkyZPnN4UrdGCOgFBAySaPDUirur0tyv87NoBM3OHe/vfWxyr4IMq R7fw==
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1732918339; s=strato-dkim-0002; d=seimet.de; h=In-Reply-To:References:Message-ID:Subject:To:From:Date:Cc:Date:From: Subject:Sender; bh=Cy+YXGGOju3JhUgTY87zQbFbvUN/KLyKGi3DjMhXfmE=; b=ljAFFh9Bp1NqE9OPYdQIc/2fhOYvCbG4k1xvfGrQN9FbzCM/6G2Ini3BDg0V2dz3A3 5CDfzAGoj1+fnE/m5C5DYullU22fPjdFlwF5BQdsmMqeOuxsFFtwPiSyon2oKf7ni/0x +1oqeLXXzsh4BJPL+bKs0THfjQwd25/11JtYXcwltOyYOoN/HyfoRcxtSB6rSCAItIpu cznj705UAbAFMJF4e0HIid/wZmQV6vu9yjKS03+LmHA7rhnOQ/q1p1+cx2nofyQ3YNVJ JvczAPaLg6Hxc1umTZuWL28qHex0yP8QuCX6RLuB4RSC2Y7n62HbeBU0Oge1N4+oYMWl rGYQ==
- Dkim-signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; t=1732918339; s=strato-dkim-0003; d=seimet.de; h=In-Reply-To:References:Message-ID:Subject:To:From:Date:Cc:Date:From: Subject:Sender; bh=Cy+YXGGOju3JhUgTY87zQbFbvUN/KLyKGi3DjMhXfmE=; b=H0FrM8LlmkpRu6DDAI4b1pUF0FULla2TJOGGiAT1+DrtRFFpn40njk5GBCTWhGhkhs 4FkIFdcTxMnXOa+rXnDg==
Hi Eero,
Thank you for elaborating on this.
> Logged messages should state from which subsystem they come from. E.g.
> consistent subsystem prefix makes things easier to filter & read.
I see. Can you please add the missing prefix?
> > @Eero You recommended to use git format last time. I did, but I don't see
> > any advantage. Neither my name nor my comment are part of the patch. Can you
> > please be more precise on the options I should use, and how I can create the
> > required format without having to locally commit my changes first?
>
> You need to do commit(s) first.
But I don't want to commit these changes to my local repository, because
they are just temporary. After the Hatari team has committed them I cannot
simply pull the changes anymore. Unless I create a throwaway branch for
this, which is a bit inconvenient, because this branch is not even used for
creating a PR. I just noticed that you also suggest this approach in your
response.
> If your name/email are missing from ~/.gitconfig, git tells (command) to
> add those first.
No they are not missing from .gitconfig. I wonder why the format generated
with git format is better for you. You still have to extract my commit
comment and my name from the email, don't you?
Best regards
Uwe