-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat(curations): Add VCS info for remaning commons-logging-api
versions
#199
Conversation
comment: |- | ||
No SCM tag in the POM: https://repo1.maven.org/maven2/commons-logging/commons-logging-api/1.0.3/commons-logging-api-1.0.3.pom | ||
vcs: | ||
type: "Git" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMO the type
and url
should be set for a range of versions or agnostic of the version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've extracted these to a version-independent curation now.
vcs: | ||
type: "Git" | ||
url: "https://github.com/apache/commons-logging.git" | ||
revision: "LOGGING_1_0_3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is ORT's matching algorithm unable to match this tag?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, ORT should be able to match it, see this test for a similar case. But I wanted to be consistent with version 1.1 here, which only uses two digits, and cannot be matched to the three-digit LOGGING_1_1_0
tag.
…ions Extract common VCS info to a version-independent curation. Note that the only other versions of `commons-logging-api` are 1.0.3 and 1.0.4 which have three digits, and thus ORT can match the corresponding three-digit tag names. Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>
095a6cd
to
cf2e87f
Compare
No description provided.