[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Fedora-packaging] Ocaml sub-package issue

On Tue, 6 Oct 2009, Richard W.M. Jones wrote:

On Tue, Oct 06, 2009 at 12:43:13PM +0300, Panu Matilainen wrote:
On Sun, 4 Oct 2009, Richard W.M. Jones wrote:

On Sun, Oct 04, 2009 at 11:03:33AM +0300, Panu Matilainen wrote:
I seem to recall commenting on this earlier (like at least a year ago :)
but just as well I might've just intended to comment... anyway.
I would've pulled it to RPM upstream ages ago if it didn't have Fedora
packaging specifics bolted in:

if [ -n "$emit_compiler_version" ]; then
    # Every OCaml program depends on the version of the
    # runtime which was used to compile it.
    echo "ocaml(runtime) = `cat /usr/lib*/ocaml/fedora-ocaml-release`"

I suppose there's a reason why this is not generated with "ocaml
-version" or "ocamlrun -version" as needed?

Yes, I can't remember why we did that either :-(

I've changed it in the OCaml package in Fedora 13, we should give it a
few more months to see if anything breaks.


Okay... added upstream now and integrated with the internal dependency
generator too so you dont need to fiddle with that in specfile, plus
the external dependency generator misses some fairly important
dependency bits.


Just a question about this: If we need to pass extra options to the
dependency scripts (as in the example specfile below), can we still do

For now, you can override the macros much like before, the only difference being the macro to override: instead of %__find_requires and %__find_provides, you'd now override %__ocaml_requires and %__ocaml_provides.

Another, perhaps nicer option would be adding per-script optional option-macros for these, something like:

%__ocaml_provides %{_rpmconfigdir}/ocaml-find-provides.sh %{?ocaml_find_provides_opts}

So in spec you'd just have to set the provides/requires options if necessary, instead of redefining the entire thing. Similar mechanism could be useful for other similar dep extractors too.

	- Panu -

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]