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

[ocaml-install]: Create Add_dependency "Process" module in dune.ml #69

Open
Tracked by #61
dmmulroy opened this issue Jan 9, 2024 · 1 comment
Open
Tracked by #61
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@dmmulroy
Copy link
Owner

dmmulroy commented Jan 9, 2024

Parent: #61

In order to install a new dependency we need to be able to append the depends stanza in our users dune-project file. We will need to create a new module called Add_dependency to the src/core/dune.ml module under the nested Dune_project module.

I'm not sure the best way to parse/update the file, so I'm open to really any implementation at the moment. You might find some useful code in the dune repo itself for how they parse dune/sexp files.

The exec function should accept an input something like { file_path: string; dependency: Dependency.t } where file_path is the path to thedune-project file.

@dmmulroy dmmulroy added help wanted Extra attention is needed good first issue Good for newcomers labels Jan 9, 2024
@dmmulroy
Copy link
Owner Author

Just gonna drop this here :)

https://github.com/dmmulroy/s-tier

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant