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

Use pfff as tool #126

Closed
priv-kweihmann opened this issue May 8, 2019 · 7 comments
Closed

Use pfff as tool #126

priv-kweihmann opened this issue May 8, 2019 · 7 comments
Labels
Feature request Not (yet) exisiting feature

Comments

@priv-kweihmann
Copy link
Owner

Maybe https://github.com/facebook/pfff is worth a look, as it seems to support a variety of languages

@priv-kweihmann priv-kweihmann added the Feature request Not (yet) exisiting feature label May 8, 2019
@priv-kweihmann
Copy link
Owner Author

According to this facebookarchive/pfff#158 (comment) pfff doesn't seem to be under active development anymore. Also this project does have major version constraints when using opam. After a deeper look into the actual performed check. I decided to close this issue as the effort does weight up the benefit of this tool.

@daghan
Copy link

daghan commented Mar 30, 2020

Hi @priv-kweihmann,
We noticed that you closed this issue because pfff is no longer maintained. In fact, the core developer for pfff is with us and we are actively maintaining sgrep.

@priv-kweihmann
Copy link
Owner Author

priv-kweihmann commented Mar 30, 2020

@daghan thanks for letting me know - still the other points do remain, like version constraints - or did that change?
Or did you just trying to advertise sgrep to me ;-)?

@aryx
Copy link

aryx commented Mar 30, 2020

the constraint right now is to use ocaml < ocaml 4.07 but there's a ticket to remove the use of camlp4 which would allow to use ocaml > 4.07 too

@priv-kweihmann
Copy link
Owner Author

If I ever going to include any ocaml based tool, it has to come up with support of the latest stable branch (afaik 4.09)

@aryx
Copy link

aryx commented Apr 2, 2020

FYI I've just updated pfff to work with recent OCaml versions, including OCaml 4.10.0.
there's a new OPAM package that just got merged here: ocaml/opam-repository#16125

@priv-kweihmann
Copy link
Owner Author

Thanks - that leaves me only with the burden of integrating ocaml into yocto;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request Not (yet) exisiting feature
Projects
None yet
Development

No branches or pull requests

3 participants