On Thu, 25 Jan 2024, Thorsten Leemhuis <linux@xxxxxxxxxxxxx> wrote: > On 25.01.24 10:36, Jani Nikula wrote: >> The one thing I find problematic is the use of shallow clones by default > > FWIW, some of what is in that text is a result of similar discussion > that happened when > Documentation/admin-guide/quickly-build-trimmed-linux.rst was submitted > about a year ago, which also uses shallow clones by default. > > Further a quick reminder, as it's easy to forget: most of us live in > areas where downloading a lot of data is not something that bothers us > or will take a lot of time. But it is different for some people that > will need this document. I'm not suggesting to ignore or forget the people for whom full clones might be slow or costly. I'm suggesting to first describe the basic principles in as simple ways as possible, and then expand on pitfalls and corner cases in separate sections. I think even the simplest and most basic kernel bisect can be intimidating for anyone doing it for the first time - and that's the target audience here - so I'd like to not scare people with all the difficulties with shallow clones right off the bat. Lure them in first! ;) Also not suggesting to throw out all the things you've written, just to structure it different. >> and, well, the use of git in ways that I myself can't figure out without >> resorting to the man pages. I think it's a lot of dark corners of git >> that's overwhelming and really unrelated to the bisection itself. >> >> If I point people at that, and they have problems, I'm just going to >> tell them to: >> >> git clone git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git >> cd linux >> git remote add stable git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git >> git fetch stable > > This could be simplified, as linux-stable.git includes mainline, it's > just sometimes a few hours behind. It's perhaps a personal preference more than anything to have origin always point at Linus' tree, and all the other repos as additional remotes. Then I can ask people to add remotes e.g. for drm subsystem to try out new stuff. YMMV. >> And I can tell them to 'git checkout v<GOOD>' and 'git checkout v<BAD>' >> and proceed from there. >> >> To me, that's the TL;DR. > > FWIW, I earlier today considered changing this myself. But then I > noticed the bundle clone instructions (more on that below) are complex > as well. :-/ > > Hmmm. Maybe switching to "do a full clone of linus' repo (without using > bundles) and just telling users to add the stable branches they might > need" by default would be a good middle ground for this document. Guess > then I'd switch quickly-build-trimmed-linux.rst over myself. > >> And then you can have a section on "what if I >> really can't do full clones" and various options to save bandwidth. >> >>> Downloading the sources using a full git clone >>> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ >>> >>> If downloading and storing a lot of data (~4,4 Gigabyte as of early >>> 2023) is nothing that bothers you, instead of a shallow clone perform a >>> full git clone instead. You then will avoid the specialties mentioned >>> above and will have all versions and individual commits at hand at any >>> time:: >>> >>> curl -L \ >>> >>> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/clone.bundle >>> \ >>> -o linux-stable.git.bundle >>> git clone linux-stable.git.bundle ~/linux/ >>> rm linux-stable.git.bundle >>> cd ~/linux/ >>> git remote set-url origin \ >>> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git >>> git fetch origin >>> git checkout --detach origin/master >> >> I mean seriously, even the full clone instructions use curl, without >> rationale. Why? > > That was the result of the discussion back then, can't remember all the > details and all the places where that happened. Part of it was reducing > the server load, but that IIRC is mainly a concern for CI systems and > something this document can ignore. Unstable internet connection might > have been the main reason -- in combination with the redirection thing > kernel.org does, which *IIRC* prevents us from using "git clone > --bundle-uri=". Perhaps server load should not be a consideration in *this* particular case? To me, all of this is just saying "git is difficult", and that's kind of the wrong message here. :/ BR, Jani. -- Jani Nikula, Intel
