On Tue, May 10, 2016 at 07:42:50PM -0400, Nicholas D Steeves wrote: > On 10 May 2016 at 09:33, David Sterba <dsterba@xxxxxxx> wrote: > > On Mon, May 09, 2016 at 08:13:29PM -0400, Nicholas D Steeves wrote: > >> Trivial fix for typos in comments; I hope this patch isn't a nuisance! > > > > No, but I don't see the typos in any of the branches (master or the > > for-next snapshots). > > Sorry, I used for-linus, while following this guide: > "git clone git://git.kernel.org/pub/scm/linux/kernel/git/mason/linux-btrfs.git > for-linus" > https://btrfs.wiki.kernel.org/index.php/Writing_patch_for_btrfs > > and "git checkout master" confirms that I'm already on 'master', but > "git rebase next" produces the error: > > fatal: Needed a single revision > invalid upstream next You'd need to have the linux-next git tree or my kernel.org tree git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git git://git.kernel.org/pub/scm/linux/kernel/git/kdave/linux.git > I'm very new to git. Should I have cloned the whole linux-btrfs.git > without "for-linus"? Am I on the a master branch for the for-linus > module, which is separate from the master branch of the main > linux-btrfs? If not sure, look at other branches, timestamps etc. The plain for-linus in chris' repo is from january and on top of 4.3. There are other for-linus branches with a version. There's no predefined style for naming so it could be confusing. Git comes with a nice feature 'git instaweb' that will present the local git repository in a web browser and it's very handy for quick peeks into other branches. > On 10 May 2016 at 10:20, David Sterba <dsterba@xxxxxxx> wrote: > Message-ID: <20160510142028.GU29353@xxxxxxxxxxxxx> > > On Mon, May 09, 2016 at 08:13:29PM -0400, Nicholas D Steeves wrote: > > I ran ispell on the strings from comments, there are like 90+ typos > > (attached) that seem worth fixing, if you like. > > Thank you for the list, I'd be happy to. Are patches against master > or next preferred (I don't see a for-next branch)? The for-next branch is in my kenrel.org repo, otherwise the snapshots (named like for-next-20160510) are hosted in my development repositories. Basing on master should be fine, the for-next branches are sort of in flux so we could miss some typos from there, but no big deal. The list I sent was from master sources. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
