Bug 226254 - Merge Review: perl-Devel-Symdump
Merge Review: perl-Devel-Symdump
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Robin Norwood
Fedora Package Reviews List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-01-31 15:35 EST by Nobody's working on this, feel free to take it
Modified: 2012-01-11 09:25 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2009-02-26 18:39:57 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
jose.p.oliveira.oss: fedora‑review+


Attachments (Terms of Use)
Specfile patch (1.70 KB, patch)
2007-02-03 14:53 EST, Jose Pedro Oliveira
no flags Details | Diff

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 15:35:15 EST
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 14:53:27 EST
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 13:01:48 EST
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 16:28:53 EST
Robin,

Looks good! Minus one in the review pile.

jpo
Comment 4 Chris Weyl 2009-02-26 18:39:57 EST
Closing... (merge review with fedora-review+)
Comment 5 Petr Pisar 2012-01-11 07:55:47 EST
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 Jon Ciesla 2012-01-11 08:07:50 EST
I think I've done what you need, let me know if not.
Comment 7 Petr Pisar 2012-01-11 08:35:19 EST
(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 Jon Ciesla 2012-01-11 09:25:44 EST
Fixed. Sorry for the confusion.

Note You need to log in before you can comment on or make changes to this bug.