git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* git-apply and git-am should provide better errors when changelog is missing
@ 2007-07-23 17:15 Timur Tabi
  0 siblings, 0 replies; only message in thread
From: Timur Tabi @ 2007-07-23 17:15 UTC (permalink / raw)
  To: git

I was trying to apply a patch, and I kept getting this odd error:

I'm trying to apply a patch, and I get this error message:

git-apply --stat mpc86xx.patch
fatal: git-apply: bad git-diff - expected /dev/null on line 55

When I take a look at line 55 of mpc86xx.patch, I see this:

  # Yes MCA RS/6000s exist but Linux-PPC does not currently support any
  config MCA
diff --git a/arch/powerpc/boot/dts/mpc86xx.dts b/arch/powerpc/boot/dts/mpc86xx.dts$
new file mode 100644
index 0000000..a8a13b4
--- /dev/null					<--- line 55
+++ b/arch/powerpc/boot/dts/mpc86xx.dts
@@ -0,0 +1,187 @@
+/*
+ * MPC86xx Device Tree Source
+ *
+ * Copyright 2007 Freescale Semiconductor Inc.

So obviously, I was confused.  Well, I figured out the problem.  The patch did not include 
a changelog.  All it had was this:

 From b878a6b32a733e72a771fffd632f57b08808fc7f Mon Sep 17 00:00:00 2001
From: xxx <xxx@freescale.com>
Date: Tue, 19 Jun 2007 12:55:59 -0500
Subject: [PATCH] Added MPC86xx Support

Signed-off-by: xxx <xxx@freescale.com>
---
  arch/powerpc/Kconfig                        |    3 +-


Could someone please update the git-am and git-apply code to provide a better error 
message in this situation?  Thanks.

-- 
Timur Tabi
Linux Kernel Developer @ Freescale

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2007-07-23 17:15 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-07-23 17:15 git-apply and git-am should provide better errors when changelog is missing Timur Tabi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).