Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add package for cli programs. #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

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