Skip to content

Comments

Add package for cli programs.#17

Open
tmcgilchrist wants to merge 1 commit intohaesbaert:masterfrom
tmcgilchrist:master
Open

Add package for cli programs.#17
tmcgilchrist wants to merge 1 commit intohaesbaert:masterfrom
tmcgilchrist:master

Conversation

@tmcgilchrist
Copy link

@tmcgilchrist tmcgilchrist commented Nov 23, 2022

This project is failing to build in ocaml-ci.

It is caused by the bin/* programs requiring extra dependencies that are not present in any opam files. I've created a raw link-bin.opam package to cover these programs and pull in the required opam packages. The wording in the package is copied straight from rawlink.opam and possibly needs to be changed.

@haesbaert
Copy link
Owner

Sorry for the absurd delay in answering this.
The idea for those binaries were to not be shipped more like a debugging tool while I was developing, especially the test-arp one. I'm not sure I see much point in making them public, I'll play with actions tomorrow and see what to do.

@tmcgilchrist
Copy link
Author

The opam supported options would be either:

  1. include them as test dependencies and use with-test to include any extra libraries that aren't required for the library.
  2. use this approach of having a raw-link-bin.opam package that you don't publish to opam.

I'm not aware of a third supported way to do this.

@haesbaert
Copy link
Owner

The opam supported options would be either:

  1. include them as test dependencies and use with-test to include any extra libraries that aren't required for the library.
  2. use this approach of having a raw-link-bin.opam package that you don't publish to opam.

I'm not aware of a third supported way to do this.

hmmmm I think the different package might be a way to go, I'll see if I can get test_arp to work with with-test, now it doesn't really need an interface name, it just needs to be run as root.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants