Red Hat Bugzilla – Bug 226254
Merge Review: perl-Devel-Symdump
Last modified: 2012-01-11 09:25:49 EST
Fedora Merge Review: perl-Devel-Symdump
Initial Owner: firstname.lastname@example.org
Created attachment 147281 [details]
The patch includes
* update to version 2.07 (and an epoch bump 604 > 7)
* expands a couple of tabs to spaces
(release and requires lines)
* removes a space
* drops the perl build requirement
(the perl package exists in the build root package set)
* changes the order of the find options
* removes the '|| :' in the %check section
* adds a 'pm' to the man pages section
Thanks for the review, Jose!
I've applied your changes, including the (yuck! But necessary.) epoch bump.
And built for FC7. Let me know how it looks.
Looks good! Minus one in the review pile.
Closing... (merge review with fedora-review+)
Package Change Request
Package Name: perl-Devel-Symdump
New Branches: f15 f16
Please grant `perl-sig' pseudo-user permissions to watch bugzilla and commits for all Fedora branches.
I think I've done what you need, let me know if not.
(In reply to comment #6)
> I think I've done what you need, let me know if not.
Unfortunately you haven't. I wanted to add perl-sig user with watchbugzilla and watchcommits to list of co-maintainers (this is equivalent to InitialCC) and not to touch owner of the package. You made perl-sig owner.
Please revert the owner to former one (I guess, `mmaslano' was it), and append perl-sig to co-maintainters with the two permission.
Fixed. Sorry for the confusion.