Skip to content

Fix misleading comment in Makefile.example #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
whitequark wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix misleading comment in Makefile.example #3

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions Makefile.examples
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,14 +76,14 @@ build_standalone_byt:
# Link our "OCaml universe" (OCaml code + runtime system) "statically" into a DLL
$(OCAMLC) -I $(CSMLDIR) $(OCAMLFLAGS) -output-obj -o lib$(NAME)_ml_byt$(EXT_DLL) -linkall $(CSMLMLLIBS) $(MLCODE)
#
# csml_byt.dll expects to find the csml C/OCaml runtime in csml_ml_byt.dll.
# CSML binding code expects to find the csml C/OCaml runtime in csml_ml_byt.dll.
# But now this runtime is in $(NAME)_ml_byt.dll, so we need to make
# a custom version of csml_byt.dll that points to this DLL instead.
# a custom version of the binding code that points to this DLL instead.
# The first step is to build a small C# file that mentions this DLL name.
$(CSML) -dllbind lib$(NAME)_ml_byt > $(NAME)_ml_byt.cs
#
# Now we can build the DLL itself. While we are at it, we can put the
# C# code of our application directly in the same DLL as well.
# Now we can build the application executable itself. While we are at it, we can put the
# C# binding code directly in the application executable.
$(CSC) /out:$(NAME)_standalone_byt.exe $(NAME)_ml_byt.cs $(CSMLCSCODE) $(CSCODE)


Expand Down