[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2016-12-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NOTABUG
  Flags|needinfo?(hadfieldster@gmai |
   |l.com)  |
Last Closed||2016-12-15 01:17:51



--- Comment #24 from Parag AN(पराग)  ---
Please reopen when you back to work on this.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2016-12-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585

Parag AN(पराग)  changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |201449 (FE-DEADREVIEW)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=201449
[Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter
response should be blocking this bug.
-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2016-07-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585

Parag AN(पराग)  changed:

   What|Removed |Added

  Flags||needinfo?(hadfieldster@gmai
   ||l.com)



--- Comment #18 from Parag AN(पराग)  ---
Steven,
   You around and still want to continue with this package?

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2016-08-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585



--- Comment #19 from Parag AN(पराग)  ---
I need updated SRPM as per comment#15 to continue this review.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585

Steven Hadfield  changed:

   What|Removed |Added

  Flags|needinfo?(hadfieldster@gmai |
   |l.com)  |



--- Comment #20 from Steven Hadfield  ---
I am willing to continue this package. I had reviewed a few other packages and
made comments, but was uncertain what the remaining steps were.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585



--- Comment #21 from Parag AN(पराग)  ---
This review created in 2011 and now there is new template to file package
reviews that provides fas id as well.

Can you tell what is your FAS id and link to the package reviews you have done?
We can proceed with remaining steps when you provide updated SPEC and SRPM as
you promised in comment#17 above.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585



--- Comment #22 from Parag AN(पराग)  ---
Also, in recent discussions on Fedora devel list, its suggested that new
contributors can use Copr project to build their packages and provide links to
it here.

Go to https://copr.fedorainfracloud.org/coprs and login with your FAS username
and password, create a New project there.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585



--- Comment #23 from Parag AN(पराग)  ---
Also, I find you have attempted only one full package review in bug 1196037 but
I am not able to understand if you are able to understand RPM packaging so do
more full package reviews.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2016-09-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585

Parag AN(पराग)  changed:

   What|Removed |Added

  Flags||needinfo?(hadfieldster@gmai
   ||l.com)



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2015-08-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585

Parag AN(पराग)  changed:

   What|Removed |Added

 CC||hadfields...@gmail.com,
   ||panem...@gmail.com
  Flags||needinfo?(hadfieldster@gmai
   ||l.com)



--- Comment #8 from Parag AN(पराग)  ---
Please respond submitter if you still want to continue with this package
submission in a week time.

Initiating
https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews#Submitter_not_responding

Can't find FAS associated with this package submitter email.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2015-08-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585

Mamoru TASAKA  changed:

   What|Removed |Added

  Flags||needinfo?(panem...@gmail.co
   ||m)



--- Comment #9 from Mamoru TASAKA  ---
Parag, why are you setting needinfo from the reporter although the last comment
on this bug is from the "reporter"? What we must do is to respond to the
comment 7, not asking the response from the reporter.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2015-08-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585



--- Comment #10 from Mamoru TASAKA  ---
Also, it is reasonable that FE-NEEDSPONSOR people has no FAS.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2015-08-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585

Mamoru TASAKA  changed:

   What|Removed |Added

  Flags|needinfo?(hadfieldster@gmai |
   |l.com)  |



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2015-08-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585

Parag AN(पराग)  changed:

   What|Removed |Added

  Flags|needinfo?(panem...@gmail.co |
   |m)  |



--- Comment #11 from Parag AN(पराग)  ---
I just asked a simple question if submitter is around? If he is, I will sponsor
him. 

(In reply to Mamoru TASAKA from comment #10)
> Also, it is reasonable that FE-NEEDSPONSOR people has no FAS.

Sorry, but I really think FAS is MUST when you want to contribute a package in
Fedora which is also documented on package review process wiki page by
following
https://fedoraproject.org/wiki/Join_the_package_collection_maintainers#Create_a_Fedora_Account

Anyways as you suggested here is my review for srpm submitted in comment#7

Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed


Issues:
===
- Package uses either %{buildroot} or $RPM_BUILD_ROOT
  Note: Using both %{buildroot} and $RPM_BUILD_ROOT
  See: http://fedoraproject.org/wiki/Packaging/Guidelines#macros

==> I think this is self-explanatory how to fix this in spec file.

- If (and only if) the source package includes the text of the license(s)
  in its own file, then that file, containing the text of the license(s)
  for the package is included in %license.
  Note: License file LICENSE is marked as %doc instead of %license
  See:
  http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text

=> For this please move LICENSE file to %license macro just below %doc as
%license LICENSE

- Ask upstream to update fsf address in LICENSE file.

- Group tag is optional in spec, you can remove it.

= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated". 1 files have unknown license. Detailed
 output of licensecheck in /home/parag/Downloads/perl-Apache-
 Htgroup/licensecheck.txt
[-]: If the package is under multiple licenses, the licensing breakdown
 must be documented in the spec.
[x]: Package does not own files or directories owned by other packages.
 Note: Dirs in package are owned also by:
 /usr/share/perl5/vendor_perl/Apache(perl-Apache-DBI, ocsinventory-
 server, perl-Apache-LogRegex, perl-XMLRPC-Lite, perl-Apache-Session-
 LDAP, perl-Apache-Session-NoSQL, perl-Apache-DBI-Cache, perl-Apache-
 Session-Browseable, perl-SOAP-Lite, perl-Apache-LogFormat-Compiler,
 perl-Apache-Htpasswd, perl-Apache-RPC, perl-Apache-Session, perl-
 Frontier-RPC, perl-Maypole)

==> This is okay based on
https://fedoraproject.org/wiki/Packaging:Guidelines#File_and_Directory_Ownership

[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[!]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf %{buildroot} present but not required

=> Please remove this removal of buildroot line from spec as its not needed
nowadays as rpm will take care of it.

[x]: Sources contain only permissible code or content.
[!]: Each %files section contains %defattr if rpm < 4.4
 Note: %defattr present but not needed

=> Please remove this defattr line from spec as its not needed nowadays as rpm
will take care of it.

[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[-]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 30720 bytes in 2 files.
[x]: Package complies to the Packaging Guidelines

=> Except the optional things now in spec is not removed.

[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: Package requires other packages for dire

[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2015-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585



--- Comment #12 from Steven Hadfield  ---
Thank you for the suggested fixes. I've updated the specs (and pushed to a
Github repo). As I recall, I asked the author about updating the license some
years ago and never received a response. I can try again, but I'm not sure I'll
get a response.

Spec:
https://raw.githubusercontent.com/steven-hadfield/rpm-specs/master/SPECS/perl-Apache-Htgroup.spec
Srpm:
https://github.com/steven-hadfield/rpm-specs/blob/master/SRPMS/perl-Apache-Htgroup-1.23-3.fc22.src.rpm

I have also signed up for a FAS account. Was there anything else I missed?

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2015-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|panem...@gmail.com
  Flags||fedora-review?



--- Comment #13 from Parag AN(पराग)  ---
Wow! I never thought you would be around. I was almost prepared to close this
after a week but its Mamoru who guided me correctly here.

As promised above I will sponsor you but you need to follow following things
please.

We have this process 
http://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group to
get sponsored into the packager group. Can you either submit few more packages
and/or some full detailed package reviews? This is needed to make sure package
submitter understands the rpm packaging well and follows the fedora packaging
guidelines.

Please go through the following links
1) http://fedoraproject.org/wiki/Package_Review_Process

2) https://fedoraproject.org/wiki/PackagingGuidelines

3) To find the packages already submitted for review, check
http://fedoraproject.org/PackageReviewStatus/

4) http://fedoraproject.org/wiki/Packaging:ReviewGuidelines and
http://fedoraproject.org/wiki/Package_Review_Process#Reviewer is useful while
doing package reviews.

5) https://fedorahosted.org/FedoraReview/ this is fedora-review tool to help
review packages in fedora. You need to use this and do un-official package
reviews of packages submitted by other contributors. While doing so mention
"This is un-official review of the package." at top of your review comment.

Good to review packages listed in
http://fedoraproject.org/PackageReviewStatus/NEW.html

When you do full package review of some packages, provide that review comment
link here so that I can look how you have reviewed those packages.

If you got any questions please ask :)

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2015-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585



--- Comment #14 from Parag AN(पराग)  ---
you can also use fedora-review on this bugzilla but it will not work as you
gave srpm link which is not direct downloadable. Let me paste here your package
links again.

Spec:
https://raw.githubusercontent.com/steven-hadfield/rpm-specs/master/SPECS/perl-Apache-Htgroup.spec
Srpm:
https://raw.githubusercontent.com/steven-hadfield/rpm-specs/master/SRPMS/perl-Apache-Htgroup-1.23-3.fc22.src.rpm

Here is how you can use it
fedora-review -b 673585 -m fedora-rawhide-x86_64

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2015-08-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585



--- Comment #15 from Parag AN(पराग)  ---
Here is my review for your -3 release srpm 

1)I see you have used Buildroot tag which is also become optional and not
needed. Can you remove it and upload again spec and srpm? No need to bump the
release tag. Also, fedora-review output review.txt showed mismatch of spec
given by spec url and spec file inside srpm as

Diff spec file in url and in SRPM
-
--- /home/parag/673585-perl-Apache-Htgroup/srpm/perl-Apache-Htgroup.spec   
2015-08-18 21:51:58.536356400 +0530
+++
/home/parag/673585-perl-Apache-Htgroup/srpm-unpacked/perl-Apache-Htgroup.spec  
2015-08-18 20:38:27.0 +0530
@@ -35,6 +35,6 @@

 %files
-%doc README
 %license LICENSE
+%doc README
 %{perl_vendorlib}/*
 %{_mandir}/man3/*
@@ -42,5 +42,5 @@
 %changelog
 * Tue Aug 18 2015 Steven Hadfield  1.23-3
-- RPM spec cleanup
+- RPM spec fixes

 * Fri Jul 6 2012 Steven Hadfield  1.23-2


2) It is okay if there is no response from upstream on fixing fsf address
issue.

3) Start reviewing other people's packages and once you did full package review
like I did for you in comment#11 here, post that review link in this bugzilla
to let me know that you are reviewing and your reviews are good where you have
pointed some issues with how to fix them.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2015-09-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585



--- Comment #16 from Parag AN(पराग)  ---
Any updates here?

Can you submit updated spec and srpm?

Can you provide links of reviews done by you for other people's packages?

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2015-10-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585



--- Comment #17 from Steven Hadfield  ---
I apologize for not updating sooner. I did update the SPEC and SRPM.

I commented on package request for 1174346 and started a review for 1196037

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2011-04-20 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=673585

PRABIN KUMAR DATTA  changed:

   What|Removed |Added

 CC||linux.n@gmail.com

--- Comment #5 from PRABIN KUMAR DATTA  2011-04-20 
21:49:06 EDT ---
* MUST: rpmlint must be run on every package [Fix  ]
--/Rpmlint output/--
$ rpmlint -i RPMS/noarch/perl-Apache-Htgroup-1.23-1.fc14.noarch.rpm 
perl-Apache-Htgroup.noarch: W: invalid-license Artistic
The value of the License tag was not recognized.  Known values are: "AAL",
"Adobe", "ADSL", "AFL", "AGPLv1", "AGPLv3", "AGPLv3 with exceptions",
"AMDPLPA", "AML", "AMPAS BSD", "APSL 2.0", "APSL 2.0+", "ARL", "Artistic 2.0",
"Artistic clarified", "ASL 1.0", "ASL 1.0+", "ASL 1.1", "ASL 1.1+", "ASL 2.0",
"ASL 2.0+", "Beerware", "BeOpen", "BitTorrent", "Boost", "BSD", "BSD
Protection", "BSD with advertising", "CATOSL", "CC0", "CeCILL", "CeCILL-B",
"CeCILL-C", "CDDL", "CNRI", "Condor", "Copyright only", "CPAL", "CPL",
"Crystal Stacker", "DOC", "dvipdfm", "ECL 1.0", "ECL 2.0", "eCos", "EFL 2.0",
"EFL 2.0+", "Entessa", "EPL", "ERPL", "EU Datagrid", "EUPL 1.1", "Eurosym",
"Fair", "FTL", "Giftware", "GL2PS", "Glide", "gnuplot", "GPL+", "GPL+ or
Artistic", "GPL+ with exceptions", "GPLv1", "GPLv2 or Artistic", "GPLv2+ or
Artistic", "GPLv2", "GPLv2 with exceptions", "GPLv2+", "GPLv2+ with
exceptions", "GPLv3", "GPLv3 with exceptions", "GPLv3+", "GPLv3+ with
exceptions", "IBM", "IJG", "ImageMagick", "iMatix", "Imlib2", "Intel ACPI",
"Interbase", "ISC", "Jabber", "JasPer", "JPython", "Knuth", "LBNL BSD",
"LGPLv2", "LGPLv2 with exceptions", "LGPLv2+", "LGPLv2+ or Artistic", "LGPLv2+
with exceptions", "LGPLv3", "LGPLv3 with exceptions", "LGPLv3+", "LGPLv3+ with
exceptions", "libtiff", "LLGPL", "Logica", "LPL", "LPPL", "mecab-ipadic",
"MirOS", "MIT", "MIT with advertising", "mod_macro", "Motosoto", "MPLv1.0",
"MPLv1.0+", "MPLv1.1", "MPLv1.1+", "MS-PL", "MS-RL", "Naumen", "NCSA",
"NetCDF", "Netscape", "Newmat", "NGPL", "Nokia", "NOSL", "Noweb", "OML",
"OpenLDAP", "OpenPBS", "OpenSSL", "OReilly", "OSL 1.0", "OSL 1.0+", "OSL 1.1",
"OSL 1.1+", "OSL 2.0", "OSL 2.0+", "OSL 2.1", "OSL 2.1+", "OSL 3.0", "OSL
3.0+", "Phorum", "PHP", "PlainTeX", "Plexus", "PostgreSQL", "psutils", "Public
Domain", "Python", "Qhull", "QPL", "Rdisc", "RiceBSD", "Romio", "RPSL",
"Ruby", "Saxpath", "SCEA", "SCRIP", "Sendmail", "Sleepycat", "SISSL", "SLIB",
"SNIA", "SPL", "TCL", "Teeworlds", "TMate", "TOSL", "TPL", "UCD", "Vim",
"VNLSL", "VOSTROM", "VSL", "W3C", "Webmin", "WTFPL", "wxWidgets", "Xerox",
"xinetd", "XSkat", "YPLv1.1", "Zend", "zlib", "zlib with acknowledgement",
"ZPLv1.0", "ZPLv1.0+", "ZPLv2.0", "ZPLv2.0+", "ZPLv2.1", "ZPLv2.1+", "CDL",
"FBSDDL", "GFDL", "IEEE", "LDPL", "OFSFDL", "Open Publication", "Public Use",
"CC-BY", "CC-BY-ND", "CC-BY-SA", "DMTF", "DSL", "EFML", "Free Art",
"GeoGratis", "Green OpenMusic", "OAL", "AMS", "Arphic", "Baekmuk", "Bitstream
Vera", "DoubleStroke", "Hershey", "IPA", "Liberation", "Lucida", "MgOpen",
"mplus", "OFL", "PTFL", "STIX", "Utopia", "Wadalab", "XANO", "Redistributable,
no modification permitted", "Freely redistributable without restriction".

perl-Apache-Htgroup.noarch: W: invalid-license GPL, see LICENSE
--skiped same as above--

1 packages and 0 specfiles checked; 0 errors, 2 warnings.

$ rpmlint -i SRPMS/perl-Apache-Htgroup-1.23-1.fc15.src.rpm 
--skiped same as rpmlint output for rpm file--

1 packages and 0 specfiles checked; 0 errors, 2 warnings.

=> it should have to be
 ..
| License:   GPLv2 or Artistic |
 ..

* MUST: The package must be named according to the Package Naming[OK  ] 
  Guidelines
Check rpmlint output for pynag-0.3-2.src.rpm

* MUST: The spec file for the package MUST be legible.   [OK  ] 


* Must: Spec file matches base package   [OK   ] 

* Must: License must be licensed with a Fedora approved license and meet the
  Licensing Guidelines   [Fix  ]
=> Check rpmlint warnings

* Must: License in spec must match actual license[Fix  ] 
=> Check rpmlint warnings

* Must: License file included in %doc[OK   ]

* Must: Spec file written in American English[OK   ]

* Must: Tar ball matches upstream[OK   ]

* Must: Package successfully builds binary RPMs  [OK   ]
local build -(f14)
Koji build -(f15) http://koji.fedoraproject.org/koji/taskinfo?taskID=3014888

* Must: No duplicate files   [OK   ]

* Must: Macro use must be consistant [OK   ]
Thou,
"Apache-Ht

[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2011-01-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=673585

stevenhadfi...@letu.edu changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

--- Comment #1 from stevenhadfi...@letu.edu 2011-01-28 15:21:40 EST ---
First posted SPEC file and I need a sponsor. I plan to submit other SPEC files
that I've generated for Perl as well as potentially ones for PHP and Python

-- 
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 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2011-02-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=673585

Ralf Corsepius  changed:

   What|Removed |Added

 CC||rc040...@freenet.de

--- Comment #2 from Ralf Corsepius  2011-02-04 01:59:39 
EST ---
Not a formal review, just some remarks:

* Invalid license field:
...
License:Distributable, see LICENSE
...

According to http://cpansearch.perl.org/src/RBOW/Apache-Htgroup-1.23/LICENSE
and
http://cpansearch.perl.org/src/RBOW/Apache-Htgroup-1.23/lib/Apache/Htgroup.pm
this package is licensed "under the same terms as Perl itself"

=> Set the license field to GPLv2+ or Artistic

* Apache-Htgroup-1.22 is outdated (from 2002)
The current version in CPAN is 1.23 (from 2010)

Please upgrade your submission to 1.23

-- 
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 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2011-02-05 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=673585

--- Comment #3 from Michael Schwendt  2011-02-05 08:30:17 
EST ---
Please enter your real name in your bugzilla account preferences. That will
also improve the current
http://fedoraproject.org/PackageReviewStatus/NEEDSPONSOR.html list.

-- 
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 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2011-02-11 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=673585

--- Comment #4 from Steven Hadfield  2011-02-11 
10:02:56 EST ---
Thank you for your assistance. I have made the mentioned changes.

Here's the updated URL to the srpm:
http://www.letu.edu/people/stevenhadfield/rpm/SRPMS/perl-Apache-Htgroup-1.23-1.fc15.src.rpm

-- 
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