Bug 427458 - Review Request: rudecgi - Library (C++ API) for reading CGI form data
Summary: Review Request: rudecgi - Library (C++ API) for reading CGI form data
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: panelfm
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michael Schwendt
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-01-03 22:34 UTC by Matt Flood
Modified: 2008-01-12 00:16 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-01-12 00:16:01 UTC
Type: ---
Embargoed:
kevin: fedora-cvs+


Attachments (Terms of Use)
patch to compile with g++ 4.3.0 (2.57 KB, patch)
2008-01-10 10:25 UTC, Michael Schwendt
no flags Details | Diff

Description Matt Flood 2008-01-03 22:34:32 UTC
Spec URL: http://homeless.fedorapeople.org/rudecgi.spec
SRPM URL: http://homeless.fedorapeople.org/rudecgi-5.0.0-1.src.rpm
Description: Library (C++ API) that allows CGI applications to access form data
submitted to a webserver using GET and POST methods.  It also extracts
path info and cookie data from the environment and supports file uploads.

Comment 1 Michael Schwendt 2008-01-10 10:25:26 UTC
Created attachment 291261 [details]
patch to compile with g++ 4.3.0

$ rpmlint rudecgi-5.0.0-1.src.rpm 
rudecgi.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 67)
rudecgi.src: W: invalid-license GPL

GPLv2+	is the licence according to your source code.

Minor issues => EasyFix. APPROVED.

Comment 2 Matt Flood 2008-01-11 16:48:56 UTC
Thanks for the review and the patch, I have made the requested changes.

Spec URL: http://homeless.fedorapeople.org/rudecgi.spec
SRPM URL: http://homeless.fedorapeople.org/rudecgi-5.1.0-1.src.rpm


New Package CVS Request
=======================
Package Name: rudecgi
Short Description: Library (C++ API) for reading CGI form data
Owners: homeless
Branches: F-7 F-8
Cvsextras Commits: yes



Comment 3 Kevin Fenzi 2008-01-11 21:49:32 UTC
cvs done.


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