pkgsrc-Bugs archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

pkg/55936: lang/ocaml : default safe string breaks certain builds

>Number:         55936
>Category:       pkg
>Synopsis:       lang/ocaml : default safe string breaks certain builds
>Confidential:   no
>Severity:       serious
>Priority:       high
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Mon Jan 18 06:45:00 +0000 2021
>Originator:     Aleksey Arens
>Release:        pkgsrc-2020Q4
NetBSD HOSTNAME_REDACTED 9.1_STABLE NetBSD 9.1_STABLE (GENERIC) #1: Sun Jan  3 20:58:50 UTC 2021  root@HOSTNAME_REDACTED:/usr/src/sys/arch/amd64/compile/obj/GENERIC amd64
The ocaml compiler built from lang/ocaml has safe strings forced by default.  This makes the `-usafe-string' compiler command line option unavailable.  The lack of this option, in particular, breaks sysutils/xentools413.

Cf. and for more info on the respective upstream change.


A very basic fix is to add an additional `configure' argument, added upstream specifically for the purpose of evading this breaking change.

I suppose that encoding of this option is a reasonable solution, because strings are safe by default anyway.

For the longer term, I would suggest to consider keeping this change for a couple of releases, and adding a patch to sysutils/xentools413 to fix the issue.  Ocaml already has mutable strings (as `Bytes') defined in its standard library under a different module, so the amount of changes to xentools413 should not be extensive.


RCS file: /cvsroot/pkgsrc/lang/ocaml/Makefile,v
retrieving revision 1.141
diff -u -r1.141 Makefile
--- Makefile	26 Dec 2020 08:36:18 -0000	1.141
+++ Makefile	18 Jan 2021 06:25:42 -0000
@@ -14,6 +14,8 @@
+CONFIGURE_ARGS+=	--disable-force-safe-string
 PLIST_VARS+=	ocaml-stub ocaml-prof ocaml-opt ocaml-nat ocaml-spacetime \

Home | Main Index | Thread Index | Old Index