lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 10 Dec 2018 00:27:22 +0100 From: Sabrina Dubroca <sd@...asysnail.net> To: Andrew Lunn <andrew@...n.ch> Cc: Heiner Kallweit <hkallweit1@...il.com>, David Miller <davem@...emloft.net>, Stephen Rothwell <sfr@...b.auug.org.au>, Networking <netdev@...r.kernel.org>, Linux Next Mailing List <linux-next@...r.kernel.org>, Linux Kernel Mailing List <linux-kernel@...r.kernel.org>, Florian Fainelli <f.fainelli@...il.com> Subject: Re: linux-next: Signed-off-by missing for commits in the net-next tree 2018-12-09, 22:49:07 +0100, Andrew Lunn wrote: > On Sun, Dec 09, 2018 at 10:33:10PM +0100, Heiner Kallweit wrote: > > On 09.12.2018 22:11, Andrew Lunn wrote: > > > On Mon, Dec 10, 2018 at 08:00:45AM +1100, Stephen Rothwell wrote: > > >> Hi all, > > >> > > >> Commits > > >> > > >> dc9d38cec71c ("net: phy: mdio-gpio: Add phy_ignore_ta_mask to platform data") > > >> 04fa26bab06d ("net: phy: mdio-gpio: Add platform_data support for phy_mask") > > >> > > >> are missing a Signed-off-by from their author. > > > > > > Hi David. > > > > > > Any idea what happened here? The version in my git repo has SOB. > > > > > > https://patchwork.ozlabs.org/patch/1009811/ also has my SOB. > > > > > Instead of > > v2 > > -- > > int -> u32 in platform data structure > > > > Signed-off-by: Andrew Lunn <andrew@...n.ch> > > > > shouldn't it be > > > > Signed-off-by: Andrew Lunn <andrew@...n.ch> > > --- > > v2 > > - int -> u32 in platform data structure > > Hi Heiner > > David said he wanted to see the version history. So i deliberately put > it above the ---. > > I'm just wondering if -- was enough to trigger something in David's > scripts? Or patchwork. The -- has disappeared in the commit which made > it into net-next. > > Andrew If you fetch the mbox from patchwork at https://patchwork.ozlabs.org/patch/1009811/mbox/ it contains: -------- 8< -------- v2 Reviewed-by: Florian Fainelli <f.fainelli@...il.com> --- int -> u32 in platform data structure Signed-off-by: Andrew Lunn <andrew@...n.ch> --- [diffstat] -------- 8< -------- That's 3 dashes instead of the 2 from your mail. If you "git am" that, the sign-off and actual history will get chopped off. Maybe Stephen's script to detect those missing sign-offs could be run as a commit/apply hook by David? This happens regularly, sometimes dropping more than just a sign-off. -- Sabrina
Powered by blists - more mailing lists