threads / bug / 3621

annotate.perl triggers rpm bug

Subject: annotate.perl triggers rpm bug

## tl;dr

3 messages between Mar 10, 2006 and Mar 11, 2006.

replies: 2people: 2as markdown or json

sean· Mar 10, 2006, 21:23 UTC · lore

Hi Ryan

Commit 87475f4 exposes a problem in the creation of rpms (at least on Fedora boxes). RPM automatically creates a dependency for any perl "use" lines, and the above commit fools it into creating a false dependecy for the nonexistent perl package "revs" because of this portion of the diff:

+       -S, --rev-file revs-file
+                       use revs from revs-file instead of calling git-rev-list
                        ^^ fools rpm

Obviously this creates a problem when someone goes to install the git-core rpm. Any chance we could get a small change to work around this bug in rpm?

Thanks, Sean

diff --git a/git-annotate.perl b/git-annotate.perl
index feea0a2..6805b8b 100755
--- a/git-annotate.perl
+++ b/git-annotate.perl
@@ -20,7 +20,7 @@ sub usage() {
        -r, --rename
                        Follow renames (Defaults on).
        -S, --rev-file revs-file
-                       use revs from revs-file instead of calling git-rev-list
+                       uses revs from revs-file instead of calling git-rev-list
        -h, --help
                        This message.
 ';

Junio C Hamano· Mar 11, 2006, 07:11 UTC · re: sean · lore

Re: annotate.perl triggers rpm bug

sean <seanlkml@sympatico.ca> writes:

> ....   RPM automatically creates a dependency for any
> perl "use" lines, and the above commit fools it into creating a false 
> dependecy...

Sheesh -- broken packaging.

Show 13 quoted lines

> diff --git a/git-annotate.perl b/git-annotate.perl
> index feea0a2..6805b8b 100755
> --- a/git-annotate.perl
> +++ b/git-annotate.perl
> @@ -20,7 +20,7 @@ sub usage() {
>         -r, --rename
>                         Follow renames (Defaults on).
>         -S, --rev-file revs-file
> -                       use revs from revs-file instead of calling git-rev-list
> +                       uses revs from revs-file instead of calling git-rev-list
>         -h, --help
>                         This message.
>  ';

How about upcasing the word like other lines?

sean· Mar 11, 2006, 07:39 UTC · re: Junio C Hamano · lore

Re: annotate.perl triggers rpm bug

On Fri, 10 Mar 2006 23:11:01 -0800 Junio C Hamano <junkio@cox.net> wrote:

> Sheesh -- broken packaging.

Yeah, not rpm's proudest moment.

> How about upcasing the word like other lines?

That seems to fix the dependency problem too and makes a little more sense.

Sean

diff --git a/git-annotate.perl b/git-annotate.perl
index feea0a2..9df72a1 100755
--- a/git-annotate.perl
+++ b/git-annotate.perl
@@ -20,7 +20,7 @@ sub usage() {
 	-r, --rename
 			Follow renames (Defaults on).
 	-S, --rev-file revs-file
-			use revs from revs-file instead of calling git-rev-list
+			Use revs from revs-file instead of calling git-rev-list
 	-h, --help
 			This message.
 ';