Re: check if one branch contains another branch

From: Alexander Mills <alexander.d.mills@gmail.com>
Date: May 8, 2020, 01:01 UTC
Message-ID: <CA+KyZp7emY12sy6QdGBJ5T9e135VCfjeqTy9u_FWh+FA1d-uFg@mail.gmail.com>
In-Reply-To: <CA+KyZp7Ur3kW3qrCe3hOz16gA9y_B7rSGin62=eKMjCCqLsR4w@mail.gmail.com>

git branch -d foo   # safe delete

the above doesn't seem to work if you squash commits into an integration branch, I will get something like:

========================================================
 git branch -d CP-10-master
warning: deleting branch 'CP-10-master' that has been merged to
         'refs/remotes/origin/CP-10-master', but not yet merged to HEAD.
Deleted branch CP-10-master (was faeb801).
==========================================================

If a branch gets squashed into another, how does merge-base do it's
thing, since it's not a fast-forward merge etc?
I assume that it's not possible, for example this script shows that if
 git merge --squash is used, git doesn't know that the unsquashed
branch is already merged:

==========================================================

#!/bin/bash

set -e

git checkout master git branch -D delete-me-1 || echo git branch -D delete-me-2 || echo

git checkout -b delete-me-1 git checkout -b delete-me-2 git commit --allow-empty -am "first new one" git commit --allow-empty -am "second new one" git commit --allow-empty -am "third new one" git checkout delete-me-1 git merge --squash delete-me-2 ### compare without squash

if git merge-base --is-ancestor delete-me-2 delete-me-1; then
  echo 'delete-me-1 is an ancestor of delete-me-2'
else
  echo 'not an ancestor'
fi
========================================================

that will print "not an ancestor" if  --squash is used..

-alex

On Thu, May 7, 2020 at 4:37 PM Alexander Mills <alexander.d.mills@gmail.com> wrote:

Show 48 quoted lines

>
> I assume that:
>
> git branch -d  xxx
>
> will prevent a delete if the current branch doesn't contain xxx..
> I don't want to have to checkout  origin/dev in order to run that command,
> that's one part of the problem
>
>
> On Thu, May 7, 2020 at 4:31 PM Randall S. Becker <rsbecker@nexbridge.com> wrote:
> >
> > On May 7, 2020 6:59 PM Alexander Mills, Wrote:
> > > To: git@vger.kernel.org
> > > Subject: check if one branch contains another branch
> > >
> > > I am looking for a command:
> > >
> > > 1>  if branch x contains branch y
> > >
> > > right now all I can find is
> > >
> > > 2> if current branch contains commit y
> > >
> > > can someone please accomplish #1 ?
> > > Crazy hard to find an answer to #1 online.
> > > The user case is to delete old local branches by comparing them with the
> > > remote integration branch.
> > >
> > > more info if needed:
> > > https://stackoverflow.com/questions/61669056/how-to-determine-if-
> > > integration-branch-contains-feature-branch
> >
> > Looking at this slightly differently, if you try to delete a branch, git branch -d feature-branch, and the branch has not been merged, then the delete will fail. A simple way of looking at it is if the HEAD of the branch has no successor commits then it is not merged (not 100% decisive, but git branch -d is). It is not really that a branch has been merged, but that a commit has successors, meaning that it has been merged. However, unless you are using GitLab, a git merge --squash will not answer your question even if the branch was merged.
> >
> > A better way of looking at this is in terms of Pull (GitHub, BitBucket) or Merge (GitLab) requests. Has there been a Pull Request for a branch and has the branch been closed? Meaning that when you do a git fetch --prune, your merged/deleted branches go away unless you are on that branch. Looking at the Pull Request history is much more useful in determining whether a branch has been integrated into a main development branch or production branch in a GitFlow process.
> >
> > It is a different way of looking at the problem, but IMHO, a more representative way when taking developers and deployment into account.
> >
> > Regards,
> > Randall
> >
>
>
> --
> Alexander D. Mills
> New cell phone # (415)730-1805
> linkedin.com/in/alexanderdmills

-- Alexander D. Mills New cell phone # (415)730-1805 linkedin.com/in/alexanderdmills