Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 281274

Summary: [EBUILD] dev-ml/sexplib-4.2.12 new package
Product: Gentoo Linux Reporter: Guillaume Horel <guillaume.horel>
Component: New packagesAssignee: Default Assignee for New Packages <maintainer-wanted>
Status: RESOLVED FIXED    
Severity: normal CC: ml
Priority: High Keywords: EBUILD
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on: 281275    
Bug Blocks:    
Attachments: sexplib-4.2.12
sexplib 4.2.12 ebuild

Description Guillaume Horel 2009-08-13 03:49:29 UTC
sexplib-4.2.12 Library for automated conversion of OCaml-values to and from S-expressions required for batteries, the new standard library for ocaml

Reproducible: Always
Comment 1 Guillaume Horel 2009-08-13 03:50:30 UTC
Created attachment 201098 [details]
sexplib-4.2.12
Comment 2 Alexis Ballier gentoo-dev 2009-08-19 08:29:06 UTC
Thanks for the ebuild.

A few remarks:
- please run repoman on the ebuild and fix the errors it rants about
- there seems to be a missing dep on type-conv
- it would be great if there was support for bytecode only, otherwise you need to depend on ocaml[ocamlopt], have a look at other ocaml ebuild to see how its done.
Comment 3 Guillaume Horel 2009-08-19 23:47:03 UTC
Created attachment 201750 [details]
sexplib 4.2.12 ebuild

I added the dependency to type-conv, and the ocamlopt use flag requirement.
It would be possible to build the byte-code only, but there is no easy option do disable native in the Makefile, that would require some patching there I think.

Also repoman should be happier now, except for forcing the emake -j1 (but parallel build fail).
Comment 4 Alexis Ballier gentoo-dev 2009-09-19 12:33:18 UTC
added, thanks. Again with minor tweaks.
Comment 5 Guillaume Horel 2009-09-20 15:31:47 UTC
(In reply to comment #4)
> added, thanks. Again with minor tweaks.
> 

Thanks! Can you also keyword it ~x86? I have tested and used it there, works fine.
Comment 6 Alexis Ballier gentoo-dev 2009-09-20 15:34:49 UTC
(In reply to comment #5)
> (In reply to comment #4)
> > added, thanks. Again with minor tweaks.
> > 
> 
> Thanks! Can you also keyword it ~x86? I have tested and used it there, works
> fine.

i cannot really do that myself since i dont have a 32bits x86, please file a bug for the x86 team and they'll do it.