(fwd) bzr shelve feedback

[Jan Hudec]

On Fri, Nov 25, 2005 at 09:07:44 -0500, Aaron Bentley wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Jan Hudec wrote:
> >>Implementation of hiding may be a problem.  If you mean to store them
> >>separately, you lose a lot of the compression advantages of weaves.
> >>Making some revisions in a weave hidden would be hard.
> > 
> > I'd rather store them in the weave, not because of space savings, but
> > mainly for ease of merging and simplicity. On the other hand when it's
> > used in the quilt way, you could quickly gather rather many shelf
> > revisions (because they are never deleted) and most of them is really
> > worthless once the feature is ready. Thus I'd prefer if fetch could not
> > bring them over unless explicitly asked for (and push/pull would not ask
> > by default).
> 
> I'm not sure that makes anything simpler.  It just puts the complexity
> into Fetcher, and Fetcher is already very greedy (it copies all revision
> of the file in the remote weave, not just the ones you need).
>
> Having two kinds of revisions stored in the repository complicates the
> model.  Teaching Fetcher about the two kinds of revisions and getting it
> to only fetch one kind would take a lot of work.  And what about the
> people who work on the code but don't realize there are two kinds of
> revisions?

Well, I think the amount of complexity is different for each case.
Because for all the other commands except pull, push and fetch, they are
normal revisions. So it stands - is it easier to teach it to combine the
revision information from two sources, or to avoid revisions with some
particular property in fetch (and prevent pull/push if head of either
branch has that property). Also since they SHOULD be fetchable
OPTIONALLY, fetch would be complicated a bit even in the 2-file case.

--
						 Jan 'Bulb' Hudec <bulb at ucw.cz>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20051125/7a131941/attachment.pgp