-
-
Notifications
You must be signed in to change notification settings - Fork 561
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
Instructions for Identifying Dependencies Unclear #631
Comments
@KinXer Thanks for chiming and this is a fair point, this is not much documented: this coming from the package scan. I will make sure this is clear in the upcoming doc for release 2.0. (e.g. both in the wiki and the CLi help and the readme) |
@KinXer if this was not clear it comes with the default scan or the |
@pombredanne Your previous reply made that clear; thank you for the replies. I do not, however, see how the packages section of the output is particularly useful for identifying dependencies. If that is a misunderstanding on my part, I ask that you make the documentation clear on that point. |
ATM npm (JS), composer (PHP) and maven (POMs) direct dependencies are collected in the Other are in the works (such as Godeps and Rubygems in https://github.com/nexB/scancode-toolkit-contrib/tree/develop/src/packagedcode2) BUT there is bug.... this is not wired correctly!
I need to fix this ASAP! |
* rename class to MavenPomPackage for clarity * fix the metafiles and the recognition for #631 * add additional api and recognize tests * clean up the API doc and otehr minor refactorings Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
This latest commit fixes the lack of Maven package collection. Other should work OK. Out of curiosity what are the package managers/formats you work with? |
Mostly Maven and Pip. Thank you for the Maven update. It should be very helpful. |
@KinXer Thanks you ++ for bringing it up ... I cannot fathom how the whole code was there but not wired in properly and that there was no proper tests on the CLI and package recognition side :| Note a couple things:
|
The maven dependencies should be properly collected in develop now. .... Still need to add proper docs. |
Some updates on how we handle dependencies now, repasting from #3828 (comment) Just a bit of updates there:
A simple process to scan all the dependencies:
|
The project description lists dependencies as one of the main categories of items Scancode detects; however, it is not clear from the wiki, the readme, or the command-line help how one actually uses Scancode to detect dependencies or whether this functionality is still missing or in development. Please update the readme or wiki to accurately reflect the state of dependency-checking in Scancode.
The text was updated successfully, but these errors were encountered: