Re: pkg/44991: [maintainer update] devel/ocaml-lwt to 2.3.0

[Thomas Klausner <wiz%NetBSD.org@localhost>]

The following reply was made to PR pkg/44991; it has been noted by GNATS.

From: Thomas Klausner <wiz%NetBSD.org@localhost>
To: NetBSD bugtracking <gnats-bugs%NetBSD.org@localhost>
Cc: 
Subject: Re: pkg/44991: [maintainer update] devel/ocaml-lwt to 2.3.0
Date: Sun, 29 May 2011 15:25:15 +0200

 On Tue, May 24, 2011 at 02:15:01PM +0000, jaapb%kerguelen.org@localhost wrote:
 > Synopsis:       [maintainer update] devel/ocaml-lwt to 2.3.0
 ...
 > This is an update of ocaml-lwt to the latest stable version. Changes
 > include:
 > - addition of many modules for standard applications (I/O, OS interaction, 
 > ...)
 > - integration with the OCaml toplevel
 > - generation of backtraces
 > - extensions to enable replacing libev with something else, if wanted
 > - and of course all sorts of improvements, optimisations and bugfixes
 > 
 > The port fully supports DESTDIR.
 
 Thank you for the update. I unpacked it and ran pkglint, which
 complains:
 ERROR: Makefile:16: "../../devel/ocaml-react" does not exist.
 ERROR: Makefile:16: There is no package in devel/ocaml-react.
 WARN: Makefile:50: Please use AUTO_MKDIRS instead of ${MKDIR}.
 WARN: options.mk:4: Unknown option "toplevel".
 WARN: patches/patch-myocamlbuild.ml:3: Comment expected.
 WARN: patches/patch-setup.ml:3: Comment expected.
 
 I looked at the Makefile, and
         ${RM} -rf ${DESTDIR}${PREFIX}/lib/ocaml/site-lib
 is a bit harsh -- if you install without destdir, this blows away any
 other files installed there. Are you sure you want this?
 
 Please try to fix the other pkglint warnings as well, especially the
 dependency problem :)
 
 Thank you,
  Thomas