threads / rfc / 23767

RFC patchgitweb: Use -M option while using format-patch

Subject: [PATCH RFC GSoC] gitweb: Use -M option while using format-patch

## tl;dr

4 messages between May 10, 2010 and May 10, 2010. Diffs are folded; open one to read it.

replies: 3people: 4as markdown or json

Pavan Kumar Sunkara· May 10, 2010, 16:08 UTC · lore

Add option '-M' to use with format-patch whcih detects renames in the commit.

Signed-off-by: Pavan Kumar Sunkara <pavan.sss1991@gmail.com>
---
  gitweb/gitweb.perl |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)
Show changes to gitweb/gitweb.perl +1 −1
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index c356e95..0184de5 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -6117,7 +6117,7 @@ sub git_commitdiff {
                         }
                         push @commit_spec, '--root', $hash;
                 }
-               open $fd, "-|", git_cmd(), "format-patch", 
'--encoding=utf8',
+               open $fd, "-|", git_cmd(), "format-patch", '-M', 
'--encoding=utf8',
                         '--stdout', @commit_spec
                         or die_error(500, "Open git-format-patch failed");
         } else {
-- 
1.7.0.4
Jakub Narebski· May 10, 2010, 16:41 UTC · re: Pavan Kumar Sunkara · lore

Re: [PATCH RFC GSoC] gitweb: Use -M option while using format-patch

CC-ed Guiseppe Bilotta, author of 'patch' and 'patches' views (you can find this information using "git blame" or "git log --grep").

On Mon, 10 May 2010, Pavan Kumar Sunkara wrote:
> Add option '-M' to use with format-patch whcih detects
> renames in the commit.
  Subject: [PATCH RFC GSoC] gitweb: Use @diff_opts while using format-patch
  Make git-format-patch (used by 'patch' and 'patches' views) use the
  same rename detection options that git-diff and git-diff-tree (used
  by 'commitdiff', 'blobdiff', etc.) use.

Note that I am not sure if it is really a good change, as it might create patches that (currently) can be applied only with git.

Show 17 quoted lines
> 
> Signed-off-by: Pavan Kumar Sunkara <pavan.sss1991@gmail.com>
> ---
> 
>   gitweb/gitweb.perl |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
> index c356e95..0184de5 100755
> --- a/gitweb/gitweb.perl
> +++ b/gitweb/gitweb.perl
> @@ -6117,7 +6117,7 @@ sub git_commitdiff {
>                          }
>                          push @commit_spec, '--root', $hash;
>                  }
> -               open $fd, "-|", git_cmd(), "format-patch", '--encoding=utf8',
> +               open $fd, "-|", git_cmd(), "format-patch", '-M', '--encoding=utf8',
  +               open $fd, "-|", git_cmd(), "format-patch", @diff_opts, '--encoding=utf8',
or
  -               open $fd, "-|", git_cmd(), "format-patch", '--encoding=utf8',
  -                        '--stdout', @commit_spec
  +               open $fd, "-|", git_cmd(), "format-patch", @diff_opts,
  +                         '--encoding=utf8', '--stdout', @commit_spec

This patsh is both line-wrapped and whitespace-damaged (tabs replaced by spacess)

>                          '--stdout', @commit_spec
>                          or die_error(500, "Open git-format-patch failed");
>          } else {
> -- 
-- >8 --
Subject: [PATCH RFC GSoC] gitweb: Use @diff_opts while using format-patch

Make git-format-patch (used by 'patch' and 'patches' views) use the same rename detection options that git-diff and git-diff-tree (used by 'commitdiff', 'blobdiff', etc.) use.

Signed-off-by: Pavan Kumar Sunkara <pavan.sss1991@gmail.com>
Acked-by: Jakub Narebski <jnareb@gmail.com>
---
 gitweb/gitweb.perl |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
Show changes to diff +2 −2
diff --git i/gitweb/gitweb.perl w/gitweb/gitweb.perl
index 225f404..d983ce5 100755
--- i/gitweb/gitweb.perl
+++ w/gitweb/gitweb.perl
@@ -6295,8 +6295,8 @@ sub git_commitdiff {
 			}
 			push @commit_spec, '--root', $hash;
 		}
-		open $fd, "-|", git_cmd(), "format-patch", '--encoding=utf8',
-			'--stdout', @commit_spec
+		open $fd, "-|", git_cmd(), "format-patch", @diff_opts,
+			'--encoding=utf8', '--stdout', @commit_spec
 			or die_error(500, "Open git-format-patch failed");
 	} else {
 		die_error(400, "Unknown commitdiff format");
Giuseppe Bilotta· May 10, 2010, 16:56 UTC · re: Jakub Narebski · lore

Re: [PATCH RFC GSoC] gitweb: Use -M option while using format-patch

On Mon, May 10, 2010 at 6:41 PM, Jakub Narebski <jnareb@gmail.com> wrote:
Show 13 quoted lines
> CC-ed Guiseppe Bilotta, author of 'patch' and 'patches' views
> (you can find this information using "git blame" or "git log --grep").
>
> On Mon, 10 May 2010, Pavan Kumar Sunkara wrote:
>
>> Add option '-M' to use with format-patch whcih detects
>> renames in the commit.
>
>  Subject: [PATCH RFC GSoC] gitweb: Use @diff_opts while using format-patch
>
>  Make git-format-patch (used by 'patch' and 'patches' views) use the
>  same rename detection options that git-diff and git-diff-tree (used
>  by 'commitdiff', 'blobdiff', etc.) use.
Excellent idea.
> Note that I am not sure if it is really a good change, as it might
> create patches that (currently) can be applied only with git.

I think we can live with that. If we get strong complains about this, we can always introduce support for extra options to cancel the rename detection. (Also, the upcoming GNU patch release seems to support the git diff format, including copies and renames.)

Show 9 quoted lines
> -- >8 --
> Subject: [PATCH RFC GSoC] gitweb: Use @diff_opts while using format-patch
>
> Make git-format-patch (used by 'patch' and 'patches' views) use the
> same rename detection options that git-diff and git-diff-tree (used
> by 'commitdiff', 'blobdiff', etc.) use.
>
> Signed-off-by: Pavan Kumar Sunkara <pavan.sss1991@gmail.com>
> Acked-by: Jakub Narebski <jnareb@gmail.com>
Acked-by: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
Show 20 quoted lines
> ---
>  gitweb/gitweb.perl |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git i/gitweb/gitweb.perl w/gitweb/gitweb.perl
> index 225f404..d983ce5 100755
> --- i/gitweb/gitweb.perl
> +++ w/gitweb/gitweb.perl
> @@ -6295,8 +6295,8 @@ sub git_commitdiff {
>                        }
>                        push @commit_spec, '--root', $hash;
>                }
> -               open $fd, "-|", git_cmd(), "format-patch", '--encoding=utf8',
> -                       '--stdout', @commit_spec
> +               open $fd, "-|", git_cmd(), "format-patch", @diff_opts,
> +                       '--encoding=utf8', '--stdout', @commit_spec
>                        or die_error(500, "Open git-format-patch failed");
>        } else {
>                die_error(400, "Unknown commitdiff format");
>
-- 
Giuseppe "Oblomov" Bilotta
Petr Baudis· May 10, 2010, 18:11 UTC · re: Pavan Kumar Sunkara · lore

Re: [PATCH RFC GSoC] gitweb: Use -M option while using format-patch

On Mon, May 10, 2010 at 09:38:45PM +0530, Pavan Kumar Sunkara wrote:
> Add option '-M' to use with format-patch whcih detects
> renames in the commit.
> 
> Signed-off-by: Pavan Kumar Sunkara <pavan.sss1991@gmail.com>

I think git-specific format is no big issue; even if you are applying the patch with plain patch(1) instead, you might very well prefer to be informed of the rename and take appropriate precautions (like mv'ing before applying to avoid unsolvable rejects).

Acked-by: Petr Baudis <pasky@suse.cz>
> ---
> 
>  gitweb/gitweb.perl |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

But please do something about the whitespace damage in the patches you send, thanks!

-- 
				Petr "Pasky" Baudis
When I feel like exercising, I just lie down until the feeling
goes away.  -- xed_over

← back to recent threads