Bug 586291 - Review Request: cURLpp - - C++ wrapper for libcURL
Summary: Review Request: cURLpp - - C++ wrapper for libcURL
Status: CLOSED DUPLICATE of bug 723053
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL: http://www.curlpp.org/
Whiteboard:
Keywords:
Depends On:
Blocks: FE-DEADREVIEW
TreeView+ depends on / blocked
 
Reported: 2010-04-27 08:58 UTC by Devil Wang
Modified: 2013-10-19 14:42 UTC (History)
8 users (show)

(edit)
* It uses the C++ Standard Library structures instead of home made ones.
    * It is exception safe. (see this)
    * It is type safe.
Clone Of:
(edit)
Last Closed: 2010-07-12 03:38:05 UTC


Attachments (Terms of Use)

Description Devil Wang 2010-04-27 08:58:31 UTC
Spec URL: <http://dl.dropbox.com/u/6190834/Curlpp%20Package/curlpp.spec>
SRPM URL: <http://dl.dropbox.com/u/6190834/Curlpp%20Package/curlpp-0.7.3-1.fc13.src.rpm>

Description: 

libcURL is described as: a free and easy-to-use client-side URL  transfer library, supporting FTP, FTPS, HTTP, HTTPS, GOPHER, TELNET, DICT, FILE and LDAP. libcurl supports HTTPS certificates, HTTP POST, HTTP PUT, FTP uploading, kerberos, HTTP  form based upload, proxies, cookies, user+password authentication, file transfer resume, http proxy tunneling and more!

libcurl is highly portable, it builds and works identically on numerous platforms, including Solaris, NetBSD, FreeBSD, OpenBSD, Darwin, HPUX, IRIX, AIX, Tru64, Linux, Windows, Amiga, OS/2, BeOs, Mac OS X, Ultrix, QNX, OpenVMS, RISC OS, Novell NetWare, DOS and more…

libcurl is free, thread-safe, IPv6 compatible, feature rich, well supported and fast.

First, I need to quote Daniel Stenberg, the maintener of libcURL:
You can use libcURL instantly from within your C++ programs. You don’t need cURLpp for that, cURLpp just adds an OO’ified layer that libcURL doesn’t normally provide.

Comment 1 Devil Wang 2010-04-27 09:01:07 UTC
Technical note added. If any revisions are required, please edit the "Technical Notes" field
accordingly. All revisions will be proofread by the Engineering Content Services team.

New Contents:
* It uses the C++ Standard Library structures instead of home made ones.
    * It is exception safe. (see this)
    * It is type safe.

Comment 2 Caius Chance 2010-04-27 12:38:27 UTC
My informal review:

Line 4: First word did not capitalized.

Line 18: Typo: "Supporting"
Line 19: Over 80 characters of width.
Line 20: Double spaced between "HTTP" and "certificates".

Line 24: This paragraph is sort of irrelevant.

Line 42, 43:  Please put %post and %postun after %clean.

Comment 3 Caius Chance 2010-04-27 12:39:15 UTC
Please make all paragraphs more readable.

Comment 4 Ralf Corsepius 2010-04-27 12:55:07 UTC
Package doesn't build in mock:
...
checking for boostlib >= 1.33.1... configure: error: We could not detect the boost libraries (version 1.33 or higher).
...

Comment 5 Howard Ning 2010-04-28 12:13:42 UTC
Also, you should at least add COPYING in %doc. http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text

Comment 6 Ralf Corsepius 2010-05-07 10:23:15 UTC
Ping? Any update on this one?

Devil Wang, shouldn't we be hearing from you within one week, I'll close this review request.

Comment 7 Devil Wang 2010-05-07 10:44:55 UTC
(In reply to comment #6)
> Ping? Any update on this one?
> 
> Devil Wang, shouldn't we be hearing from you within one week, I'll close this
> review request.    


I was busy this week, i will update this package as soon as possible.

Comment 8 Ralf Corsepius 2010-05-07 11:20:23 UTC
Thank you for your promptly response. I am looking forward to hearing from you.

Comment 9 Ralf Corsepius 2010-05-29 04:07:36 UTC
Another 3 weeks without update have passed. Closing review as FAILED.

Devil Wang, should you or others be interested in submitting this package to Fedora, feel free to open a new review request.

Comment 10 Alex Lancaster 2010-07-12 03:38:05 UTC
Closing as NOTABUG as per: https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews

Comment 11 Susi Lehtola 2011-07-19 10:57:03 UTC

*** This bug has been marked as a duplicate of bug 723053 ***


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