-
Notifications
You must be signed in to change notification settings - Fork 308
-
Notifications
You must be signed in to change notification settings - Fork 308
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
Bazel: MODULE.bazel
files present in a local registry should not be considered as managed files
#9076
Comments
I mean the obvious solution to this problem is to create an exclude for |
Sounds like a use-case for ort/analyzer/src/main/kotlin/PackageManager.kt Lines 235 to 238 in 97a81dd
|
Running the Analyzer on a project depending on packages present in a local registry currently fails with "Unable to create the AnalyzerResult as it contains packages and projects with the same ids". This is because the MODULE.bazel files present in the local registry should not be considered as managed files (issue [1]). This commit adds a test for this issue. [1]: oss-review-toolkit#9076 Signed-off-by: Nicolas Nobelis <nicolas.nobelis@bosch.com>
Running the Analyzer on a project depending on packages present in a local registry currently fails with "Unable to create the AnalyzerResult as it contains packages and projects with the same ids". This is because the MODULE.bazel files present in the local registry should not be considered as managed files (issue [1]). This commit adds a test for this issue. [1]: oss-review-toolkit#9076 Signed-off-by: Nicolas Nobelis <nicolas.nobelis@bosch.com>
Running the Analyzer on a project depending on packages present in a local registry currently fails with "Unable to create the AnalyzerResult as it contains packages and projects with the same ids". This is because the MODULE.bazel files present in the local registry should not be considered as managed files (issue [1]). This commit adds a test for this issue. [1]: oss-review-toolkit#9076 Signed-off-by: Nicolas Nobelis <nicolas.nobelis@bosch.com>
Running the Analyzer on a project depending on packages present in a local registry currently fails with "Unable to create the AnalyzerResult as it contains packages and projects with the same ids". This is because the MODULE.bazel files present in the local registry should not be considered as managed files (issue [1]). This commit adds a test for this issue. [1]: oss-review-toolkit#9076 Signed-off-by: Nicolas Nobelis <nicolas.nobelis@bosch.com>
I created #9090 as a test for this issue. However I think the issue is more severe as what is described here. There is first:
But afterwards comes this exception:
|
Running the Analyzer on a project depending on packages present in a local registry currently fails with "Unable to create the AnalyzerResult as it contains packages and projects with the same ids". This is because the `MODULE.bazel` files present in the local registry should not be considered as definition files (issue [1]). This commit adds a test for this issue. [1]: oss-review-toolkit#9076 Signed-off-by: Nicolas Nobelis <nicolas.nobelis@bosch.com>
Running the Analyzer on a project depending on packages present in a local registry currently fails with "Unable to create the AnalyzerResult as it contains packages and projects with the same ids". This is because the `MODULE.bazel` files present in the local registry should not be considered as definition files (issue [1]). This commit adds a test for this issue. [1]: oss-review-toolkit#9076 Signed-off-by: Nicolas Nobelis <nicolas.nobelis@bosch.com>
…ored Running the Analyzer on a project depending on packages present in a local registry currently fails with "Unable to create the AnalyzerResult as it contains packages and projects with the same ids". This is because the `MODULE.bazel` files present in the local registry should not be considered as definition files (issue [1]). This commit adds a test for this issue. [1]: oss-review-toolkit#9076 Signed-off-by: Nicolas Nobelis <nicolas.nobelis@bosch.com>
…ored Running the Analyzer on a project depending on packages present in a local registry currently fails with "Unable to create the AnalyzerResult as it contains packages and projects with the same ids". This is because the `MODULE.bazel` files present in the local registry should not be considered as definition files (issue [1]). This commit adds a test for this issue. [1]: #9076 Signed-off-by: Nicolas Nobelis <nicolas.nobelis@bosch.com>
`MODULE.bazel` files present in the local registry should not be considered as definition files. Fixes oss-review-toolkit#9076. Signed-off-by: Nicolas Nobelis <nicolas.nobelis@bosch.com>
I have the following Bazel project:
The directory
registry
is a local registry defined in.bazelrc
:common --registry=file://%workspace%/registry/
In
%workspace%/MODULE.bazel
, a dependency toA
is defined:bazel_dep(name = "A", version = "1.0")
In
%workspace%/registry/modules/A/1.0/MODULE:bazel
, a dependency toB
is defined:bazel_dep(name = "B", version = "1.0")
With such a setup, ORT Analyzer throws the following error:
The problem is that the Bazel package manager collects all
MODULE.bazel
s in the project directory and considers them all as managed files. TheMODULES.bazel
s present in a (registry) should not be picked up by the package manager as there are not "really" sub dependency trees.The dependency of
A
toB
is picked by ORT by callingbazel mod graph
on the top levelMODULE.bazel
.Is there a mechanism in ORT to exclude managed files during a package manager run ?
I will create a test for this ASAP.
The text was updated successfully, but these errors were encountered: