Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 230802 (perl-Callback) - Review Request: perl-Callback - Object interface for function callbacks
Summary: Review Request: perl-Callback - Object interface for function callbacks
Alias: perl-Callback
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: Fedora Package Reviews List
Depends On:
TreeView+ depends on / blocked
Reported: 2007-03-02 23:09 UTC by Madhavprasad Pai
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2007-09-02 17:30:11 UTC
tibbs: fedora-review+
wtogami: fedora-cvs+

Attachments (Terms of Use)

Description Madhavprasad Pai 2007-03-02 23:09:26 UTC
Spec URL:
Description: Object interface for function callbacks

Comment 1 Madhavprasad Pai 2007-03-05 10:08:30 UTC
I need a sponsor.

Comment 2 Ralf Corsepius 2007-03-05 11:45:48 UTC
License:        CHECK(GPL or Artistic)

The license of this package isn't clear to me:

All I can find inside of the sources is this:

   Copyright (C) 1994, 2000, 2002 David Muir Sharnoff. All rights reserved.
    License hearby granted for anyone to use this module at their own risk.
    Please feed useful changes back to

This is not the standard Perl "GPL or Artistic" license, but something
different. In particular, this only covers usage ("use"), but doesn't cover

I'd suggest you to contact the author.

Comment 3 Madhavprasad Pai 2007-03-16 08:57:02 UTC
David Muir has updated the package.
The new license is as follows:

    Copyright (C) 1994, 2000, 2002 David Muir Sharnoff. All rights reserved.
    This module may be licensed on the same terms as Perl itself.

The new spec file is at
The new srpm is at

Comment 4 Chris Weyl 2007-04-08 17:51:51 UTC
While cpanspec does an excellent job creating a base specfile, it still needs
some tweaks:

* the license tag should be corrected
* due to the impending perl/perl-devel split, you should buildrequires

Comment 5 Jason Tibbitts 2007-05-16 04:26:46 UTC
Anything happening here?  It's a always a good idea to respond to reviewer
comments as quickly as possible, but it's especially important if you require

Comment 6 Jason Tibbitts 2007-05-24 17:47:49 UTC
Well, it's been more than another week.  I'll close this bug soon if there's no
further response.

Comment 7 Madhavprasad Pai 2007-05-25 09:53:24 UTC
I am terribly sorry for not updating this BZ earlier. This won't happen again.

I have made the suggested modifications to the spec file and created a new
source rpm. The revised files are at the available at the above locations.

Comment 8 Madhavprasad Pai 2007-05-25 09:55:50 UTC
A clarification is in order here.

The spec file is at 
The souce rpm is at .

Comment 9 Jason Tibbitts 2007-06-03 23:57:06 UTC
Usually we use "GPL or Artistic" to indicate the regular license of Perl
modules.  Looking through my CVS tree, I don't see any using a slash as you've
done, but I suppose it's not really incorrect.

I think you should have a build requirement on perl(Storable), so that you'll
still get complete test coverage in case it gets split out of the main perl package.

* source files match upstream:
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text not included upstream.
* latest version is being packaged.
? BuildRequires are proper.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* rpmlint is silent.
* final provides and requires are sane:
   perl(Callback) = 1.07
   perl-Callback = 1.07-1.fc8
* %check is present and all tests pass:
   All tests successful.
   Files=4, Tests=19,  0 wallclock secs ( 0.09 cusr +  0.06 csys =  0.15 CPU)
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

Comment 10 Madhavprasad Pai 2007-06-04 15:24:45 UTC
>I think you should have a build requirement on perl(Storable), so that you'll
>still get complete test coverage in case it gets split out of the main perl

I have changed the spec file as per this suggestion.

The spec file is at
The source rpm is

Comment 11 Jason Tibbitts 2007-06-06 22:36:00 UTC
OK, looks good to me.


Note that you're currently up to the "Get a Fedora Account" step of

Comment 12 Jason Tibbitts 2007-06-14 19:23:58 UTC

Comment 13 Madhavprasad Pai 2007-06-20 14:34:34 UTC
I have joined PackageMaintainers
Thank you

Comment 14 Jason Tibbitts 2007-06-20 16:57:44 UTC
I've already approved this package, so you're currently either at the "Install
the Build-System Client Tools" or maybe "Add Package to CVS and Set Owner" step

Comment 15 Madhavprasad Pai 2007-07-01 03:11:11 UTC
Hi Jason,

I am having some trouble with my privileges. I am working on it at the moment.

Thanks and regards,
-- Pai

Comment 16 Madhavprasad Pai 2007-07-06 11:19:25 UTC
New Package CVS Request
Package Name: perl-Callback
Short Description: Object interface for function callbacks
Owners: M.K.Pai, David Muir
Branches: FC-6 FC-7

Comment 17 Kevin Fenzi 2007-07-06 18:02:21 UTC
Can you resubmit your request with email addresses in the Owners: field?
Thats the format the cvs scripts expect. 

Comment 18 Madhavprasad Pai 2007-07-09 07:51:21 UTC
New Package CVS Request
Package Name: perl-Callback
Short Description: Object interface for function callbacks
Branches: FC-6 FC-7

Comment 19 Jason Tibbitts 2007-07-20 00:39:14 UTC
Any reason this hasn't been built yet?

Comment 20 Madhavprasad Pai 2007-07-20 11:55:17 UTC
Hi Jason, 

I have successfully built the package for devel. But I have some problems with
the  FC-6 and F-7 branches. Its the same error. I have attached the session below.

[mkpai@mkpai FC-6]$ make upload FILES=Callback-1.07.tar.gz 

Checking : Callback-1.07.tar.gz on
This file (270e8cde126409c45294886d34b17408  Callback-1.07.tar.gz) is already

Source upload succeeded. Don't forget to commit the new ./sources file
M sources
M .cvsignore
[mkpai@mkpai FC-6]$ make upload FILES=perl-Callback.spec 

Checking : perl-Callback.spec on
This file (74d114962ff13c080befd96c74a543b9  perl-Callback.spec) is already uploaded

Source upload succeeded. Don't forget to commit the new ./sources file
M sources
M .cvsignore
[mkpai@mkpai FC-6]$ cvs commit
cvs commit: Examining .
**** Access allowed: mkpai is in ACL for rpms/perl-Callback/FC-6.
Checking in .cvsignore;
/cvs/pkgs/rpms/perl-Callback/FC-6/.cvsignore,v  <--  .cvsignore
new revision: 1.2; previous revision: 1.1
Checking in sources;
/cvs/pkgs/rpms/perl-Callback/FC-6/sources,v  <--  sources
new revision: 1.2; previous revision: 1.1
Running syncmail...
...syncmail done.
cvs diff: [11:47:33] waiting for mkpai's lock in /cvs/pkgs/rpms/perl-Callback/FC-6
Running syncmail...
...syncmail done.
cvs diff: [11:48:03] obtained lock in /cvs/pkgs/rpms/perl-Callback/FC-6
[mkpai@mkpai FC-6]$ make tag
cvs tag  -c perl-Callback-1_07-1_fc6
cvs tag: Tagging .
T .cvsignore
T Makefile
T branch
T sources
Tagged with: perl-Callback-1_07-1_fc6

[mkpai@mkpai FC-6]$ make build
perl-Callback.spec not tagged with tag perl-Callback-1_07-1_fc6
make: *** [build-check] Error 1

The error with the F-7 branch is the same. What should I do ?

Thanks for your patience,
-- Pai

Comment 21 Jason Tibbitts 2007-07-20 15:13:56 UTC
It doesn't look to me like you've even imported your sources to the F-7 and FC-6
branches.  A checkout doesn't show spec files there.  How did you import your

Comment 22 Madhavprasad Pai 2007-07-23 06:14:49 UTC
I followed the procedure at
and successfully imported for the devel branch.

If I am required to do something else for FC-6 and F-7, please tell me what it
is. I will update the wiki accordingly.

-- Pai

Comment 23 Jason Tibbitts 2007-07-23 16:22:20 UTC
Well, all you're required to do for the release branches is described in step 11
of that document; I see nothing incorrect there that would need to be updated.

Comment 24 Madhavprasad Pai 2007-07-25 11:51:41 UTC
I had omitted to cvs add the spec file after copying. That was the problem. The
packages have been built now.

Please let me know if I should now close this BZ.

Thanks a lot again.
-- Pai

Comment 25 Jason Tibbitts 2007-07-27 21:51:49 UTC
You should close this ticket once the package is built and pushed to the repos.
 Bodhi will do this for you if you reference the ticket number when you create
the push request.

Comment 26 Jason Tibbitts 2007-08-25 16:56:01 UTC
So it's been another month; this package is in rawhide and was built for but
never pushed to F7.  Do you need help getting your package released?  Or can you
close this ticket?

Comment 27 Madhavprasad Pai 2007-08-27 12:00:55 UTC
I am surprised this has not been pushed yet. I got the following mail from on 30th July.


mkpai has submitted a new update for Fedora 7

    Release: Fedora 7
     Status: pending
       Type: bugfix
      Notes: Object interface for function callbacks
  Submitter: mkpai
  Submitted: 2007-07-30 05:58:33

Comment 28 Jason Tibbitts 2007-09-02 17:30:11 UTC
The package has now been pushed to F7.  This ticket should be closed by the
submitter, but I see no reason why it should stay open and in the interests of
expediency and keeping my overly long open ticket list down I'll go ahead and
close this myself.

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