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

FastClasspathScanner is outdated -- consider porting to ClassGraph #9

Open
classgraph-robot opened this issue Aug 26, 2018 · 3 comments

Comments

@classgraph-robot
Copy link

Your project, drogatkin/TJWS2, depends on the outdated library FastClasspathScanner in the following source files:

FastClasspathScanner has been significantly reworked since the version your code depends upon:

  • a significant number of bugs have been fixed
  • some nontrivial API changes have been made to simplify and unify the API
  • FastClasspathScanner has been renamed to ClassGraph: https://github.com/classgraph/classgraph

ClassGraph is a significantly more robust library than FastClasspathScanner, and is more future-proof. All future development work will be focused on ClassGraph, and FastClasspathScanner will see no future development.

Please consider porting your code over to the new ClassGraph API, particularly if your project is in production or has downstream dependencies:

Feel free to close this bug report if this code is no longer in use. (You were sent this bug report because your project depends upon FastClasspathScanner, and has been starred by 50 users. Apologies if this bug report is not helpful.)

@drogatkin
Copy link
Owner

Currently TJWS code tied to old FastClasspathScanner and since API changed, it will require to rework TJWS code. Currently I do not have bandwidth to do that, but will happily accept pull requests.

@lukehutch
Copy link

I have even less bandwidth to work on this... I sent out these notifications to the top 100 users of FCS, as a courtesy, and I can't fix all those projects. Porting is not hard though. Feel free to close this bug report if you don't see the benefit.

@drogatkin
Copy link
Owner

No worry, the initial version works fine for me. I just backported few lines of your latest code to support bytecode version 53.

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