/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to doc/partial-commit.txt

  • Committer: Martin Pool
  • Date: 2006-03-10 06:09:25 UTC
  • mto: This revision was merged to the branch mainline in revision 1609.
  • Revision ID: mbp@sourcefrog.net-20060310060925-7b26b6236c7cd6e5
[patch] LocalTransport.list_dir should return url-quoted strings (ddaa)

When trying to convert a Launchpad branch into metadir format, I hit a
bug caused by incorrect url quoting in LocalTransport (one of weaves has
a '/' in its id).

The attached patch fix LocalTransport.list_dir to allow converting
Launchpad to metadir. It should probably be augmented by a number of
test cases, and probably overlaps wildly with John's local transport
quoting branch. But I'm posting it here for the record.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
Partial commit
2
 
**************
3
 
 
4
 
In the first cut, commit covers the whole tree.
5
 
 
6
 
Secondly, people might want to commit only particular files or
7
 
subdirectories.  Semantics of this in the case of renames/adds/deletes
8
 
may be a bit complex, but I think it can be done.  The basic point is
9
 
to build an inventory which includes only the specified changes from
10
 
the working copy. 
11
 
 
12
 
Beyond that, it can be very nice in darcs to commit only selected
13
 
changes to a file.  I think this should not be the default though;
14
 
perhaps only do it with ``-i``.  Even better (for some users) kick off
15
 
a graphical tool to select the particular regions.  Beyond a certain
16
 
point it may become easier for the user to explicitly set aside some
17
 
changes and commit others first.
18
 
 
19
 
In any case, it is a good idea to run a test suite on the revision to
20
 
be committed, to make sure there were no missed dependencies.
 
 
b'\\ No newline at end of file'