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

new file: packages/orf/orf.1.0.1/opam

Open UnixJunkie opened this issue 2 years ago • 5 comments

compiles w/ ocaml-5 and many other versions probably

UnixJunkie avatar Jan 05 '23 04:01 UnixJunkie

The 1.0.1 tag/archive doesn’t exist

kit-ty-kate avatar Jan 09 '23 16:01 kit-ty-kate

Sorry, now it does; forgot to push the tags.

UnixJunkie avatar Jan 10 '23 00:01 UnixJunkie

There are lower bound issues re: Seq:


#=== ERROR while compiling orf.1.0.1 ==========================================#
# context              2.2.0~alpha~dev | linux/x86_64 | ocaml-base-compiler.4.11.2 | pinned(https://github.com/UnixJunkie/orf/archive/refs/tags/v1.0.1.tar.gz)
# path                 ~/.opam/4.11/.opam-switch/build/orf.1.0.1
# command              ~/.opam/opam-init/hooks/sandbox.sh build dune build -p orf -j 31
# exit-code            1
# env-file             ~/.opam/log/orf-7-009d12.env
# output-file          ~/.opam/log/orf-7-009d12.out
### output ###
# File "/home/opam/.opam/4.11/lib/bst/bst.dune", line 1, characters 0-0:
# Warning: .dune files are ignored since 2.0. Reinstall the library with dune
# >= 2.0 to get rid of this warning and enable support for the subsystem this
# library provides.
#       ocamlc src/.orf.objs/byte/orf__RFC.{cmo,cmt} (exit 2)
# (cd _build/default && /home/opam/.opam/4.11/bin/ocamlc.opt -w -40 -g -bin-annot -I src/.orf.objs/byte -I src/.orf.objs/public_cmi -I /home/opam/.opam/4.11/lib/batteries -I /home/opam/.opam/4.11/lib/bytes -I /home/opam/.opam/4.11/lib/cpm -I /home/opam/.opam/4.11/lib/cpu -I /home/opam/.opam/4.11/lib/dolog -I /home/opam/.opam/4.11/lib/line_oriented -I /home/opam/.opam/4.11/lib/num -I /home/opam/.opam/4.11/lib/ocaml/threads -I /home/opam/.opam/4.11/lib/parany -intf-suffix .ml -no-alias-deps -open Orf -o src/.orf.objs/byte/orf__RFC.cmo -c -impl src/RFC.ml)
# File "src/RFC.ml", line 214, characters 24-37:
# 214 |                      ) (IntSet.to_seq values)
#                               ^^^^^^^^^^^^^
# Error: Unbound value IntSet.to_seq
#     ocamlopt src/.orf.objs/native/orf__RFC.{cmx,o} (exit 2)
# (cd _build/default && /home/opam/.opam/4.11/bin/ocamlopt.opt -w -40 -g -I src/.orf.objs/byte -I src/.orf.objs/native -I src/.orf.objs/public_cmi -I /home/opam/.opam/4.11/lib/batteries -I /home/opam/.opam/4.11/lib/bytes -I /home/opam/.opam/4.11/lib/cpm -I /home/opam/.opam/4.11/lib/cpu -I /home/opam/.opam/4.11/lib/dolog -I /home/opam/.opam/4.11/lib/line_oriented -I /home/opam/.opam/4.11/lib/num -I /home/opam/.opam/4.11/lib/ocaml/threads -I /home/opam/.opam/4.11/lib/parany -intf-suffix .ml -no-alias-deps -open Orf -o src/.orf.objs/native/orf__RFC.cmx -c -impl src/RFC.ml)
# File "src/RFC.ml", line 214, characters 24-37:
# 214 |                      ) (IntSet.to_seq values)
#                               ^^^^^^^^^^^^^
# Error: Unbound value IntSet.to_seq

You probably either need to depend on OCaml >= 4.12, or add a dependency to a stdlib compatibility package.


There are also some lower-bounds issues with a Cmp.Utls module


#=== ERROR while compiling orf.1.0.1 ==========================================#
# context              2.2.0~alpha~dev | linux/x86_64 | ocaml-base-compiler.4.14.1 | pinned(https://github.com/UnixJunkie/orf/archive/refs/tags/v1.0.1.tar.gz)
# path                 ~/.opam/4.14/.opam-switch/build/orf.1.0.1
# command              ~/.opam/opam-init/hooks/sandbox.sh build dune build -p orf -j 31
# exit-code            1
# env-file             ~/.opam/log/orf-7-2b7e59.env
# output-file          ~/.opam/log/orf-7-2b7e59.out
### output ###
# File "/home/opam/.opam/4.14/lib/bst/bst.dune", line 1, characters 0-0:
# Warning: .dune files are ignored since 2.0. Reinstall the library with dune
# >= 2.0 to get rid of this warning and enable support for the subsystem this
# library provides.
#       ocamlc src/.model.eobjs/byte/dune__exe__Model.{cmi,cmo,cmt} (exit 2)
# (cd _build/default && /home/opam/.opam/4.14/bin/ocamlc.opt -w -40 -g -bin-annot -I src/.model.eobjs/byte -I /home/opam/.opam/4.14/lib/batteries -I /home/opam/.opam/4.14/lib/bst -I /home/opam/.opam/4.14/lib/bytes -I /home/opam/.opam/4.14/lib/cpm -I /home/opam/.opam/4.14/lib/cpu -I /home/opam/.opam/4.14/lib/dolog -I /home/opam/.opam/4.14/lib/line_oriented -I /home/opam/.opam/4.14/lib/minicli -I /home/opam/.opam/4.14/lib/molenc -I /home/opam/.opam/4.14/lib/num -I /home/opam/.opam/4.14/lib/ocaml/threads -I /home/opam/.opam/4.14/lib/parany -I src/.orf.objs/public_cmi -no-alias-deps -open Dune__exe -o src/.model.eobjs/byte/dune__exe__Model.cmo -c -impl src/model.ml)
# File "src/model.ml", line 31, characters 14-31:
# 31 |   let folds = Cpm.Utls.cv_folds nfolds training_set in
#                    ^^^^^^^^^^^^^^^^^
# Error: Unbound module Cpm.Utls

I don't know where that comes from. You probably just need to add some lower bound constraint to whatever package pulls this in.

raphael-proust avatar Mar 29 '23 13:03 raphael-proust

@UnixJunkie would you mind taking a look at the issues above please, or closing this?

avsm avatar Jun 07 '23 18:06 avsm

Hi @UnixJunkie do you mind to revisit this PR?

If you want to go with the approach to add lower bounds, https://github.com/ocaml/opam-repository/wiki/Lower-bound-check---DYI might be useful.

haochenx avatar Sep 20 '23 13:09 haochenx

More lower bounds issues -- @UnixJunkie would you mind looking at those?

-> installed bst.3.0.0
-> installed conf-python-3.1.0.0
-> installed dune.2.8.0
-> installed ocamlbuild.0.14.0
-> installed ocamlfind.1.8.1
-> installed dolog.4.0.0
-> installed minicli.5.0.2
-> installed cpu.2.0.0
-> installed num.1.0
-> installed parany.11.0.2
-> installed parmap.1.0-rc11
-> installed batteries.3.3.0
-> installed line_oriented.1.0.0
-> installed cpm.11.0.0
-> installed molenc.5.0.0
[ERROR] The compilation of orf.1.0.1 failed at "dune build -p orf -j 255".

#=== ERROR while compiling orf.1.0.1 ==========================================#
# context              2.2.0~beta2~dev | linux/x86_64 | ocaml-base-compiler.4.12.1 | pinned(https://github.com/UnixJunkie/orf/archive/refs/tags/v1.0.1.tar.gz)
# path                 ~/.opam/4.12/.opam-switch/build/orf.1.0.1
# command              ~/.opam/opam-init/hooks/sandbox.sh build dune build -p orf -j 255
# exit-code            1
# env-file             ~/.opam/log/orf-7-bc5fe1.env
# output-file          ~/.opam/log/orf-7-bc5fe1.out
### output ###
# File "/home/opam/.opam/4.12/lib/bst/bst.dune", line 1, characters 0-0:
# Warning: .dune files are ignored since 2.0. Reinstall the library with dune
# >= 2.0 to get rid of this warning and enable support for the subsystem this
# library provides.
#       ocamlc src/.model.eobjs/byte/dune__exe__Model.{cmi,cmo,cmt} (exit 2)
# (cd _build/default && /home/opam/.opam/4.12/bin/ocamlc.opt -w -40 -g -bin-annot -I src/.model.eobjs/byte -I /home/opam/.opam/4.12/lib/batteries -I /home/opam/.opam/4.12/lib/bst -I /home/opam/.opam/4.12/lib/bytes -I /home/opam/.opam/4.12/lib/cpm -I /home/opam/.opam/4.12/lib/cpu -I /home/opam/.opam/4.12/lib/dolog -I /home/opam/.opam/4.12/lib/line_oriented -I /home/opam/.opam/4.12/lib/minicli -I /home/opam/.opam/4.12/lib/molenc -I /home/opam/.opam/4.12/lib/num -I /home/opam/.opam/4.12/lib/ocaml/threads -I /home/opam/.opam/4.12/lib/parany -I src/.orf.objs/native -I src/.orf.objs/public_cmi -no-alias-deps -open Dune__exe -o src/.model.eobjs/byte/dune__exe__Model.cmo -c -impl src/model.ml)
# File "src/model.ml", line 34, characters 26-39:
# 34 |        let actual = L.map Mol.get_value test in
#                                ^^^^^^^^^^^^^
# Error: Unbound value Mol.get_value
# Hint: Did you mean get_name?

samoht avatar Apr 02 '24 11:04 samoht

@shonfeder ok, let's try

UnixJunkie avatar Jun 20 '24 00:06 UnixJunkie

That took care of the bounds checks!

We are also seeing a failure on windows https://github.com/ocaml/opam-repository/actions/runs/9589562081/job/26443537300

#=== ERROR while compiling conf-autoconf.0.2 ==================================#
"autoconf": command not found.


<><> Error report <><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><>
+- The following actions failed
| - fetch batteries     3.8.0
| - fetch cmdliner      1.3.0
| - fetch conf-python-3 9.0.0
| - build conf-autoconf 0.2
+- 
- No changes have been performed
# Run (& opam env) -split '\r?\n' | ForEach-Object { Invoke-Expression $_ } to update the current shell environment
orf.1.0.1 failed to build.

But that looks to be an issue with conf-autoconf.

The last thing here looks to be checksum, which should be updated to accord with the new policy:

2024-06-20 00:15.45: Error in orf.1.0.1: Weak checksum algorithm(s) provided. Please use SHA-256 or SHA-512. Details: opam field url contains only MD5 as checksum for https://github.com/UnixJunkie/orf/archive/refs/tags/v1.0.1.tar.gz

Once that is done this should be good to go. Thanks for your patience in getting the package configured, and for taking the time to share this :)

shonfeder avatar Jun 20 '24 02:06 shonfeder

this is now using a sha256 checksum instead of md5

UnixJunkie avatar Jun 20 '24 03:06 UnixJunkie

Thank you!

shonfeder avatar Jun 20 '24 11:06 shonfeder

The windows CI failure should be addressed by https://github.com/ocaml/opam-repository/pull/26140/files (iiuc)

shonfeder avatar Jul 03 '24 13:07 shonfeder

Thank you for the contribution!

raphael-proust avatar Jul 03 '24 13:07 raphael-proust