verified-network-toolchain / petr4

Petr4: Formal Semantics for P4
Apache License 2.0
75 stars 21 forks source link

Fix the coq installation problem. #411

Closed txyyss closed 1 year ago

txyyss commented 1 year ago

Fix the coq file installation problem by adding back (package poulet4) to the file coq/lib/dune , below (name Poulet4).

Ref: https://dune.readthedocs.io/en/latest/coq.html#coq-theory If the package field is present, Dune generates install rules for the .vo files of the theory. pkg_name must be a valid package name.

Since last time, this line is moved by comment message "fix build". I'm not sure it will break ci. It builds and installs without problem in my machine.

hackedy commented 1 year ago

Thanks for fixing this!