git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Remove unused t/t8005/iso8859-5.txt file
@ 2009-08-29  7:16 Nanako Shiraishi, しらいし ななこ
  2009-08-29  7:48 ` Junio C Hamano
  0 siblings, 1 reply; 4+ messages in thread
From: Nanako Shiraishi, しらいし ななこ @ 2009-08-29  7:16 UTC (permalink / raw)
  To: git

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=UTF-8, Size: 676 bytes --]

This file is no longer used since commit 54bc13c.

Signed-off-by: ’¤·’¤é’¤¤’¤· ’¤Ê’¤Ê’¤³ <nanako3@lavabit.com>
---
 t/t8005/iso8859-5.txt |  Bin 74 -> 0 bytes
 1 files changed, 0 insertions(+), 0 deletions(-)
 delete mode 100644 t/t8005/iso8859-5.txt

diff --git a/t/t8005/iso8859-5.txt b/t/t8005/iso8859-5.txt
deleted file mode 100644
index 2e4b80c8df4da30722561049c46cca778e49cd2f..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 74
zcmeYa_P4MwwTw57_jB}hwN=`2>B3!w{Z}77xOeHsbA^L9uG|B%l(;<M%6x;}ZIupP
Zefa3!rF&Nu9^Sim@#WRKH?Asi0RTCvCqV!J

-- 
1.6.4.1

-- 
Nanako Shiraishi
http://ivory.ap.teacup.com/nanako3/

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Remove unused t/t8005/iso8859-5.txt file
  2009-08-29  7:16 [PATCH] Remove unused t/t8005/iso8859-5.txt file Nanako Shiraishi, しらいし ななこ
@ 2009-08-29  7:48 ` Junio C Hamano
  2009-08-29  8:08   ` Nanako Shiraishi
  0 siblings, 1 reply; 4+ messages in thread
From: Junio C Hamano @ 2009-08-29  7:48 UTC (permalink / raw)
  To: Nanako Shiraishi; +Cc: git

Nanako Shiraishi <nanako3@lavabit.com> writes:

> This file is no longer used since commit 54bc13c.

Hmm, the patch is grossly encoding damaged, so I'll hand apply.
Out of curiosity, how did you find this?

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Remove unused t/t8005/iso8859-5.txt file
  2009-08-29  7:48 ` Junio C Hamano
@ 2009-08-29  8:08   ` Nanako Shiraishi
  2009-08-30 20:14     ` Junio C Hamano
  0 siblings, 1 reply; 4+ messages in thread
From: Nanako Shiraishi @ 2009-08-29  8:08 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

Quoting Junio C Hamano <gitster@pobox.com>

> Out of curiosity, how did you find this?

Soon after you posted your original in http://thread.gmane.org/gmane.comp.version-control.git/121881, I applied it to my tree, and I have been running 'git pull --rebase' for ever. I noticed that rebase didn't remove the commit, even after your 54bc13c.

Today I found that the rebased patch only removed that file. The real question is how you lost the deletion of the file when you applied. What happened?

-- 
Nanako Shiraishi
http://ivory.ap.teacup.com/nanako3/

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Remove unused t/t8005/iso8859-5.txt file
  2009-08-29  8:08   ` Nanako Shiraishi
@ 2009-08-30 20:14     ` Junio C Hamano
  0 siblings, 0 replies; 4+ messages in thread
From: Junio C Hamano @ 2009-08-30 20:14 UTC (permalink / raw)
  To: Nanako Shiraishi; +Cc: git

Nanako Shiraishi <nanako3@lavabit.com> writes:

> Quoting Junio C Hamano <gitster@pobox.com>
>
>> Out of curiosity, how did you find this?
>
> Soon after you posted your original in
> http://thread.gmane.org/gmane.comp.version-control.git/121881, I applied
> it to my tree, and I have been running 'git pull --rebase' for ever. I
> noticed that rebase didn't remove the commit, even after your 54bc13c.
>
> Today I found that the rebased patch only removed that file. The real
> question is how you lost the deletion of the file when you applied. What
> happened?

My stupidity, and a little bit of danger in trusting "am -3" blindly.

The patch sent to the list was against the master, but the actual commit
created, 54bc13c, was on maint.  Back then, there was a commit in master
but not in maint that stopped using cp1251.txt and started using
iso8859-5.txt instead.

Because maint did not have iso8859-5.txt (which the patch wanted to
remove), when "am -3" did its fake 3-way merge, the removal of the path
did not even conflict.  It was the case of "neither side wants to have
this file".  As the result, cp1251.txt was left in the tree (which is
half-Ok, even though the updated test never looked at it anymore), and
merging the result back to master did not touch iso8859-5.txt.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2009-08-30 20:14 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-08-29  7:16 [PATCH] Remove unused t/t8005/iso8859-5.txt file Nanako Shiraishi, しらいし ななこ
2009-08-29  7:48 ` Junio C Hamano
2009-08-29  8:08   ` Nanako Shiraishi
2009-08-30 20:14     ` Junio C Hamano

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).