From: Heiko Voigt <hvoigt@hvoigt.net>
To: Jeff King <peff@peff.net>
Cc: "Jeffrey S. Haemer" <jeffrey.haemer@gmail.com>,
Jens Lehmann <Jens.Lehmann@web.de>,
Git Issues <git@vger.kernel.org>
Subject: [PATCH 0/3] fix cloning superprojects from "."
Date: Tue, 13 Nov 2012 09:32:33 +0100 [thread overview]
Message-ID: <20121113083233.GA38188@book.hvoigt.net> (raw)
In-Reply-To: <20121109184225.GA1190@book.hvoigt.net>
Hi,
On Fri, Nov 09, 2012 at 07:42:26PM +0100, Heiko Voigt wrote:
> Since this is a change in behaviour I would like to further think about
> the implications this brings if we fix this. Not sure how many people
> clone from ".". The correct behavior (as documented) is the one you
> introduce with your patch. If we decide to fix this we should also correct
> the path calculation in git-submodule.sh.
Ok I think this corner case is not that commonly used since most people
work with remote remotes which you can not cd into to clone from ".".
Here is a patch series to clean this handling up.
Cheers Heiko
Heiko Voigt (3):
Fix relative submodule setup of submodule tests
ensure that relative submodule url needs ./ or ../
fix corner case for relative submodule path calculation
git-submodule.sh | 22 +++++++++++++++++
t/t7400-submodule-basic.sh | 56 ++++++++++++++++++++++++++++++++++++++++++++
t/t7403-submodule-sync.sh | 2 ++
t/t7406-submodule-update.sh | 2 ++
t/t7407-submodule-foreach.sh | 2 ++
t/t7506-status-submodule.sh | 2 ++
6 files changed, 86 insertions(+)
--
1.8.0.3.gaed4666
next prev parent reply other threads:[~2012-11-13 8:32 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-04 19:50 git commit/push can fail silently when clone omits ".git" Jeffrey S. Haemer
2012-11-08 18:56 ` Jeff King
2012-11-09 18:42 ` Heiko Voigt
2012-11-13 8:32 ` Heiko Voigt [this message]
2012-11-13 8:34 ` [PATCH 1/3] Fix relative submodule setup of submodule tests Heiko Voigt
2012-11-13 8:35 ` [PATCH 2/3] ensure that relative submodule url needs ./ or ../ Heiko Voigt
2012-11-13 8:35 ` [PATCH 3/3] fix corner case for relative submodule path calculation Heiko Voigt
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=20121113083233.GA38188@book.hvoigt.net \
--to=hvoigt@hvoigt.net \
--cc=Jens.Lehmann@web.de \
--cc=git@vger.kernel.org \
--cc=jeffrey.haemer@gmail.com \
--cc=peff@peff.net \
/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).