git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Jeff King <peff@peff.net>
To: Michal Novotny <clime@redhat.com>
Cc: git@vger.kernel.org
Subject: Re: A few questions regarding git annotated tags
Date: Sun, 6 Jan 2019 01:50:33 -0500	[thread overview]
Message-ID: <20190106065032.GC4207@sigill.intra.peff.net> (raw)
In-Reply-To: <CANT8FXRRTpAaW0JxYCt94f52eKAz1cBAGpPA84CUTMJUgQrkuw@mail.gmail.com>

On Sat, Jan 05, 2019 at 04:50:30PM +0100, Michal Novotny wrote:

> I could potentially make it so that I tag subtrees instead of commits
> and then derive the needed information from these subtree tags. This
> could be useful if I have multiple rpm packages in different subtrees
> of the same repo. I could then tag the subtree where the rpm package
> is placed.
> 
> This could bring some simplification into the code but as far as I
> know, you cannot easily checkout a tree tag, which is something a
> packager should be able to do easily: to checkout a state of repo when
> a certain subpackage was tagged. This is the first question. Can you
> e.g. do:
> 
> git tag somename HEAD:
> 
> and then do something similar to
> 
> git checkout somename
> 
> which would restore the repository or at least the respective subtree
> of it into the state when "somename" tag was created?

No, there's no easy way to check out a bare tree (and in fact, HEAD is
forbidden to point to a non-commit).

You could hack around that by making a new commit that wraps the tree,
like:

  commit=$(echo 'wrap foo package' | git commit-tree HEAD:foo)
  git tag foo-1.2.3 $commit

There are also useful things to do with the tag of the bare tree. E.g.,
export it via git-archive, diff it, "git checkout -p" changes from it,
etc. But actually creating a working tree state from it is awkward:

  # move to being on an "unborn" branch foo
  git checkout --orphan foo

  # load the desired tree state; "-u" will update the working tree
  # files
  git read-tree -m -u foo-1.2.3

  # if we were to commit now, it would become the root commit of the
  # "foo" branch, with no parents. That would make it pretty useful for
  # things like merging between tags.
  git commit -m 'kind of weird'

So it seems kind of awkward and useless.  I'm not sure I totally
understand your problem space, but if you can have actual commits with a
logical progression (i.e., where the parent links actually mean
something), I think Git's tools will be more useful.

> Right now, I am putting a package name directly into tag name so I
> know what tags belong to what package based on that. And I am using
> normal annotated tags. This works quite well, I would say, but at one
> point I need to use shared state to move the discovered package name
> from one part of the code to another so that the other part can work
> with the correct subset of the available annotated tags. I wouldn't
> need to do that if I could derive the correct tag subset based just on
> the path to the subtree where a package is placed.

I'm not sure I understand this bit. Even if you tag a subtree, like:

  git tag foo-1.2.3 HEAD:foo

then that tree doesn't "know" that it was originally at the path "foo".
You'd have to tag the root tree, and then know to look in the "foo"
subtree from there. At which point you might as well tag the commit that
contains that root tree. Whether it happens to touch the "foo" path or
not, it represents a particular state.

> Alternative approach to creating the tree tags would be to store the
> path information into annotated tag message, which I could do. But is
> there a relatively simple way to filter tags based on their message
> content? Can I put the information into some other part of tag than
> name or the message so that it can be easily filtered?

I don't think there's an easy way to show only tags matching a pattern.
You could do something like:

  git tag -m 'path: foo' foo

  git for-each-ref --format='%(refname:strip=2) %(subject)' refs/tags/ |
  grep 'path: foo' |
  awk '{print $1}'

to grep their subjects (or body, if you want to make the grep stage a
little more clever). Obviously that is not really a structured lookup,
but if you control the tag contents, it might be OK.

In commit messages there's a concept of machine-readable trailers, like
"Signed-off-by", etc, and even some tools for displaying those. But
there's not currently any support for parsing them out of tag objects.


I sort of answered your questions literally, but TBH I'm still not
entirely sure what you're trying to accomplish. So hopefully it was
useful, but feel free to follow up with more questions. ;)

-Peff

  reply	other threads:[~2019-01-06  6:50 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-05 15:50 A few questions regarding git annotated tags Michal Novotny
2019-01-06  6:50 ` Jeff King [this message]
2019-01-10 13:24   ` Michal Novotny

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190106065032.GC4207@sigill.intra.peff.net \
    --to=peff@peff.net \
    --cc=clime@redhat.com \
    --cc=git@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).