mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
To: Wolfgang Denk <wd@denx.de>
Cc: barebox@lists.infradead.org
Subject: Re: Code "borrowed" without attribution to original authors
Date: Thu, 7 Oct 2010 09:38:42 +0200	[thread overview]
Message-ID: <20101007073842.GP29673@pengutronix.de> (raw)
In-Reply-To: <20101006135644.8D7051539A0@gemini.denx.de>

Hello Wolfgang,

On Wed, Oct 06, 2010 at 03:56:44PM +0200, Wolfgang Denk wrote:
> From the commits as they are now, most uninformed readers would
> assume that the code was written by Jean-Christophe, which is not
> correct.
I really wonder why you care about the commit log.  I would care more
about the copyright statements in the code.  (Nevertheless I consider it
correct and fair to attribute in the commit log where the code is taken
from.)

I don't know if you want to make it easier for people to get the
annotations right/better when copying from U-Boot or not.  But if you
do, you could change your standard copyright header to include the
project's name.  Something like:

	/*
	 * (C) Copyright bla blub
	 *
	 * This file is part of das U-Boot, a versatile boot loader for
	 * many platforms.  U-Boot is free software ...
	 * ...
	 */

This doesn't help for the commit logs though.

> As I wrote before in my message to JC, I consider this an accident,
> i. e. rather ignorance than intention.  However, I would be seriously
> disappointed if such things should happen again.
I don't speak for barebox in general or Sascha or Pengutronix or
Jean-Christophe, but I think you cannot expect this to happen never ever
again.  I think the best you can achieve here is that the barebox
contributers get more aware of these issues and try to get it done
better in the future.

(Note, there is no offence intended in the whole mail.)

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

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

  reply	other threads:[~2010-10-07  7:38 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-05 13:52 Wolfgang Denk
2010-10-05 16:00 ` Jean-Christophe PLAGNIOL-VILLARD
2010-10-05 16:20   ` Jean-Christophe PLAGNIOL-VILLARD
2010-10-05 17:53     ` Wolfgang Denk
2010-10-06  3:03       ` Jean-Christophe PLAGNIOL-VILLARD
2010-10-06  7:27 ` Sascha Hauer
2010-10-06 13:56   ` Wolfgang Denk
2010-10-07  7:38     ` Uwe Kleine-König [this message]
     [not found]       ` <20101007080810.EFDC4153A7E@gemini.denx.de>
2010-10-07  8:37         ` Uwe Kleine-König
2010-10-07  8:06     ` Sascha Hauer
2010-10-07  8:45       ` Wolfgang Denk
2010-10-07 12:45         ` Sascha Hauer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20101007073842.GP29673@pengutronix.de \
    --to=u.kleine-koenig@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=wd@denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox