Skip to content

Commit

Permalink
ctypes: depend explicitly on ctypes.foreign
Browse files Browse the repository at this point in the history
In ctypes < 0.21.0, the ctypes and ctypes.foreign libraries were
installed in the same directory, so depending on one would make the
other one visible. ctypes 0.21.0 installs them in different
directories so this makes it an error. tgls actually uses
ctypes.foreign so this adds the dependency.

Closes #29.
  • Loading branch information
emillon authored and dbuenzli committed Jul 17, 2023
1 parent 56c3357 commit eda88cf
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion CHANGES.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@


* Fix build system. Explicitely depend on `ctypes.foreign`.
Thanks to Etienne Millon for the patch (#29).
* Fix `Gl.debug_message_callback` raising `Ffi_stubs.CallToExpiredClosure`.
Thanks to Edwin Török for the report and the patch (#6).

Expand Down
2 changes: 1 addition & 1 deletion _tags
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ true: thread, bin_annot, safe_string
<support/**> : package(xmlm)

<src> : include
<src/tgl{3,4,es2,es3}.{ml,mli}> : package(ctypes)
<src/tgl{3,4,es2,es3}.{ml,mli}> : package(ctypes), package(ctypes.foreign)
<src/tgl{3,4,es2,es3}.{cma,cmxa,cmxs}> : custom, thread, \
package(ctypes), package(ctypes.foreign)
<src/tgl{3,4,es2,es3}_top.*> : package(compiler-libs.toplevel)
Expand Down

0 comments on commit eda88cf

Please sign in to comment.