Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 88697 - Request for cgilib (dependency of upcoming rrdtool release)
Summary: Request for cgilib (dependency of upcoming rrdtool release)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: High enhancement
Assignee: Gentoo Netmon project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 88698
  Show dependency tree
 
Reported: 2005-04-11 03:36 UTC by Frederic Mangeant
Modified: 2005-06-25 14:19 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Ebuild (cgilib-0.5.ebuild,537 bytes, text/plain)
2005-04-11 03:36 UTC, Frederic Mangeant
Details
Ebuild (corrected) (cgilib-0.5.ebuild,516 bytes, text/plain)
2005-04-11 05:21 UTC, Frederic Mangeant
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Frederic Mangeant 2005-04-11 03:36:20 UTC
Here's an ebuild for cgilib-0.5, which is required to build soon to come rrdtool 1.2

Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Comment 1 Frederic Mangeant 2005-04-11 03:36:53 UTC
Created attachment 55972 [details]
Ebuild
Comment 2 Aaron Walker (RETIRED) gentoo-dev 2005-04-11 04:31:16 UTC
Frederic, a few comments regarding your ebuild.

 - src_unpack() is unnecessary since it doesn't do anything.
 - there are some missing variables (SLOT and IUSE).
 - probably want to use dolib.a over dolib for static archives.

If you haven't already checkout the Gentoo Developer Handbook (http://www.gentoo.org/proj/en/devrel/handbook/handbook.xml).

Besides the above-mentioned issues, the ebuild looks great.  Can you attach an updated version?

Thanks.
Comment 3 Frederic Mangeant 2005-04-11 05:21:14 UTC
Created attachment 55982 [details]
Ebuild (corrected)

Ebuild corrected :
- src_unpack() was removed
- SLOT and IUSE variables were added
- dolib.a used instead of dolib
Comment 4 Aaron Walker (RETIRED) gentoo-dev 2005-04-12 05:02:16 UTC
Ok, a few other things that I should've noticed the first go, but didn't:

KEYWORDS on new packages (or version bumps) must be ~arch and only for those archs on which you have actually tested.  Because this is a dep of the new rrdtool, I have to CC all archs for which rrdtool is KEYWORD'd so that they can test it and mark it ~arch accordingly.

The second thing is that the inherit eutils is unnecessary since there's nothing in the ebuild from eutils.

Don't worry about attaching an updated ebuild as I have already fixed them and am about to commit.
Comment 5 Aaron Walker (RETIRED) gentoo-dev 2005-04-12 05:15:11 UTC
In CVS.  CC'd archs: this package is a dependency of the latest version of net-analyzer/rrdtool (see bug 88698).  Please test and mark ~arch.
Comment 6 Joe Jezak (RETIRED) gentoo-dev 2005-04-12 20:17:11 UTC
cookies.c:23:17: cgi.h: No such file or directory

^^ Is there a dependancy missing?  Compilation fails looking for this.
Comment 7 Aaron Walker (RETIRED) gentoo-dev 2005-04-13 07:24:02 UTC
Oops, no I used emake CFLAGS=${CFLAGS} which overrode the -I. in the Makefile's CFLAGS.  'Tis fixed.  Sorry about that.
Comment 8 Joe Jezak (RETIRED) gentoo-dev 2005-04-13 17:12:52 UTC
Marked ~ppc.
Comment 9 Jason Wever (RETIRED) gentoo-dev 2005-04-17 10:14:58 UTC
It builds on sparc.  Do we have any test cases to confirm functionality?
Comment 10 Aaron Walker (RETIRED) gentoo-dev 2005-04-17 10:36:55 UTC
> It builds on sparc.  Do we have any test cases to confirm functionality?

The latest rrdtool uses it (which of course I can't commit until everyone keywords it).  The best I can give you for a test case is the ebuild in my overlay, http://butsugenjitemple.org/~ka0ttic/overlay/net-analyzer/rrdtool/.
Comment 11 Jason Wever (RETIRED) gentoo-dev 2005-04-17 11:24:00 UTC
Unfortunately that doesn't build for me with the tcltk useflag enabled as tclrrd.c cannot find rrd_tool.h and rrd_format.h when compiling.
Comment 12 Frederic Mangeant 2005-04-18 07:18:37 UTC
I've made another rrdtool 1.2.0-rc8 ebuild here : http://bugs.gentoo.org/show_bug.cgi?id=88698

- rd_tool.h and rrd_format.h are located in $(top_srcdir)
- librrd_private.a doesn't seem to be needed anymore

It builds fine with USE="tcltk"

Regards,

Fr
Comment 13 Frederic Mangeant 2005-04-18 07:18:37 UTC
I've made another rrdtool 1.2.0-rc8 ebuild here : http://bugs.gentoo.org/show_bug.cgi?id=88698

- rd_tool.h and rrd_format.h are located in $(top_srcdir)
- librrd_private.a doesn't seem to be needed anymore

It builds fine with USE="tcltk"

Regards,

Frédéric
Comment 14 Jason Wever (RETIRED) gentoo-dev 2005-04-20 18:42:37 UTC
New ebuild worked well, cgilib is now ~sparc'd.
Comment 15 SpanKY gentoo-dev 2005-04-24 02:35:01 UTC
amd64/hppa/ia64 marked
Comment 16 Bryan Østergaard (RETIRED) gentoo-dev 2005-04-24 13:59:02 UTC
~alpha keyworded.
Comment 17 Frederic Mangeant 2005-04-26 01:34:51 UTC
Hi all

rrdtool 1.2.0 was released yesterday : http://people.ee.ethz.ch/~oetiker/webtools/rrdtool/newin12.txt

I've submitted an ebuild for it : http://bugs.gentoo.org/show_bug.cgi?id=88698
Comment 18 Daniel Webert 2005-06-03 15:35:41 UTC
can we close this? because cgilib 0.5 is in portage ...
Comment 19 Marcelo Goes (RETIRED) gentoo-dev 2005-06-25 14:19:42 UTC
Already in Portage.