Skip to content

Commit

Permalink
Ensure installed stdlib artefacts have correct case
Browse files Browse the repository at this point in the history
  • Loading branch information
dra27 committed Mar 19, 2021
1 parent 1a59019 commit 6512c85
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 6 deletions.
11 changes: 6 additions & 5 deletions Changes
Expand Up @@ -114,11 +114,12 @@ Working version
- #9582: Add Array.{find_opt,find_map,split,combine}.
(Nicolás Ojeda Bär, review by Daniel Bünzli and Gabriel Scherer)

* #10169, #10270: Use capitalized module names in the Standard Library prefixing
scheme to match Dune, e.g. Stdlib__String instead of Stdlib__string. This is a
breaking change only to code which attempted to use the internal names before.
The Standard Library generated by the Dune rules is now equivalent to the main
build (the Dune rules still do not generate a distributable compiler).
* #10169, #10270, #10301: Use capitalized module names in the Standard Library
prefixing scheme to match Dune, e.g. Stdlib__String instead of Stdlib__string.
This is a breaking change only to code which attempted to use the internal
names before. The Standard Library generated by the Dune rules is now
equivalent to the main build (the Dune rules still do not generate a
distributable compiler).
(David Allsopp and Mark Shinwell, review by Gabriel Scherer)

### Other libraries:
Expand Down
10 changes: 9 additions & 1 deletion stdlib/Makefile
Expand Up @@ -56,6 +56,10 @@ allopt: stdlib.cmxa std_exit.cmx
opt.opt: allopt

.PHONY: install
# Ensure any pre-4.13 lowercased artefacts are removed on macOS and Windows
install::
rm -f $(INSTALL_LIBDIR)/stdlib__*.cmi

install::
$(INSTALL_DATA) \
stdlib.cma std_exit.cmo *.cmi camlheader_ur \
Expand All @@ -81,7 +85,11 @@ endif
installopt: installopt-default

.PHONY: installopt-default
installopt-default:
# Ensure any pre-4.13 lowercased artefacts are removed on macOS and Windows
installopt-default::
rm -f $(INSTALL_LIBDIR)/stdlib__*.cmx

installopt-default::
$(INSTALL_DATA) \
stdlib.cmxa stdlib.$(A) std_exit.$(O) *.cmx \
"$(INSTALL_LIBDIR)"
Expand Down

0 comments on commit 6512c85

Please sign in to comment.