Subject: Re: CVS commit: pkgsrc/lang/smlnj
To: None <>
From: Vesa Norrman <>
List: tech-pkg
Date: 06/06/2004 22:44:50
Christopher Richards wrote:
> Greetings from the lang/smlnj maintainer.
> On 9 Apr 2004 23:19:37 GMT, Soren Jacobsen wrote:
>>Modified Files:
>>	pkgsrc/lang/smlnj: Makefile PLIST distinfo
>>	pkgsrc/lang/smlnj/patches: patch-aa patch-al
>>Log Message:
>>Update to smlnj-110.45.
> Please revert this package back to version 110.42.  I ask because
> * the PR for this change, PR pkg/24586, was submitted over my
>   objection.  I objected because
> * the change breaks lang/twelf, and most other SML-based projects
>   I know of; and
> * the change duplicates the work of Vesa Norrman's smlnj-current
>   package in pkgsrc-wip.
> In summary: Please revert smlnj to 110.42; and please talk to Vesa
> Norrman <> about integrating this change with
> smlnj-current, and (if necessary) importing smlnj-current into
> pkgsrc proper.

It would be nice if wip/smlnj-current is imported. I think any one 
writing new programs should use that version and not the old one. 
(Isabelle is now configured to the new version also.)

Wip/smlnj-current is working fine so it could be imported.

However, there's a large set of patches which is in two places.

Do you think the patches controlling CFLAGS variable are necessary?
I think it has no benefit and makes maintenance more difficult.