git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Matt Thompson <fortran@gmail.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org
Subject: Re: Bug: Git sees branch as valid commit ref and works; should fail
Date: Mon, 5 Aug 2024 12:58:57 -0400	[thread overview]
Message-ID: <CAFb48S8+X0=Zqi8oisB0fAgx7HoyQrahF-RGQdagXTX3RdfSNQ@mail.gmail.com> (raw)
In-Reply-To: <xmqqy15b2aiz.fsf@gitster.g>

Well, I tried the "for-each-ref" command and:

$ git for-each-ref | grep /bugfix/mathomp4/trivial-ci-commit-gcc14
$ echo $?
1

Indeed, if I try a few other greps, there are no "trivial" or "gcc14":

$ git for-each-ref | grep -i gcc14
$ echo $?
1
$ git for-each-ref | grep -i trivial
$ echo $?
1

But, I did some experimenting and I found that "ci-gcc14" does
trigger...something:

$ git for-each-ref | grep -i ci-gcc14
mathomp4@gslwl2023080107 ~/GitBugReport/fvdycore geos/main
$ echo $?
0

But I'm confused as to how the grep returned a status of 0 but didn't
print anything? So it said "Yes I see this string" but it actually
didn't? And it can switch to it:

$ git switch ci-gcc14
fatal: a branch is expected, got commit 'ci-gcc14'
hint: If you want to detach HEAD at the commit, try again with the
--detach option.
mathomp4@gslwl2023080107 ~/GitBugReport/fvdycore geos/main
$ git switch --detach ci-gcc14
HEAD is now at cc14d30 removed an unused module use in fv_sg


On Mon, Aug 5, 2024 at 12:35 PM Junio C Hamano <gitster@pobox.com> wrote:
>
> Matt Thompson <fortran@gmail.com> writes:
>
> > Thank you for filling out a Git bug report!
> > Please answer the following questions to help us understand your issue.
> >
> > What did you do before the bug happened? (Steps to reproduce your issue)
> >
> > $ git clone https://github.com/GEOS-ESM/GFDL_atmos_cubed_sphere.git fvdycore
> > $ cd fvdycore
> > $ git checkout bugfix/mathomp4/trivial-ci-commit-gcc14
> >
> > What did you expect to happen? (Expected behavior)
> >
> > I expected a failure as there is no branch named
> > 'bugfix/mathomp4/trivial-ci-commit-gcc14' in the repository.
>
> If you run
>
>   $ git for-each-ref | grep /bugfix/mathomp4/trivial-ci-commit-gcc14
>
> I suspect you have a remote-tracking branch that matches the
> pattern.  In such a case, "bugfix/mathomp4/trivial-ci-commit-gcc14"
> is a very valid way to refer to a commit to any Git command.  It is
> handy to say "git show bugfix/mathomp4/trivial-ci-commit-gcc14", for
> example.
>
> And when "git checkout" is given a commit that is not a local
> branch, what it did below (by the way, thanks for giving a very
> accurate "Actual behavior" description in your report) is very much
> expected behaviour, including the part that it gave a message to
> advise how to work on a detached HEAD.
>
> > What happened instead? (Actual behavior)
> >
> > $ git checkout bugfix/mathomp4/trivial-ci-commit-gcc14
> > Note: switching to 'bugfix/mathomp4/trivial-ci-commit-gcc14'.
> > ...
>


-- 
Matt Thompson
   “The fact is, this is about us identifying what we do best and
   finding more ways of doing less of it better” -- Director of Better
Anna Rampton

  reply	other threads:[~2024-08-05 16:59 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-08-05 15:07 Bug: Git sees branch as valid commit ref and works; should fail Matt Thompson
2024-08-05 16:35 ` Junio C Hamano
2024-08-05 16:58   ` Matt Thompson [this message]
2024-08-13 11:53     ` Jeff King
2024-08-13 12:45       ` [PATCH] get_oid(): enforce minimum length for "-g<hex>" names Jeff King
2024-08-13 13:02         ` Jeff King
2024-08-13 15:41       ` Bug: Git sees branch as valid commit ref and works; should fail Junio C Hamano

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='CAFb48S8+X0=Zqi8oisB0fAgx7HoyQrahF-RGQdagXTX3RdfSNQ@mail.gmail.com' \
    --to=fortran@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.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).