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 <vnorrman@cc.hut.fi> 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.

Vesa