-
Notifications
You must be signed in to change notification settings - Fork 315
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
Analyzer issue when scanning non VCS directory #8187
Comments
Closing this as a duplicate of #2896 and linked issues. In short, analyzing a directory that is not under version control is currently not supported. |
Just curious @alexandruz, is your specific use-case maybe related to #4242, i.e. the analysis of archives uploaded to FOSSID via the API? |
Yes, it's pretty much a similar workflow. Scanning archives without a VCS repository is the main use-case. |
Thanks for confirming. Can you share some more details about FOSSID's use of ORT? I've heard rumors that FOSSID plans to use another SCA solution. However, if it'd help you continuing to use ORT, and if it'd also help @nnobelis for his use-case, we might be able to prioritize work on this. |
Yes, we definitely plan to continue using ORT, and many of our customers are very much relying on ORT in combination with our tools. However, it's also true that we are developing other solutions that do not rely on ORT. In the future, we expect to offer a mix of both ORT-based and non-ORT-based solutions. |
Thanks again for the insights @alexandruz! Esp. if at least the ORT analyzer is a part of ORT you'll continue to use, let's maybe have a chat how we could potentially collaborate. Feel free to reach out to me under my commit email address. |
Scanning a non-VCS directory gives the following error:
Steps:
ort analyze
Based on the https://github.com/oss-review-toolkit/ort/blob/main/model/src/main/kotlin/OrtResult.kt#L230 it shouldn't terminate in
Exception in thread main
, right?Is there a workaround or ORT can only scan VCS repositories?
Thanks.
The text was updated successfully, but these errors were encountered: