abderrahim commented on code in PR #87:
URL:
https://github.com/apache/buildstream-plugins/pull/87#discussion_r2072437685
##########
src/buildstream_plugins/sources/git.py:
##########
@@ -986,6 +1027,25 @@ def validate_cache(self):
warning_token=CoreWarnings.REF_NOT_IN_TRACK,
)
+ def collect_source_info(self):
+ #
+ # Currently we cannot implement version guessing, because we do not
save any tag
+ # information in the ref at tracking time.
+ #
+ # Also we do *not* support reporting on submodules, anyway as the
toplevel
+ # git repo determines the git commits of submodules, we consider the
toplevel
+ # git repo to be a comprehensive version of the overall input.
+ #
+ return [
+ self.create_source_info(
+ self.mirror.url,
+ SourceInfoMedium.GIT,
+ SourceVersionType.COMMIT,
+ self.mirror.ref,
Review Comment:
Yes, the git plugin accepts any string that git would accept on the command
line. In that respect it's more lenient than git_repo from
buildstream-plugins-community.
But even without doing this hack, the `git` plugin supports setting the ref
to a git-describe format when tracking by using the `ref-format: git-describe`
configuration. See https://apache.github.io/buildstream-plugins/sources/git.html
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
