git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Michael J Gruber <git@drmicha.warpmail.net>
To: Steffen Daode Nurpmeso <sdaoden@googlemail.com>
Cc: git@vger.kernel.org
Subject: Re: Jabber, question on push,pull and --tags, and no help but jabber
Date: Tue, 07 Jun 2011 07:47:02 +0200	[thread overview]
Message-ID: <4DEDBB56.7000200@drmicha.warpmail.net> (raw)
In-Reply-To: <20110606214639.GA38620@sherwood.local>

Steffen Daode Nurpmeso venit, vidit, dixit 06.06.2011 23:46:
> @ Michael J Gruber <git@drmicha.warpmail.net> wrote (2011-06-06 16:31+0200):
>> "git tag" and "git verify-tag" call out to "gpg". That could be easily
>> adapted to call out to "openssl smime", or put your S/MIME signatures in
>> a note.
>>
>> Cheers
>> Michael
> 
> Hum.  It will indeed be possible to place a wrapper script 'gpg'
> in the path on my box (and catch '--verify' - or sign otherwise).

I didn't mean to shove a disguised openssl-smime into the path, I meant
that that there is little to change in code because git calls out to gpg
rather than doing it itself.

> But in the meanwhile i've found out that git(1) is heavily
> developed, stale .git_vtag_ files of an 1.7.3? version are no
> longer produced by 'git version 1.7.6.rc0' to which i've updated
> after i've seen those.  So maybe there is hope that the hardcoded
> gpg invocation will be replaced by configuration options in the
> future, too?

I don't know if it needs to be configurable. That may open a can of worms.

> I still don't understand the design with pull and --tags.
> Because, if i do 'git log' it'll display the relationship as in
> 
>     commit fd040fb[...] (tag: refs/tags/v0.3.0, refs/remotes/origin/master)

git log does that only when you ask it to decorate the commits.
"decoration" means looking up all refs and checking whether one of them
references that commit. Neither the tag (object) nor the ref names (tag
name, branch name) are part of the commit, so:

> So i'll push this commit object as part of pushing a branch, and
> the tag refers to *it*.  I don't want to be impertinent though,

The tag name is not pushed, but the commit object is and has the same
sha1 on "both sides", which is why the remote branch name shows up as a
decoration.

> and it's better that explicit way than implicitely pushing some
> distressing stuff.  Still i would have appreciated a note in the
> docu, because it took a look at the mentioned webspace to realize
> the situation.  I'll append a short diff to be able to provide
> something useful.  (No attachments allowed here i guess.)
> 
> I'll try to be less tiny from the start the next time.
> --
> Ciao, Steffen
> sdaoden(*)(gmail.com)
> () ascii ribbon campaign - against html e-mail
> /\ www.asciiribbon.org - against proprietary attachments
> 
> --
> diff --git a/Documentation/git-push.txt b/Documentation/git-push.txt
> index 88acfcd..da4a71a 100644
> --- a/Documentation/git-push.txt
> +++ b/Documentation/git-push.txt
> @@ -69,7 +69,7 @@ nor in any Push line of the corresponding remotes file---see below).
>  
>  --all::
>  	Instead of naming each ref to push, specifies that all
> -	refs under `refs/heads/` be pushed.
> +	refs under `refs/heads/` be pushed explicitely.

I don't mind but I don't think it adds clarity.

>  
>  --mirror::
>  	Instead of naming each ref to push, specifies that all
> @@ -98,7 +98,7 @@ nor in any Push line of the corresponding remotes file---see below).
>  --tags::
>  	All refs under `refs/tags` are pushed, in
>  	addition to refspecs explicitly listed on the command
> -	line.
> +	line.  Note that tags are not pushed automatically.

That is implicit in the line before it. In any case: The main problem of
git-push(1) seems to be that one has to read all the way down (through
all options) in order to grasp the default case, so I feel the first
paragraph needs to improve.

>  
>  --receive-pack=<git-receive-pack>::
>  --exec=<git-receive-pack>::
> 

  reply	other threads:[~2011-06-07  5:47 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-06 13:02 Jabber, question on push,pull and --tags, and no help but jabber Steffen Daode Nurpmeso
2011-06-06 14:31 ` Michael J Gruber
2011-06-06 15:21   ` Felipe Contreras
2011-06-06 21:46   ` Steffen Daode Nurpmeso
2011-06-07  5:47     ` Michael J Gruber [this message]
2011-06-07 10:24       ` Steffen Daode Nurpmeso
     [not found] ` <9215090.63086.1307370716794.JavaMail.trustmail@mail1.terreactive.ch>
2011-06-06 15:34   ` Victor Engmark
2011-06-06 17:58     ` Junio C Hamano
2011-06-07 10:48 ` [PATCH] Notes that tags need to pushed explicitely Steffen Daode Nurpmeso
2011-06-07 14:12   ` Junio C Hamano
2011-06-07 15:33     ` [PATCH] Remarks that tags need to be pushed explicitly Steffen Daode Nurpmeso
2011-06-10 20:39     ` [PATCH v2] " Steffen Daode Nurpmeso
2011-06-10 21:24       ` Junio C Hamano
2011-06-11 17:20         ` Steffen Daode Nurpmeso

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

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

  git send-email \
    --in-reply-to=4DEDBB56.7000200@drmicha.warpmail.net \
    --to=git@drmicha.warpmail.net \
    --cc=git@vger.kernel.org \
    --cc=sdaoden@googlemail.com \
    /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.
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 NNTP newsgroup(s).