Re: Code "borrowed" without attribution to original authors

[Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>]

On 19:53 Tue 05 Oct     , Wolfgang Denk wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
> 
> In message <20101005162019.GK25320@game.jcrosoft.org> you wrote:
> >
> > > >  d424ce7  image: factorise image printing contents
> > > >  0ceafe1  Replace direct header access with the API routines
> > 
> > > >  aba80a2  image: rename IH_CPU to IH_ARCH to be more concistant
> > > >  7bd7d59  image: factorise string helper
> > for the record the two are not borrowed at all from U-Boot but done based on
> > barebox
> 
> Oh. Interesting statement. Let me check again. 
> 
> Your commit:
> 
> 	commit aba80a2d2d0f1b25185246925577ce2108247dde
> 	Author: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> 	Date:   Thu Sep 23 06:48:42 2010 +0200
> 
> 	    image: rename IH_CPU to IH_ARCH to be more concistant
> 
> So BB used to have IH_CPU and switched to IH_ARCH :
> 
> Please compare against this U-Boot commit:
> 
> 	commit b97a2a0a21f279d66de8a9bdbfe21920968bcb1c
> 	Author: Marian Balakowicz <m8@semihalf.com>
> 	Date:   Tue Jan 8 18:14:09 2008 +0100
> 
> 	    [new uImage] Define a API for image handling operations
> 	    
> 	    - Add inline helper macros for basic header processing
> 	    - Move common non inline code common/image.c
> 	    - Replace direct header access with the API routines
> 	    - Rename IH_CPU_* to IH_ARCH_*
> 	    
> 	    Signed-off-by: Marian Balakowicz <m8@semihalf.com>
> 
> Pay special attention to the "Rename IH_CPU_* to IH_ARCH_*" part.
> 
> Not borrowed?  What an extremely unlikely coincidence then!  I think
> you should try to explain that.
> 
> > and as we all known the uImage format come from U-Boot I just sync it
> > recently without commit details as we do for Kconfig we do not trac
> > modifications details and we respect the copyright. SO I'll repeat myself
> > if Marian copyright is missing We will add it with pleasure.
> 
> 
> I recommend you to get a clue about what is needed to properly
> attribute code to the people who actually wrote it, instead of
> submitting it without any due reference and then even claiming you
> developed it yourself.  I guess you are not interested in having a
> reputation as a plagiarist?
I'll repeat I'ev not borrow these two they look similar I agree but I do not
borrow them. And I'm fine to put Marian copyright I eally do not carefor the
other 2 commits. But these two I just had the same idea by looking at the
code of barebox when I was working to re-introduce the mutli-image support.

The two other commits are here for sync before finish the full sync of uImage
and I will not take Marina work as mine. I'll put my SoB as I apply it and
responsible for this integration as I did for kconfig and others without take
the copyright.

Best Regards,
J.

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox