Arch Linux

I am trying to build this one, and it turns out that camlp4 is a required dependency, so please add that to the dependency list :)

Alright guys, then there must've been something wrong on my system. I've now uploaded a version with staticlibs, hope that solves the issue. Thanks for letting me know!

Whoops, my bad, the patch is in fact necessary for camomileLibrary.a; but, I'm not seeing any of the *.a files installed without the staticlibs option.

Err, I'm noticing what kythyria is. The patch does nothing. Instead, adding "staticlibs" to the options solves the missing .a files issue.

kythyria, but the .a files are included in the package already, /usr/lib/ocaml/camomile/camomile.a and /usr/lib/ocaml/camomile/camomileLibrary.a.

You need to add staticlibs to the options array, otherwise the .a files get eaten patch or no (and liquidsoap-git won't compile)

Sure thing, just updated the PKGBUILD. Thanks for notifying!

Unfortunately the upstream build fails to install "camomileLibrary.a" which is needed for building certain software against this.

At least two other distributions apply patches to fix this:

Slackware: http://slackbuilds.org/slackbuilds/14.1/libraries/ocaml-camomile/0002-Install-missing-camomileLibrary.a.patch
Debian: http://patch-tracker.debian.org/patch/series/view/camomile/0.8.4-2/0002-Install-missing-camomileLibrary.a.patch

Would you mind adding this patch to the build?

Thanks!

Ok, perfect then.

Namcap does report OCAML as not needed as a dependency. I use the Liquidsoap package which requires ocaml-camomile and which works perfectly without ocaml.

Moved them both to makedepends. Not sure if this is a good idea in the case of ocaml, but oh well, if someone will complain it can be still moved back.

Could you please move ocaml and ocaml-findlib to the builddepends rather than depends?

This is an error:

> sed -ie 's|ocamlfind ...

The -i flag interprets the e as a backup extension. What you want is either "sed -i -e 's|ocamlfind ..." or "sed -i.orig -e 's|ocamlfind ..."
or "sed -i 's|ocamlfind ..." with no -e (since an -e is inferred when only one command expression is supplied.)

That said, this error is harmless because sed is assuming no -e is present, which it happily accepts. And creating a backup file "Makefilee" isn't hurting anyone.

But I thought I should point it out because this mistake might bite the maintainer another time.

Disowned.