All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [GIT PULL] Please pull u-boot-arm
Date: Fri, 11 Apr 2008 13:55:37 +0200	[thread overview]
Message-ID: <200804111355.37988.sr@denx.de> (raw)
In-Reply-To: <20080411112752.2347D248A8@gemini.denx.de>

On Friday 11 April 2008, Wolfgang Denk wrote:
> In message <Pine.LNX.4.64.0804111309060.15272@axis700.grange> you wrote:
> > Sorry, do I understand it right, that you are going to keep your patches
> > as you have committed them and now apply this patch from Magnus on the
> > top? If yes, I don't think this is a correct decision. I think, you
> > should revert your commits and re-apply the original unchanged patches.
>
> I agree with Guennadi. The only solution is to throw away the current
> state of the repository, and restart from the state  before  all  the
> corrupted patches were applied, adding clean, unmodified patches this
> time.

ACK. Yes, please make a fresh start.

> As for the public U-Boot repository, where such a manipulation cannot
> be done easily, I think I would like to revert the commits in
> question, i. e. this list of commits:
>
> 	066bebd6353e33af3adefc3404560871699e9961
> 	7a837b7310166ae8fc8b8d66d7ef01b60a80f9d6
> 	c88ae20580b2b01487b4cdcc8b2a113f551aee36
> 	a147e56f03871bba4f05058d5e04ce7deb010b04
> 	d6674e0e2a6a1f033945f78838566210d3f28c95
> 	8c8463cce44d849e37744749b32d38e1dfb12e50
> 	c98b47ad24b2d91f41c09a3d62d7f70ad84f4b7d
> 	8bf69d81782619187933a605f1a95ee1d069478d
> 	8c16cb0d3b971f46fbe77c072664c0f2dcd4471d
> 	a574a73852a527779234e73e17e7597fd8128882
> 	1377b5583a48021d983e1fd565f7d40c89e84d63
> 	1704dc20917b4f71e373e2c888497ee666d40380
>
> Comments welcome.

I agree that all this should be reverted and applied again in a clean way.

Just my 2 cents.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

  parent reply	other threads:[~2008-04-11 11:55 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-30 11:04 [U-Boot-Users] [GIT PULL] Please pull u-boot-arm Peter Pearse
2008-04-07 22:06 ` Wolfgang Denk
2008-04-08 23:32   ` [U-Boot-Users] Working config to compare? pat
2008-04-10 10:24 ` [U-Boot-Users] [GIT PULL] Please pull u-boot-arm Wolfgang Denk
2008-04-10 11:52   ` Magnus Lilja
2008-04-11  9:04     ` Peter Pearse
2008-04-11 11:11       ` Guennadi Liakhovetski
2008-04-11 11:27         ` Wolfgang Denk
2008-04-11 11:38           ` Guennadi Liakhovetski
2008-04-11 12:32             ` Peter Pearse
2008-04-11 12:40               ` Philip Balister
2008-04-11 12:51                 ` Stefan Roese
2008-04-11 12:53               ` Guennadi Liakhovetski
2008-04-11 13:13               ` Jerry Van Baren
2008-04-11 13:17               ` Wolfgang Denk
2008-04-11 13:29                 ` Peter Pearse
2008-04-17 14:22                   ` Dirk Behme
2008-04-11 11:55           ` Stefan Roese [this message]
2008-04-11 11:24       ` Wolfgang Denk

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=200804111355.37988.sr@denx.de \
    --to=sr@denx.de \
    --cc=u-boot@lists.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.