On Wed, Jul 06, 2016 at 06:15:32PM +1200, [email protected] wrote:
> Hi ports@,
> 
> Please help to review and test the port. Maybe someone is ready to
> import it into CVS?
> 

the port is fairly complete. Just some points:
  - patches/patch-mlton_Makefile
    "PREFIX has to be /usr/local for OpenBSD"

    in fact, "/usr/local" shouldn't be directly used: you should use
    LOCALBASE variable (default to /usr/local)

  - patches/patch-runtime_platform_openbsd_c
    "This has been added into the development branch of MLton"

    could you add a link to the commit in the description ?

Thanks.
-- 
Sebastien Marie

Reply via email to