{"thread":{"id":"51575","subject":"bad patch created when I use --ignore-blank-lines option","startedAt":"2019-08-02T15:40:00Z","lastAt":"2019-08-02T15:58:19Z","messageCount":2,"participants":["Eugen Konkov"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"379831","messageId":"1543205721.20190802183954@yandex.ru","threadId":"51575","inReplyTo":null,"subject":"bad patch created when I use --ignore-blank-lines option","fromName":"Eugen Konkov","fromEmail":"kes-kes@yandex.ru","sentAt":"2019-08-02T15:39:54Z","receivedAt":"2019-08-02T15:40:00Z","isPatch":false,"sender":{"key":"kes-kes@yandex.ru","avatar":null},"body":"Hello Git,\n\nI add and remove only blank lines at my file:\n\n$ git log -p t/29-cmd_l.t\n\n--- a/t/29-cmd_l.t\n+++ b/t/29-cmd_l.t\n@@ -218,7 +218,6 @@ __DATA__\n @@ list\n -e:0005  1;    \n -e:0002    2;\n-\n -e\n     0: use Devel::DbBatch split(/,/,q{list.conf;b 2;a 2 1;s 2;l .;q});;\n     1: sub t {\n@@ -226,26 +225,26 @@ ab>>2:     2;\n     3: }\n     4:\n    x5: 1;\n+\n @@ list from first\n -e:0010  t2();\n-\n -e\n     0: use Devel::DbBatch split(/,/,q{list.conf;l 0;q});;\n     1: sub t0 {\n    x2:     1;\n     3: }\n+\n @@ list from last\n -e:0010  t2();\n-\n -e\n     14:\n     15:\n     16:\n     17: # Perl implicitly adds one new line after this one\n     18:\n+\n @@ list middle\n -e:0010  t2();\n-\n -e\n    x5:     t0();\n     6: }\n\n\n\nBut  when I use --ignore-blank-lines option I see broken patch. I have\nnot added '-e' line, but it is displayed as it was added. Notice '+-e'\n\n$ git log -p --ignore-blank-lines t/29-cmd_l.t\n\ndiff --git a/t/29-cmd_l.t b/t/29-cmd_l.t\nindex 4b77279..d7739d2 100755\n--- a/t/29-cmd_l.t\n+++ b/t/29-cmd_l.t\n@@ -219,6 +219,7 @@ __DATA__\n -e:0005  1;    \n -e:0002    2;\n \n+-e\n     0: use Devel::DbBatch split(/,/,q{list.conf;b 2;a 2 1;s 2;l .;q});;\n     1: sub t {\n ab>>2:     2;\n@@ -228,6 +229,7 @@ ab>>2:     2;\n @@ list from first\n -e:0010  t2();\n \n+-e\n     0: use Devel::DbBatch split(/,/,q{list.conf;l 0;q});;\n     1: sub t0 {\n    x2:     1;\n@@ -235,6 +237,7 @@ ab>>2:     2;\n @@ list from last\n -e:0010  t2();\n \n+-e\n     14:\n     15:\n     16:\n@@ -243,6 +246,7 @@ ab>>2:     2;\n @@ list middle\n -e:0010  t2();\n \n+-e\n    x5:     t0();\n     6: }\n     7: sub t2 {\n\n\n$ git --version\ngit version 2.22.0\n\n\n-- \nBest regards,\nEugen Konkov\n\n"},{"id":"379832","messageId":"56155941.20190802185256@yandex.ru","threadId":"51575","inReplyTo":"1543205721.20190802183954@yandex.ru","subject":"Re: bad patch created when I use --ignore-blank-lines option","fromName":"Eugen Konkov","fromEmail":"kes-kes@yandex.ru","sentAt":"2019-08-02T15:52:56Z","receivedAt":"2019-08-02T15:58:19Z","isPatch":false,"sender":{"key":"kes-kes@yandex.ru","avatar":null},"body":"Hello Eugen,\n\nsorry, everything is OK. rafasc suggested me what is the problem.\nI should use --full-history \n\n\nFriday, August 2, 2019, 6:39:54 PM, you wrote:\n\n> Hello Git,\n\n> I add and remove only blank lines at my file:\n\n> $ git log -p t/29-cmd_l.t\n\n> --- a/t/29-cmd_l.t\n> +++ b/t/29-cmd_l.t\n> @@ -218,7 +218,6 @@ __DATA__\n>  @@ list\n>  -e:0005  1;    \n>  -e:0002    2;\n> -\n>  -e\n>      0: use Devel::DbBatch split(/,/,q{list.conf;b 2;a 2 1;s 2;l .;q});;\n>      1: sub t {\n> @@ -226,26 +225,26 @@ ab>>2:     2;\n>      3: }\n>      4:\n>     x5: 1;\n> +\n>  @@ list from first\n>  -e:0010  t2();\n> -\n>  -e\n>      0: use Devel::DbBatch split(/,/,q{list.conf;l 0;q});;\n>      1: sub t0 {\n>     x2:     1;\n>      3: }\n> +\n>  @@ list from last\n>  -e:0010  t2();\n> -\n>  -e\n>      14:\n>      15:\n>      16:\n>      17: # Perl implicitly adds one new line after this one\n>      18:\n> +\n>  @@ list middle\n>  -e:0010  t2();\n> -\n>  -e\n>     x5:     t0();\n>      6: }\n\n\n\n> But  when I use --ignore-blank-lines option I see broken patch. I have\n> not added '-e' line, but it is displayed as it was added. Notice '+-e'\n\n> $ git log -p --ignore-blank-lines t/29-cmd_l.t\n\n> diff --git a/t/29-cmd_l.t b/t/29-cmd_l.t\n> index 4b77279..d7739d2 100755\n> --- a/t/29-cmd_l.t\n> +++ b/t/29-cmd_l.t\n> @@ -219,6 +219,7 @@ __DATA__\n>  -e:0005  1;    \n>  -e:0002    2;\n>  \n> +-e\n>      0: use Devel::DbBatch split(/,/,q{list.conf;b 2;a 2 1;s 2;l .;q});;\n>      1: sub t {\n ab>>>2:     2;\n> @@ -228,6 +229,7 @@ ab>>2:     2;\n>  @@ list from first\n>  -e:0010  t2();\n>  \n> +-e\n>      0: use Devel::DbBatch split(/,/,q{list.conf;l 0;q});;\n>      1: sub t0 {\n>     x2:     1;\n> @@ -235,6 +237,7 @@ ab>>2:     2;\n>  @@ list from last\n>  -e:0010  t2();\n>  \n> +-e\n>      14:\n>      15:\n>      16:\n> @@ -243,6 +246,7 @@ ab>>2:     2;\n>  @@ list middle\n>  -e:0010  t2();\n>  \n> +-e\n>     x5:     t0();\n>      6: }\n>      7: sub t2 {\n\n\n> $ git --version\n> git version 2.22.0\n\n\n\n\n\n-- \nBest regards,\nEugen Konkov\n\n"}]}