On Sat, 25 Jun 2016, John Crispin wrote:



On 25/06/2016 10:47, Zoltan HERPAI wrote:


On Sat, 25 Jun 2016, Mathias Kresin wrote:

Hi Zoltan,

I've noticed that you have imported a lot of patches from the LEDE
repo into the OpenWrt github repo.

I'm concerted about the fact that you have omitted at least a few of
my Signed-off-bys[1][2], so that it isn't any longer noticeable that
the changes are my work. Even though it may not be legally
problematic, but from my point of view this is unnecessarily rude.

But where I'm getting really pissed is if someone removes/omits my
copyright line as it was done in [3] vs. [4]. You might know how much
work it is to disassemble a bootloader and find what you are looking
for in the assembler code. Albeit the patch changes only two lines, it
wasn't a 5 minute job. Same as above, might be legally fine but it's
rude.

Would you be please so kind to keep an eye on this in the future and
fix at least my patches in the OpenWrt github repo!

Mathias


[1]
https://github.com/openwrt/openwrt/commit/1ea639b8d081bf7ace93e1c4e130615d1684ad5e

[2]
https://github.com/openwrt/openwrt/commit/e71983fca5fc9eed5e73a33e678ffcf2a2f9a8e9

[3]
https://github.com/openwrt/openwrt/commit/e496316324925a978037f298c7b978ef97e13e7e

[4]
https://git.lede-project.org/?p=source.git;a=blobdiff;f=package/boot/uboot-lantiq/patches/0112-MIPS-add-board-support-for-Arcadyan-VGV7510KW22.patch;h=ccc3505a968edec70d334d68bc46802afd696da6;hp=e46d374dc0e9791c43bcaffee6a86a48a0c19a93;hb=a22feb4c78f51716772738af84180d58bd877c45;hpb=382282eca922e106f5b01597e4676bab8c7176db



Absolutely happy to update the commit(s) - throughout the resync in a
few cases, patches and file changes had to be pulled together because of
file renames and dropped earlier kernel supports which we didn't want to
drop, thus the patches didn't apply anymore at all. Some examples of
this was also seen in the LEDE tree (not more than a few I agree),
having said that the signed-off-by issue was an oversight in this case
which I'll fix, and thanks for bringing attention to this.

Thanks,
Zoltan H

Hi Zoltan,

this one aswell please

https://github.com/openwrt/openwrt/commit/15458e85bc42672eb06dee5fb45597298b4a7162

Ack John, thanks.

Regards,
Zoltan H
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to