[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-05-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #17 from Fedora Update System upda...@fedoraproject.org 
2010-05-07 13:26:17 EDT ---
perl-Sys-CPU-0.51-2.fc12 has been pushed to the Fedora 12 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-05-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-Sys-CPU-0.51-2.fc12
 Resolution||ERRATA

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #14 from Kevin Fenzi ke...@tummy.com 2010-05-06 11:46:52 EDT ---
CVS done (by process-cvs-requests.py).

(We are not doing F14 branches yet, so left that out)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #15 from Shakthi Kannan shakthim...@gmail.com 2010-05-06 12:01:41 
EDT ---
What about EL-5?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-05-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||fedora-review+

--- Comment #12 from Marcela Mašláňová mmasl...@redhat.com 2010-05-04 
03:28:14 EDT ---
OK vrpmlint on every package.
OK name of package accords to the Naming Guidelines.
OK spec file name match the base package %{name}.
OK package meet the Packaging Guidelines.
OK package has a good license.
OK spec file is written in American English.
OK spec file for the package must be legible.
OK The sources used to build the package must match the upstream source.
OK successful koji compilation.
OK correct BuildRequires, Requires.
OK proper use of %find_lang macro.
OK shared library must call ldconfig in %post and %postun.
OK relocatable package must state this fact.
OK package must own their directories.
OK permissions on files must be set properly.
OK %clean section with rm -rf %{buildroot}.
OK consistent use of macros.
OK package contains code, or permissable content.
OK large documentation go in a -doc subpackage.
OK %doc must not affect the runtime of the application.
OK header files must be in a -devel package.
OK static libraries must be in a -static package.
OK pkgconfig(.pc) files must 'Requires: pkgconfig'.
OK library files with a suffix (.so) must go in -devel.
OK usually devel packages must require the base package.
OK Remove .la libtool archives.
OK GUI applications must include a %{name}.desktop file.
OK %install section starts with rm -rf %{buildroot}.

ACCEPT

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

 CC||tcall...@redhat.com
 Blocks|182235(FE-Legal)|

--- Comment #10 from Tom spot Callaway tcall...@redhat.com 2010-05-03 
10:29:06 EDT ---
For this package, this is the appropriate entry:

# Some code was copied from Unix::Processors, which is LGPLv3 or Artistic 2.0
# The rest of the code is under the standard Perl license (GPL+ or Artistic)
License: (GPL+ or Artistic) and (LGPLv3 or Artistic 2.0)

Lifting FE-Legal.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #11 from Shakthi Kannan shakthim...@gmail.com 2010-05-03 12:05:19 
EDT ---
Thanks Tom spot Callaway for clearing out this one! Updated at:

SPEC: http://shakthimaan.fedorapeople.org/SPECS/perl-Sys-CPU.spec
SRPM:
http://shakthimaan.fedorapeople.org/SRPMS/perl-Sys-CPU-0.51-2.fc12.src.rpm

$ rpmlint perl-Sys-CPU.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

$ rpmlint perl-Sys-CPU-0.51-2.fc12.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint perl-Sys-CPU-0.51-2.fc12.i686.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

Koji builds for F-12, F-13, F-14, EL-5 are successful.

http://koji.fedoraproject.org/koji/taskinfo?taskID=2158115
http://koji.fedoraproject.org/koji/taskinfo?taskID=2158120
http://koji.fedoraproject.org/koji/taskinfo?taskID=2158123
http://koji.fedoraproject.org/koji/taskinfo?taskID=2158126

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-04-29 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #6 from Marcela Mašláňová mmasl...@redhat.com 2010-04-29 12:33:28 
EDT ---
And it also says they've copied part from
http://search.cpan.org/~wsnyder/Unix-Processors-2.042/, which is proclaimed to
be GPL+ or Artistic, but in code they mentioned GPLv3+ or Artistic 2.0. So now
what? If you don't know about guideline for this case, then you should probably
send an email to legal (fedora-legal-l...@redhat.com)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-04-29 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #7 from Shakthi Kannan shakthim...@gmail.com 2010-04-29 12:55:47 
EDT ---
I was able to contact Marc Koderer at a different e-mail address (cpan AT marc 
HYPHEN koderer dot de) and this was his reply:

I am not the author - Matt (mzsanford at cpan DOT org) only transfered the
package to me. But it must be released as Perl licence. So far I understand
the hint in CPU.xs.

@Matt: is SYS::CPU a dual licence code? I think I will add a header in the code
if it's ok for you.

I am yet to hear from Matt on the above.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-04-29 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #8 from Marcela Mašláňová mmasl...@redhat.com 2010-04-29 13:05:58 
EDT ---
Well, I'd rather check with legal just to be sure. They've proclaimed they are
using part of code from different module. For example perl is using unicode
tables, so it must have also unicode license. I'm not sure about this.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-04-29 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

 Blocks||182235(FE-Legal)

--- Comment #9 from Jason Tibbitts ti...@math.uh.edu 2010-04-29 15:13:24 EDT 
---
Easiest way to check with legal is to block FE-Legal.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-04-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #5 from Shakthi Kannan shakthim...@gmail.com 2010-04-28 12:28:38 
EDT ---
Haven't yet received a reply from upstream. In the CPU.xs file, it has been
mentioned that:

  All code contained herein in free to use and edit under the same licence as
Perl itself.

So, the GPL+ or Artistic License mentioned in the .spec file holds good?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-04-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #4 from Marcela Mašláňová mmasl...@redhat.com 2010-04-26 02:55:24 
EDT ---
(In reply to comment #3)
 I have actually enquired the same to upstream, and awaiting a response. Will
 update as soon as I get a reply from upstream.
 
 Thanks for your reply.

Without license the review can't pass. Ping me when upstream answer.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|mmasl...@redhat.com

--- Comment #2 from Marcela Mašláňová mmasl...@redhat.com 2010-04-25 10:46:18 
EDT ---
Only one problem here. There is not specified license by upstream. Please tell
me, where you find it or ask upstream.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #3 from Shakthi Kannan shakthim...@gmail.com 2010-04-26 00:07:11 
EDT ---
I have actually enquired the same to upstream, and awaiting a response. Will
update as soon as I get a reply from upstream.

Thanks for your reply.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 585336] Review Request: perl-Sys-CPU - Getting CPU information

2010-04-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=585336

--- Comment #1 from Shakthi Kannan shakthim...@gmail.com 2010-04-23 14:32:15 
EDT ---
$ rpmlint ../SPECS/perl-Sys-CPU.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

$ rpmlint perl-Sys-CPU-0.51-1.fc12.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint perl-Sys-CPU-0.51-1.fc12.i686.rpm
perl-Sys-CPU-debuginfo-0.51-1.fc12.i686.rpm 
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

Successful Koji builds for F-12, F-13, F-14 and EL-5:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2134310
http://koji.fedoraproject.org/koji/taskinfo?taskID=2134315
http://koji.fedoraproject.org/koji/taskinfo?taskID=2134319
http://koji.fedoraproject.org/koji/taskinfo?taskID=2134318

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review