Bug 253941 - Review Request: cgi-util - A C library for creating Common Gateway Interface ("CGI") programs
Summary: Review Request: cgi-util - A C library for creating Common Gateway Interface ...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-08-23 01:52 UTC by W. Michael Petullo
Modified: 2007-11-30 22:12 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-09-25 23:56:52 UTC
Type: ---
Embargoed:
mtasaka: fedora-review+


Attachments (Terms of Use)

Description W. Michael Petullo 2007-08-23 01:52:22 UTC
Spec URL: http://flyn.org/SRPMS/cgi-util.spec
SRPM URL: http://flyn.org/SRPMS/cgi-util-2.2.1-9.src.rpm
Description:
cgi-util is a library which provides a set of C functions you can
use to create Common Gateway Interface ("CGI") programs.  Simply call
cgi-util's initialization function and send HTML (or any other kind of
data) out from your program and to the user's web browser.

This package was orphaned more than six months ago.

Comment 1 W. Michael Petullo 2007-08-24 00:20:20 UTC
Spec URL: http://flyn.org/SRPMS/cgi-util.spec
SRPM URL: http://flyn.org/SRPMS/cgi-util-2.2.1-10.src.rpm

Modified the autotools patch.

Comment 2 Mamoru TASAKA 2007-09-20 12:23:53 UTC
For 2.2.1-10:

* autotools patch
  - Please don't use a meaninglessly large patch like
    Patch0.
    This type of patch makes it very difficult to for us to
    check what points are to be fixed by this patch.
    * Apply patch for the points that are really needed fixing
      and really call autotools if needed.
    * And please check it carefully if autotools really has
      to be called.

* disttag
  - Please consider to use disttag.
    http://fedoraproject.org/wiki/Packaging/DistTag

* License
  - License should be "LGPLv2+" as no source specifies the version.

* Macro style
  - Please choose whether you use %name or %{name}

* rpmlint
-------------------------------------------------------
cgi-util.i386: W: spurious-executable-perm /usr/share/doc/cgi-util-2.2.1/test.cgi
cgi-util.i386: W: spurious-executable-perm
/usr/share/doc/cgi-util-2.2.1/filetest.cgi
-------------------------------------------------------
  - First of all, these two files are actually wrong because
    these files uses non-existing builddir.
    i.e. These two files can work at build time and no longer
         works when installed as this
  - And the permission should be 0644 as this is marked as documents.


Comment 3 W. Michael Petullo 2007-09-22 02:26:20 UTC
Spec URL: http://flyn.org/SRPMS/cgi-util.spec
SRPM URL: http://www.flyn.org/SRPMS/cgi-util-2.2.1-12.fc8.src.rpm

Comment 4 Mamoru TASAKA 2007-09-22 11:40:07 UTC
Well, -12 srpm contains some problems.

* Builds does not honor Fedora specific compilation flags.
* libcgi-util.so does not have soname
--------------------------------------------------
gcc -g -Wall -ansi -pedantic -fPIC cgi-util.c -c -o libcgi-util.o
gcc -shared libcgi-util.o -o libcgi-util.so
--------------------------------------------------
  You must use linker option.
* Permission of the library is not right
--------------------------------------------------
+ /usr/bin/install -D -m 0644 libcgi-util.so
/var/tmp/cgi-util-2.2.1-12.fc8-root-kojibuilder/usr/lib/libcgi-util.so.2.2.1
--------------------------------------------------
  Permission should be 0755.
* And for symlink:
--------------------------------------------------
+ ln -s /usr/lib/libcgi-util.so.2.2.1
/var/tmp/cgi-util-2.2.1-12.fc8-root-kojibuilder/usr/lib/libcgi-util.so
--------------------------------------------------
  This symlink must be relative.

Comment 5 W. Michael Petullo 2007-09-22 20:41:41 UTC
Spec URL: http://flyn.org/SRPMS/cgi-util.spec
SRPM URL: http://www.flyn.org/SRPMS/cgi-util-2.2.1-13.fc8.src.rpm

Comment 6 Mamoru TASAKA 2007-09-23 02:22:17 UTC
For -13:

------------------------------------------------------
%{__ld} -shared -soname=libcgi-util.so.2 libcgi-util.o -o libcgi-util.so
------------------------------------------------------
  - This cannot be accepted as
    http://koji.fedoraproject.org/koji/taskinfo?taskID=170767

    Please check:
    http://www.redhat.com/archives/rhl-devel-list/2007-August/msg00481.html

Comment 7 W. Michael Petullo 2007-09-23 03:25:30 UTC
Spec URL: http://flyn.org/SRPMS/cgi-util.spec
SRPM URL: http://www.flyn.org/SRPMS/cgi-util-2.2.1-14.fc8.src.rpm

Comment 8 Mamoru TASAKA 2007-09-23 06:13:18 UTC
For -14:

* To keep timestamp, please use "-p" option when using
  "install" or "cp" commands

* From rpmlint:
-------------------------------------------------
[tasaka1@localhost cgi-util]$ rpmlint *-14*rpm
cgi-util.i386: E: no-ldconfig-symlink /usr/lib/libcgi-util.so.2.2.1
[tasaka1@localhost cgi-util]$ rpmlint -I no-ldconfig-symlink
no-ldconfig-symlink :
The package should not only include the shared library itself, but
also the symbolic link which ldconfig would produce. (This is
necessary, so that the link gets removed by rpm automatically when
the package gets removed, even if for some reason ldconfig would not be
run at package postinstall phase.)
-------------------------------------------------
    In short, symlink "libcgi-util.so.2" is missing.

Comment 9 W. Michael Petullo 2007-09-23 18:25:59 UTC
Spec URL: http://flyn.org/SRPMS/cgi-util.spec
SRPM URL: http://www.flyn.org/SRPMS/cgi-util-2.2.1-15.fc8.src.rpm

Comment 10 Mamoru TASAKA 2007-09-24 12:40:53 UTC
Okay.

-------------------------------------------------------
   This package (cgi-utils) is APPROVED by me
-------------------------------------------------------


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