Attention is currently required from: fixeria.

pespin has posted comments on this change by fixeria. ( 
https://gerrit.osmocom.org/c/osmo-pcap/+/42838?usp=email )

Change subject: server: fix misleading data length validation log message
......................................................................


Patch Set 1:

(1 comment)

Commit Message:

https://gerrit.osmocom.org/c/osmo-pcap/+/42838/comment/28af5fd6_6757c58b?usp=email
 :
PS1, Line 14: Co-Authored-By: Claude Opus 4.8 (1M context) 
<[email protected]>
> It's not about advertising specific tools I used while writing these patches. 
> […]
Unless specific information is provided to showcase how exactly to reproduce 
finding the bug with such a tool in a deterministic way, I see no point in 
stating that kind of partial information which really only shows up as a free 
advertisement of a tool.


To me it's like adding on each commit: "I use Arch btw (TM)" because I found 
the bug using archlinux.

In any case, I'm not aware of any such requirements in this repo, the same way 
we don't require nor want a Signed-off-by in our commits. So I don't really see 
why should now adding a new line on each commit which really provides no direct 
use.



--
To view, visit https://gerrit.osmocom.org/c/osmo-pcap/+/42838?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: osmo-pcap
Gerrit-Branch: master
Gerrit-Change-Id: Ice95c1f8ad1aa8de259364bd70eba0db8918b19e
Gerrit-Change-Number: 42838
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Fri, 19 Jun 2026 20:52:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <[email protected]>
Comment-In-Reply-To: pespin <[email protected]>

Reply via email to