Skip to contentSkip to conversation

patman: use -D option for git format-patch

PATCH New Masahiro Yamada 3 participants 4 messages Aug 18, 2015 – Aug 22, 2015

lore
Share & tools
lore

Masahiro Yamada

author
#0 lore raw

PATCH 1/1 patman: use -D option for git format-patch

This allows Patman to generate smaller patches for file removal.

Signed-off-by Masahiro Yamada <[email protected]>

---

 tools/patman/gitutil.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
tools/patman/gitutil.py +1 -1
index 9e739d8..67b086b 100644
@@ -264,7 +264,7 @@ def CreatePatches(start, count, series):
     """
     if series.get('version'):
         version = '%s ' % series['version']
-    cmd = ['git', 'format-patch', '-M', '--signoff']
+    cmd = ['git', 'format-patch', '-D', '-M', '--signoff']
     if series.get('cover'):
         cmd.append('--cover-letter')
     prefix = series.GetPatchPrefix()
Signature · 1 line
1.9.1
#2 lore raw

PATCH 1/1 patman: use -D option for git format-patch

In reply to Masahiro Yamada

Hello Masahiro,

Am 18.08.2015 um 04:30 schrieb Masahiro Yamada:

Quoted 7 lines

This allows Patman to generate smaller patches for file removal.

Signed-off-by Masahiro Yamada <[email protected]>

---

  tools/patman/gitutil.py | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

Thanks!

Acked-by Heiko Schocher <[email protected]>

bye,
Heiko

Signature · 2 lines
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
#3 lore raw

PATCH 1/1 patman: use -D option for git format-patch

Hi,

Quoted 9 lines · Heiko Schocher

Hello Masahiro,

Am 18.08.2015 um 04:30 schrieb Masahiro Yamada:

Quoted 6 lines

This allows Patman to generate smaller patches for file removal.

Signed-off-by Masahiro Yamada <[email protected]>

---

Quoted 7 lines
Quoted 2 lines
  tools/patman/gitutil.py | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

Thanks!

Acked-by Heiko Schocher <[email protected]>

I'm picking this up now since I think the current behaviour is actually wrong.

Applied to u-boot-x86, thanks!