Thank you for merging. I have confirmed that this modification has resulted in the intended behaviour.
``` $ cygport --version cygport 0.36.8 Copyright (C) 2020 Cygport authors This program comes with NO WARRANTY, to the extent permitted by law. You may redistribute copies of this program under the terms of the GNU General Public License as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version. For more information about these matters, see the file named COPYING. Written for the Cygwin project <https://cygwin.com/>. $ head -3 agbsum-15-1bl1.cygport HOMEPAGE="https://mandelbrot.dk/${PN}"; GIT_URI="https://mandelbrot.dk/${PN}"; GIT_TAG="${PV}" $ cygport agbsum-15-1bl1.cygport fetch *** Info: Trying to enable case sensitivity on /tmp/agbsum/agbsum-15-1bl1.x86_64 git clone --depth 1 --branch 15 --no-checkout https://mandelbrot.dk/agbsum agbsum Cloning into 'agbsum'... fatal: dumb http transport does not support shallow capabilities *** Warning: git clone failed, retrying without --depth option git clone --branch 15 --no-checkout https://mandelbrot.dk/agbsum agbsum Cloning into 'agbsum'... Fetching objects: 251, done. git checkout tags/15 HEAD is now at bef1780 Rename source directory: 'src' => 'source'; Update naming convention; Update copyright holder name; Update code style; ``` On Mon, Feb 12, 2024 at 2:09 AM Jon Turney via Cygwin-apps <cygwin-apps@cygwin.com> wrote: > > On 03/12/2023 21:53, Brian Inglis via Cygwin-apps wrote: > > On 2023-12-03 13:34, Jon Turney via Cygwin-apps wrote: > >> On 30/11/2023 12:17, Daisuke Fujimura via Cygwin-apps wrote: > >>> Implementations that conditionally branch on variables are simple. > >>> > >>> The proposed retry implementation complicates git.cygclass, but I > >>> think it reduces the maintainer's effort. > >>> > >>> I have created a patch for a retry implementation. > >>> Could you review it? > >> > > Thanks very much to Fujimura-san for all his work on this. > > >> Attached is the patch after my edits. > > I've applied a reheated version of this patch. Hopefully that works well > enough, but obviously can be further refined if needed. > > > Looks like straight curl HEAD -I tells you about smart transport if you > > want a quick check rather than a dry run: > > > > $ time curl -ILSs > > https://repo.or.cz/r/git.git/info/refs?service=git-upload-pack | grep > > -qi '^content-type:\sapplication/x-git-upload-pack'; echo $? > > > > real 0m0.630s > > user 0m0.077s > > sys 0m0.123s > > 0 > > $ time curl -ILSs > > https://github.com/BrianInglis/apt-cyg.git/info/refs?service=git-upload-pack > > | grep -qi '^content-type:\sapplication/x-git-upload-pack'; echo $? > > > > real 0m0.440s > > user 0m0.061s > > sys 0m0.184s > > 1 > > Thanks for this. > > Uh, but it seems like 'git clone --depth 1' works with both of these > URLs, so... um... I'm not sure what's going on. >
