Bug 226254

Summary: Merge Review: perl-Devel-Symdump
Product: [Fedora] Fedora Reporter: Nobody's working on this, feel free to take it <nobody>
Component: Package ReviewAssignee: Robin Norwood <robin.norwood>
Status: CLOSED RAWHIDE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: ppisar, robin.norwood
Target Milestone: ---Flags: jose.p.oliveira.oss: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-02-26 23:39:57 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Specfile patch none

Description Nobody's working on this, feel free to take it 2007-01-31 20:35:15 UTC
Fedora Merge Review: perl-Devel-Symdump

http://cvs.fedora.redhat.com/viewcvs/devel/perl-Devel-Symdump/
Initial Owner: rnorwood@redhat.com

Comment 1 Jose Pedro Oliveira 2007-02-03 19:53:27 UTC
Created attachment 147281 [details]
Specfile patch

Robin,

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
  (%setup line)
* 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

jpo

Comment 2 Robin Norwood 2007-02-04 18:01:48 UTC
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.

Comment 3 Jose Pedro Oliveira 2007-02-04 21:28:53 UTC
Robin,

Looks good! Minus one in the review pile.

jpo


Comment 4 Chris Weyl 2009-02-26 23:39:57 UTC
Closing... (merge review with fedora-review+)

Comment 5 Petr Pisar 2012-01-11 12:55:47 UTC
Package Change Request
======================
Package Name: perl-Devel-Symdump
New Branches: f15 f16
Owners: 
InitialCC: perl-sig

Please grant `perl-sig' pseudo-user permissions to watch bugzilla and commits for all Fedora branches.

Comment 6 Gwyn Ciesla 2012-01-11 13:07:50 UTC
I think I've done what you need, let me know if not.

Comment 7 Petr Pisar 2012-01-11 13:35:19 UTC
(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.

Comment 8 Gwyn Ciesla 2012-01-11 14:25:44 UTC
Fixed. Sorry for the confusion.