Bug 396511 - Review Request: webcpp - Convert C++ code to HTML
Summary: Review Request: webcpp - Convert C++ code to HTML
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Lubomir Rintel
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-11-23 11:11 UTC by Michal Fabry
Modified: 2009-01-07 18:09 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-03-24 14:55:32 UTC
Type: ---
Embargoed:
lkundrak: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Michal Fabry 2007-11-23 11:11:47 UTC
Spec URL: http://michal.fabry.sk/data/fedora/webcpp/webcpp.spec
SRPM URL: http://michal.fabry.sk/data/fedora/webcpp/webcpp-0.8.4-1.fc9.src.rpm
Description: 

Web C Plus Plus is a command line utility that takes your source 
code, and converts it into an HTML file using a fully customizable 
syntax highlighting engine and colour schemes. This is useful if 
you want to post your code online and make it easier to read, or 
to make online programming tutorials.

Comment 1 Lubomir Kundrak 2007-11-23 11:18:34 UTC
Taking this for a review.

Comment 2 Lubomir Kundrak 2007-11-23 11:20:05 UTC
* Fri Nov 23 2007  <furby fedoraproject.org> - 0.8.4-1

This is incorrect Changelog format, it needs your name, like:

* Fri Nov 23 2007 Chrobak Truhlik <furby> - 0.8.4-1

Comment 3 Lubomir Kundrak 2007-11-23 11:21:16 UTC
./configure

Any reasons for not using %configure macro?
If there are some, please add a comment.

Comment 4 Lubomir Kundrak 2007-11-23 11:22:00 UTC
%setup -n %{name}-%{version}-src

Setup should be quiet, like:

%setup -qn %{name}-%{version}-src

Comment 5 Lubomir Kundrak 2007-11-23 11:24:00 UTC
Group:          Utility/TextTools

This is not a valid group.
Please look into /usr/share/doc/rpm-[0-9]*/GROUPS for a list of valid groups

I'd suggest either Development/Tools or Applications/Text

Please note that none of current tools use groups anyway

Comment 6 Michal Fabry 2007-11-23 11:54:17 UTC
Spec file repaired and package rebuilded :)

Spec URL: http://michal.fabry.sk/data/fedora/webcpp/webcpp.spec
SRPM URL: http://michal.fabry.sk/data/fedora/webcpp/webcpp-0.8.4-2.fc9.src.rpm

Comment 7 Lubomir Kundrak 2007-11-23 11:58:32 UTC
Wonderful. RPMlint is all silent, and all outstanding issues have been
addressed. Builds just fine in mock.

APPROVED

Comment 8 Parag AN(पराग) 2007-11-23 12:34:36 UTC
lkundrak,
  Are you also interested to sponsor the package submitter?  I see that he needs
sponsor.

you may like to see http://fedoraproject.org/wiki/PackageMaintainers/SponsorProcess

Comment 9 Lubomir Kundrak 2007-11-23 13:10:58 UTC
Parag: I'd be glad to.

Comment 10 Parag AN(पराग) 2007-11-23 14:00:28 UTC
thanks.

Comment 11 Marek Mahut 2007-11-25 09:19:19 UTC
Parag,

But Lubomir is not a sponsor. 

 
https://admin.fedoraproject.org/accounts/dump-group.cgi?group=cvsextras&role_type=&format=html

Can you do it, if possible?

Comment 12 Parag AN(पराग) 2007-11-25 12:44:19 UTC
(In reply to comment #11)
> Parag,
> 
> But Lubomir is not a sponsor. 
> 
>  
>
https://admin.fedoraproject.org/accounts/dump-group.cgi?group=cvsextras&role_type=&format=html
> 
> Can you do it, if possible?

well, If you ask me for sponsorship then I need to check if package submitter 
has posted at least few preliminary reviews or not to other's package review
requests? So that I will come to know that package submitter is knowing well
about packaging.
 If he has done it already then I will be glad to sponsor him. But I saw he has
only single package submission and no preliminary reviews done. So I think he
should start doing that. Till then he need to wait from my side.

 

Comment 13 Michal Fabry 2008-03-07 09:25:56 UTC
New Package CVS Request
=======================
Package Name: webcpp
Short Description: Convert C++ code to HTML
Owners: furby
Branches: F-8
InitialCC: 
Cvsextras Commits: yes

Comment 14 Kevin Fenzi 2008-03-07 17:46:51 UTC
cvs done.

Comment 15 Lubomir Kundrak 2008-03-24 14:39:35 UTC
Michal: Close this with NEXTRELEASE in case you build the package.


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