[OMake] Announcing OMake 0.9.8.5 - a substantial feature enhancements release

[Jason Hickey]

Oops, here is a patch for this one too.  The problem is that the name  
of the scanner should be scan-ocaml-reins.mli with a .mli suffix,  
not .cmi.

I also committed this (and actually tested it myself too:)

Jason

Index: lib/build/OCaml.om
===================================================================
--- lib/build/OCaml.om  (revision 11946)
+++ lib/build/OCaml.om  (working copy)
@@ -883,7 +883,7 @@
     $(CMI):
        section rule
           if $(target-exists $(MLI))
-            $(CMI): $(MLI) :scanner: scan-ocaml-%
+            $(CMI): $(MLI) :scanner: scan-ocaml-$(basename $(name)).mli
                  $(OCamlC) -c $<
           elseif $(NATIVE_ENABLED)
              $(NATIVE_TARGETS) $(CMI): $(NATIVE_DEPS)

On Aug 9, 2007, at 1:56 PM, Mike Furr wrote:

> Jason Hickey wrote:
>> Mike,
>> I've checked in an update to 0.9.8.5 that should fix these  
>> problems.  Can you try it when you get a chance?
>> The problem with ocamldep-omake came up in the switch to OCaml  
>> 3.10, and the problem with OCamlPackage is because, well, we never  
>> tested it:(
>> Thanks!  I've attached a patch too (for OCaml.om).
> I applied this patch (and added a space on line 295 to make the  
> indentation line up) and it seems to fix the problems with my  
> current setup (thanks!).  However, it still does not allow me to  
> supply an .mli for the packed module.
>
> *** omake error:
>    File /home/furrm/om/dist/lib/omake/build/OCaml.om: line 883,  
> characters 4-11
>    Do not know how to build "<scanner src/scan-ocaml-reins.cmi>"  
> required for "src/reins.cmi"
>
> To recap, this is when I have the rule
>
>  $(OCamlProgram reins, $(FILES))
>
> and reins.mli exists.
>
> Cheers,
> -Mike
> _______________________________________________
> Omake mailing list
> Omake at metaprl.org
> https://lists.metaprl.org/mailman/listinfo/omake

--
Jason Hickey                  http://www.cs.caltech.edu/~jyh
Caltech Computer Science      Tel: 626-395-6568 FAX: 626-792-4257