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

RFE: add API for parsing licenses #13

Open
ignatenkobrain opened this issue Jan 5, 2018 · 2 comments
Open

RFE: add API for parsing licenses #13

ignatenkobrain opened this issue Jan 5, 2018 · 2 comments

Comments

@ignatenkobrain
Copy link

It would be really awesome if there would be API after crate parsed license, so it could be hooked to some tools (e.g. in rust2rpm in my case).

@wking
Copy link
Contributor

wking commented Jan 5, 2018

I don't have enough Rust to understand the implementation, but there's some AST discussion in #5. Maybe look that over?

@Nemo157
Copy link
Contributor

Nemo157 commented May 7, 2018

I have just taken a quick look at #5, for some reason it's not compiling at the moment. It looks ok, but I have no experience with lalrpop and really don't like the 2000+ line generated file being committed to the repository.

I'm currently thinking of redoing this from scratch using a nom based parser (because I'm somewhat familiar with nom), it's a very simple language to parse so I don't think losing out on the existing parser work is very bad.

For my reference, here's a couple of projects used by other package managers that will likely be useful sources of test data:

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

No branches or pull requests

3 participants