opam-repository icon indicating copy to clipboard operation
opam-repository copied to clipboard

[new release] argon2 (1.0.2)

Open Khady opened this issue 2 years ago • 4 comments

OCaml bindings to Argon2

Khady avatar Aug 25 '23 08:08 Khady

this seems to be missing a depext?

samoht avatar Aug 29 '23 08:08 samoht

Hi! Any way we can help out to get this merge? It resolves not only the issue with argon2 not working on OS X but also with Nix

MarcCoquand avatar Jan 09 '24 18:01 MarcCoquand

I’ve added the depexts for the systems I know about

Khady avatar Jan 12 '24 01:01 Khady

There seems to be linking issues though:

#=== ERROR while compiling FPauth-strategies.1.0.0 ============================#
# context              2.2.0~alpha4~dev | linux/x86_64 | ocaml-base-compiler.4.14.1 | file:///home/opam/opam-repository
# path                 ~/.opam/4.14/.opam-switch/build/FPauth-strategies.1.0.0
# command              ~/.opam/opam-init/hooks/sandbox.sh build dune runtest -p FPauth-strategies -j 255
# exit-code            1
# env-file             ~/.opam/log/FPauth-strategies-7-dd1073.env
# output-file          ~/.opam/log/FPauth-strategies-7-dd1073.out
### output ###
# File "test/strategies/dune", line 2, characters 7-11:
# 2 |  (name unit)
#            ^^^^
# (cd _build/default/test/strategies && ./unit.exe)
# Fatal error: exception Dl.DL_error("./unit.exe: undefined symbol: argon2_type2string")
# Raised at Dl._report_dl_error in file "src/ctypes-foreign/dl.ml.unix", line 44, characters 20-44
# Called from Ctypes_foreign_basis.Make.foreign in file "src/ctypes-foreign/ctypes_foreign_basis.ml", line 47, characters 19-47
# Re-raised at Ctypes_foreign_basis.Make.foreign in file "src/ctypes-foreign/ctypes_foreign_basis.ml", line 49, characters 50-59
# Called from Argon2.Kind.argon2_type2string in file "src/argon2.ml", line 18, characters 4-98

mseri avatar Jan 24 '24 22:01 mseri

@Khady could you have a look at the linking issue?

# Fatal error: exception Dl.DL_error("./unit.exe: undefined symbol: argon2_type2string")

Is this expected?

samoht avatar Apr 02 '24 11:04 samoht

I'm merging the PR anyway as it seems to fix user issues (from @MarcCoquand report). But feel free to investigate why one reverse dependency is broken :-)

samoht avatar Apr 02 '24 11:04 samoht

the linking issue is not expected. It is being tracked in https://github.com/Khady/ocaml-argon2/issues/13

Khady avatar Apr 03 '24 02:04 Khady

may the linking issue be related / identical to https://discuss.ocaml.org/t/how-to-properly-define-a-library-with-c-bindings/14419/5 ? (I've no clue, and am not sure whether argon uses ctypes)

hannesm avatar Apr 03 '24 10:04 hannesm