From: Andreas Dilger <adilger@turbolabs.com>
To: Jeff Garzik <jgarzik@mandrakesoft.com>
Cc: Patrick Mochel <mochel@osdl.org>, linux-kernel@vger.kernel.org
Subject: Re: [bk patch] Make cardbus compile in -pre4
Date: Sat, 9 Feb 2002 00:29:20 -0700 [thread overview]
Message-ID: <20020209002920.Z15496@lynx.turbolabs.com> (raw)
In-Reply-To: <Pine.LNX.4.33.0202081824070.25114-100000@segfault.osdlab.org> <20020208203931.X15496@lynx.turbolabs.com> <3C649F4F.7E190D26@mandrakesoft.com>
In-Reply-To: <3C649F4F.7E190D26@mandrakesoft.com>; from jgarzik@mandrakesoft.com on Fri, Feb 08, 2002 at 11:02:23PM -0500
On Feb 08, 2002 23:02 -0500, Jeff Garzik wrote:
> Andreas Dilger wrote:
> > I don't see why everyone who is using BK is expecting Linus to do a pull.
> > In the non-BK case, wasn't it always a "push" model, and Linus would not
> > "pull" from URLs and such? Why are people not simply doing:
> >
> > !bk send -r+ (other options) -
> >
> > from within their editor (or equivalent) to inline the CSET in the email?
> > This has the added advantage that other people reading the email can also
> > import the CSET immediately if they so desire.
>
> 'bk pull' is probably most useful to high volume submitters, where the
> contents of most patches is either obvious and/or uninteresting.
The problem is that (AFAIK) bk pull does not let Linus pick-and-choose
which patches he wants to accept as easily as importing them at the time
he reads each email. It basically assumes that he wants everything that
is in the repository he is pulling from.
> 'bk send -d -r<rev> -' should be fine for importing.
Yes, that would be my thought as well. Sadly, running the command
bk send -d -r+ -wgzip_uu -
does not work as I would _hope_ it would, namely putting a regular context
diff at the beginning of the email, and gzip_uu for only the CSET. That
would give us the best of both worlds, namely a diff to look at (which
most people can read easily), and the compressed CSET for people who
have BK.
I suppose it is possible to do exactly what I want by running both:
bk changes -r<rev> # generates Changelog
bk export -tpatch -h -du -r<rev> # generates context diff
bk send -wgzip_uu -r<rev> - # generates gzipped/uuencoded CSET
This has the added benefit that 'bk export' does not contain changes to
the BK files themselves, only the real changes.
I have no idea if this would confuse BK if you tried to recieve from
a file which had both of these in it...
> But this is still a trial run of BK, so who knows what will wind up to
> be the best policy for casual submitters.
>
> And there's nothing wrong at all with sending GNU patches...
Oh, I agree that for people without BK they can keep sending patches.
I would prefer that people who _do_ have BK to send the CSET along with
the patch to the mailing list so that you don't have to go hunting for a
specific CSET, or if you can't because you are not currently connected.
This might also be possible if BK could export/import a whole changeset
in patch form, plus some magic stuff at the beginning/end (gzip_uu) which
had all of the BK metadata in it, but I don't know if that is possible or
desirable.
Cheers, Andreas
--
Andreas Dilger
http://sourceforge.net/projects/ext2resize/
http://www-mddsp.enel.ucalgary.ca/People/adilger/
next prev parent reply other threads:[~2002-02-09 7:30 UTC|newest]
Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-02-09 2:25 Patrick Mochel
2002-02-09 3:39 ` Andreas Dilger
2002-02-09 4:02 ` Jeff Garzik
2002-02-09 7:29 ` Andreas Dilger [this message]
2002-02-09 7:41 ` Larry McVoy
2002-02-10 2:39 ` Jeff Garzik
2002-02-10 3:52 ` Linus Torvalds
2002-02-10 7:47 ` Andreas Dilger
2002-02-10 20:57 ` Linus Torvalds
2002-02-11 18:38 ` Andreas Dilger
2002-02-09 5:12 ` Larry McVoy
2002-02-09 5:32 ` Andrew Morton
2002-02-09 9:36 ` Rob Landley
2002-02-09 9:57 ` Momchil Velikov
2002-02-09 10:01 ` Alexander Viro
2002-02-09 18:09 ` Rob Landley
2002-02-09 15:08 ` Daniel Phillips
2002-02-10 4:07 ` Linus Torvalds
2002-02-09 10:14 ` David Lang
2002-02-09 15:54 ` Larry McVoy
2002-02-09 16:50 ` Tom Rini
2002-02-09 17:05 ` Larry McVoy
2002-02-09 21:01 ` David Lang
2002-02-09 21:41 ` Larry McVoy
2002-02-09 23:36 ` Andreas Dilger
2002-02-09 23:45 ` Tom Rini
2002-02-10 0:42 ` Andreas Dilger
2002-02-09 23:52 ` Larry McVoy
2002-02-10 4:13 ` Linus Torvalds
2002-02-10 18:02 ` Tom Rini
2002-02-10 5:25 ` William Stearns
2002-02-11 17:30 ` Padraig Brady
2002-02-13 11:59 ` Padraig Brady
2002-02-09 9:27 ` pull vs push (was Re: [bk patch] Make cardbus compile in -pre4) Rob Landley
2002-02-09 10:08 ` Andreas Dilger
2002-02-09 18:12 ` Stelian Pop
2002-02-09 20:59 ` Linus Torvalds
2002-02-09 20:12 ` Stelian Pop
2002-02-09 20:26 ` Larry McVoy
2002-02-09 20:51 ` Stelian Pop
2002-02-09 23:45 ` Jeff Garzik
2002-02-09 23:49 ` Larry McVoy
2002-02-09 20:57 ` Pau Aliagas
2002-02-09 21:07 ` David Lang
2002-02-09 21:13 ` Pau Aliagas
2002-02-09 21:45 ` Rob Landley
2002-02-10 0:19 ` Andreas Dilger
2002-02-10 0:36 ` Herbert Xu
2002-02-10 0:54 ` ssh primer (was Re: pull vs push (was Re: [bk patch] Make cardbus compile in -pre4)) Jeff Garzik
2002-02-10 0:59 ` Herbert Xu
2002-02-10 1:24 ` Jeff Garzik
2002-02-10 8:13 ` Herbert Xu
2002-02-13 17:13 ` Aaron Lehmann
2002-02-14 0:22 ` Rob Landley
2002-02-14 6:57 ` Aaron Lehmann
2002-02-14 11:00 ` Harald Arnesen
2002-02-10 0:59 ` Ben Pfaff
2002-02-10 1:14 ` David Lang
2002-02-10 1:22 ` ssh primer (was Re: pull vs push (was Re: [bk patch] Make cardbuscompile " Jeff Garzik
2002-02-10 2:46 ` pull vs push (was Re: [bk patch] Make cardbus compile in -pre4) Alan Cox
2002-02-11 11:51 ` Pavel Machek
2002-02-11 18:42 ` John Alvord
2002-02-09 11:44 ` [bk patch] Make cardbus compile in -pre4 Peter Osterlund
2002-02-10 20:12 Chris Adams
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=20020209002920.Z15496@lynx.turbolabs.com \
--to=adilger@turbolabs.com \
--cc=jgarzik@mandrakesoft.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mochel@osdl.org \
/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
all inboxes | Powered by JetHome®