Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 266426 - ebuild for yajl JSON parser C library
Summary: ebuild for yajl JSON parser C library
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement with 1 vote (vote)
Assignee: Default Assignee for New Packages
URL:
Whiteboard:
Keywords: EBUILD
Depends on:
Blocks:
 
Reported: 2009-04-16 20:45 UTC by Nick Leippe
Modified: 2011-01-14 15:00 UTC (History)
2 users (show)

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


Attachments
Here is a first crack at an ebuild to install the yajl JSON parser C library (yajl-1.0.2.ebuild,8.11 KB, text/plain)
2009-04-16 20:47 UTC, Nick Leippe
Details
patch for 64bit arches (64bit_use_int32_t.patch,5.65 KB, patch)
2009-04-16 20:48 UTC, Nick Leippe
Details | Diff
fix custom configure script so that --prefix argument is honored (configure_honor_prefix.patch,730 bytes, patch)
2009-04-16 20:49 UTC, Nick Leippe
Details | Diff
ebuild for yajl 1.0.9 using cmake (yajl-1.0.9.ebuild,986 bytes, text/plain)
2010-02-14 04:20 UTC, Robert G. Jakabosky
Details
patch for 1.0.9 ebuild to fix multilib support. (yajl-1.0.9_multilib.patch,1.63 KB, patch)
2010-02-14 04:21 UTC, Robert G. Jakabosky
Details | Diff
fix doxygen config. for 1.0.9 (yajl-1.0.9_fix_doc.patch,571 bytes, patch)
2010-02-14 04:22 UTC, Robert G. Jakabosky
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Leippe 2009-04-16 20:45:51 UTC
Found here: http://lloydforge.org/projects/yajl/



Reproducible: Always
Comment 1 Nick Leippe 2009-04-16 20:47:28 UTC
Created attachment 188608 [details]
Here is a first crack at an ebuild to install the yajl JSON parser C library

The author has written his own configure script in ruby, and uses cmake.
The script defaults prefix to /usr/local
I have a patch that fixes it to honor the --prefix flag to allow for --prefix /usr.
However, the "make all" target compiles, runs the tests, and installs everything to a subdir anyways, which I just move to ${D}/usr, since his "make install" target does not include the docs.

I have also written a patch for the source that makes it pass his integer.json test case.
This case fails on amd64 because in 32-bit it expects 2147483648 to cause an overflow error, which now doesn't since long int is 64 bits. The decision was thus:
a) allow 64-bit platforms to handle larger numerical values and change the testing to match
b) make sure that all platforms exhibit the exact same behavior
I think (b) is a better option, and probably consistent with javascript in browsers. I could find nothing in the rfc specifying numerical limits for json.
So, I chose (b) and patched the sources to use int32_t for integers.
I only apply the patch on 64 bit platforms.
Comment 2 Nick Leippe 2009-04-16 20:48:36 UTC
Created attachment 188609 [details, diff]
patch for 64bit arches

use int32_t on 64bit machines to make numerical limits behave the same as on 32bit
Comment 3 Nick Leippe 2009-04-16 20:49:17 UTC
Created attachment 188610 [details, diff]
fix custom configure script so that --prefix argument is honored

it works, but is not used by the ebuild atm
Comment 4 Serkan Kaba (RETIRED) gentoo-dev 2009-04-19 13:45:33 UTC
Removing Java
Comment 5 Robert G. Jakabosky 2010-02-14 04:20:27 UTC
Created attachment 219571 [details]
ebuild for yajl 1.0.9 using cmake

This is a ebuild for yajl 1.0.9 that uses CMake to build yajl.  It is based of the ebuild for dev-libs/qjson-0.7.1.

Two patch files to follow.
Comment 6 Robert G. Jakabosky 2010-02-14 04:21:39 UTC
Created attachment 219573 [details, diff]
patch for 1.0.9 ebuild to fix multilib support.

fix multilib support.  for verion 1.0.9
Comment 7 Robert G. Jakabosky 2010-02-14 04:22:13 UTC
Created attachment 219575 [details, diff]
fix doxygen config.  for 1.0.9
Comment 8 Markos Chandras (RETIRED) gentoo-dev 2010-06-04 12:51:06 UTC
FLameeyes pushed it on tree so this bug may close now

Ondrej, Diego ( flameeyes ) pushed yajl-1.0.9 on portage tree. I recommend you to talk to him to include the live ebuild from sunrise as well since as per policy, we do not allow packages from portage tree to be on sunrise as well.

Please let me know about your further actions. We can either drop the live ebuild from sunrise or talk to Diego and move it to portage 

Thanks
Comment 9 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2011-01-14 15:00:48 UTC
The live ebuild has been removed from Sunrise.