git@vger.kernel.org list mirror (unofficial, one of many)
 help / color / mirror / code / Atom feed
* Bug With git rebase -p
@ 2018-03-19 20:23 Joseph Strauss
  2018-03-19 20:53 ` Junio C Hamano
  0 siblings, 1 reply; 8+ messages in thread
From: Joseph Strauss @ 2018-03-19 20:23 UTC (permalink / raw)
  To: git

I found the following erroneous behavior with "git rebase -p".

My current version is git version 2.16.2.windows.1

I made an example at GitHub, https://github.com/jkstrauss/git-bug/

There seem to be two problems when rebasing merge commits with git rebase -p :
  1. All lines of merge commits' messages get collapse into a single line.
  2. When an asterisk is present in the middle of the line it gets replaced with the file names of the current directory.

  


Joseph Kalman Strauss
Lifecycle Management Engineer
B&H Photo
212-239-7500 x2212
josephst@bhphoto.com

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

* Re: Bug With git rebase -p
  2018-03-19 20:23 Bug With git rebase -p Joseph Strauss
@ 2018-03-19 20:53 ` Junio C Hamano
  2018-03-20 14:52   ` Johannes Schindelin
  0 siblings, 1 reply; 8+ messages in thread
From: Junio C Hamano @ 2018-03-19 20:53 UTC (permalink / raw)
  To: Joseph Strauss; +Cc: git

Joseph Strauss <josephst@bhphoto.com> writes:

> I found the following erroneous behavior with "git rebase -p".
>
> My current version is git version 2.16.2.windows.1
>
> I made an example at GitHub, https://github.com/jkstrauss/git-bug/
>
> There seem to be two problems when rebasing merge commits with git rebase -p :
>   1. All lines of merge commits' messages get collapse into a single line.
>   2. When an asterisk is present in the middle of the line it gets replaced with the file names of the current directory.

I suspect that this has already been independently discovered
(twice) and corrected with

https://public-inbox.org/git/20180208204241.19324-1-gregory.herrero@oracle.com/

and is included in v2.17-rc0 (and later ;-).



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

* Re: Bug With git rebase -p
  2018-03-19 20:53 ` Junio C Hamano
@ 2018-03-20 14:52   ` Johannes Schindelin
  2018-03-20 15:18     ` Joseph Strauss
  0 siblings, 1 reply; 8+ messages in thread
From: Johannes Schindelin @ 2018-03-20 14:52 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: Joseph Strauss, git

Hi,

On Mon, 19 Mar 2018, Junio C Hamano wrote:

> Joseph Strauss <josephst@bhphoto.com> writes:
> 
> > I found the following erroneous behavior with "git rebase -p".
> >
> > My current version is git version 2.16.2.windows.1
> >
> > I made an example at GitHub, https://github.com/jkstrauss/git-bug/
> >
> > There seem to be two problems when rebasing merge commits with git rebase -p :
> >   1. All lines of merge commits' messages get collapse into a single line.
> >   2. When an asterisk is present in the middle of the line it gets replaced with the file names of the current directory.
> 
> I suspect that this has already been independently discovered
> (twice) and corrected with
> 
> https://public-inbox.org/git/20180208204241.19324-1-gregory.herrero@oracle.com/
> 
> and is included in v2.17-rc0 (and later ;-).

As it is included in v2.17.0-rc0, Joseph, could you verify that the
version at

https://github.com/git-for-windows/git/releases/tag/v2.17.0-rc0.windows.1

fixes this issue for you?

Thanks,
Johannes

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

* RE: Bug With git rebase -p
  2018-03-20 14:52   ` Johannes Schindelin
@ 2018-03-20 15:18     ` Joseph Strauss
  2018-03-21 22:33       ` Johannes Schindelin
  0 siblings, 1 reply; 8+ messages in thread
From: Joseph Strauss @ 2018-03-20 15:18 UTC (permalink / raw)
  To: Johannes Schindelin, Junio C Hamano; +Cc: git

Perfect. Thank you.

-----Original Message-----
From: Johannes Schindelin [mailto:Johannes.Schindelin@gmx.de] 
Sent: Tuesday, March 20, 2018 10:53 AM
To: Junio C Hamano <gitster@pobox.com>
Cc: Joseph Strauss <josephst@bhphoto.com>; git@vger.kernel.org
Subject: Re: Bug With git rebase -p

Hi,

On Mon, 19 Mar 2018, Junio C Hamano wrote:

> Joseph Strauss <josephst@bhphoto.com> writes:
> 
> > I found the following erroneous behavior with "git rebase -p".
> >
> > My current version is git version 2.16.2.windows.1
> >
> > I made an example at GitHub, https://github.com/jkstrauss/git-bug/
> >
> > There seem to be two problems when rebasing merge commits with git rebase -p :
> >   1. All lines of merge commits' messages get collapse into a single line.
> >   2. When an asterisk is present in the middle of the line it gets replaced with the file names of the current directory.
> 
> I suspect that this has already been independently discovered
> (twice) and corrected with
> 
> https://public-inbox.org/git/20180208204241.19324-1-gregory.herrero@or
> acle.com/
> 
> and is included in v2.17-rc0 (and later ;-).

As it is included in v2.17.0-rc0, Joseph, could you verify that the version at

https://github.com/git-for-windows/git/releases/tag/v2.17.0-rc0.windows.1

fixes this issue for you?

Thanks,
Johannes

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

* RE: Bug With git rebase -p
  2018-03-20 15:18     ` Joseph Strauss
@ 2018-03-21 22:33       ` Johannes Schindelin
  2018-03-22 16:38         ` Junio C Hamano
  0 siblings, 1 reply; 8+ messages in thread
From: Johannes Schindelin @ 2018-03-21 22:33 UTC (permalink / raw)
  To: Joseph Strauss; +Cc: Junio C Hamano, git

Hi Joseph,

On Tue, 20 Mar 2018, Joseph Strauss wrote:

> Perfect. Thank you.

You are welcome.

I am puzzled, though... does your message mean that you tested the Git for
Windows v2.17.0-rc0 installer and it did fix your problem? Or do you
simply assume that it does fix your problem because Junio & I expect it to
fix your problem?

Ciao,
Johannes

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

* Re: Bug With git rebase -p
  2018-03-21 22:33       ` Johannes Schindelin
@ 2018-03-22 16:38         ` Junio C Hamano
  2018-03-22 16:47           ` Joseph Strauss
  0 siblings, 1 reply; 8+ messages in thread
From: Junio C Hamano @ 2018-03-22 16:38 UTC (permalink / raw)
  To: Johannes Schindelin; +Cc: Joseph Strauss, git

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

> On Tue, 20 Mar 2018, Joseph Strauss wrote:
>
>> Perfect. Thank you.
>
> You are welcome.
>
> I am puzzled, though... does your message mean that you tested the Git for
> Windows v2.17.0-rc0 installer and it did fix your problem? Or do you
> simply assume that it does fix your problem because Junio & I expect it to
> fix your problem?

Thanks for asking, as I was curious about the same thing after
interpreting what Joseph said as "oh, perfect that there is a
packaged thing I can readily test" (implying "I'll get back to you
after seeing if it helps").

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

* RE: Bug With git rebase -p
  2018-03-22 16:38         ` Junio C Hamano
@ 2018-03-22 16:47           ` Joseph Strauss
  2018-03-23  9:39             ` Johannes Schindelin
  0 siblings, 1 reply; 8+ messages in thread
From: Joseph Strauss @ 2018-03-22 16:47 UTC (permalink / raw)
  To: Junio C Hamano, Johannes Schindelin; +Cc: git

I meant to say that I installed 2.17.0-rc0, and it worked perfectly. Sorry for the ambiguity.

-----Original Message-----
From: Junio C Hamano [mailto:jch2355@gmail.com] On Behalf Of Junio C Hamano
Sent: Thursday, March 22, 2018 12:39 PM
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: Joseph Strauss <josephst@bhphoto.com>; git@vger.kernel.org
Subject: Re: Bug With git rebase -p

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

> On Tue, 20 Mar 2018, Joseph Strauss wrote:
>
>> Perfect. Thank you.
>
> You are welcome.
>
> I am puzzled, though... does your message mean that you tested the Git 
> for Windows v2.17.0-rc0 installer and it did fix your problem? Or do 
> you simply assume that it does fix your problem because Junio & I 
> expect it to fix your problem?

Thanks for asking, as I was curious about the same thing after interpreting what Joseph said as "oh, perfect that there is a packaged thing I can readily test" (implying "I'll get back to you after seeing if it helps").

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

* RE: Bug With git rebase -p
  2018-03-22 16:47           ` Joseph Strauss
@ 2018-03-23  9:39             ` Johannes Schindelin
  0 siblings, 0 replies; 8+ messages in thread
From: Johannes Schindelin @ 2018-03-23  9:39 UTC (permalink / raw)
  To: Joseph Strauss; +Cc: Junio C Hamano, git

Hi Joseph,

On Thu, 22 Mar 2018, Joseph Strauss wrote:

> I meant to say that I installed 2.17.0-rc0, and it worked perfectly.

Thank you for testing.

Ciao,
Johannes

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

end of thread, other threads:[~2018-03-23  9:39 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-19 20:23 Bug With git rebase -p Joseph Strauss
2018-03-19 20:53 ` Junio C Hamano
2018-03-20 14:52   ` Johannes Schindelin
2018-03-20 15:18     ` Joseph Strauss
2018-03-21 22:33       ` Johannes Schindelin
2018-03-22 16:38         ` Junio C Hamano
2018-03-22 16:47           ` Joseph Strauss
2018-03-23  9:39             ` Johannes Schindelin

git@vger.kernel.org list mirror (unofficial, one of many)

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://public-inbox.org/git
	git clone --mirror http://ou63pmih66umazou.onion/git
	git clone --mirror http://czquwvybam4bgbro.onion/git
	git clone --mirror http://hjrcffqmbrq6wope.onion/git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V1 git git/ https://public-inbox.org/git \
		git@vger.kernel.org
	public-inbox-index git

Example config snippet for mirrors.
Newsgroups are available over NNTP:
	nntp://news.public-inbox.org/inbox.comp.version-control.git
	nntp://7fh6tueqddpjyxjmgtdiueylzoqt6pt7hec3pukyptlmohoowvhde4yd.onion/inbox.comp.version-control.git
	nntp://ie5yzdi7fg72h7s4sdcztq5evakq23rdt33mfyfcddc5u3ndnw24ogqd.onion/inbox.comp.version-control.git
	nntp://4uok3hntl7oi7b4uf4rtfwefqeexfzil2w6kgk2jn5z2f764irre7byd.onion/inbox.comp.version-control.git
	nntp://news.gmane.io/gmane.comp.version-control.git
 note: .onion URLs require Tor: https://www.torproject.org/

code repositories for project(s) associated with this inbox:

	https://80x24.org/mirrors/git.git

AGPL code for this site: git clone https://public-inbox.org/public-inbox.git