932
by Martin Pool
- more diff TODOs |
1 |
.. -*- mode: indented-text; compile-command: "make -C doc" -*-
|
321
by Martin Pool
doc: revfile storage and related things |
2 |
|
3 |
||
4 |
*******************
|
|
5 |
Things to do in bzr
|
|
6 |
*******************
|
|
7 |
||
287
by Martin Pool
- todo: plugins |
8 |
|
293
by Martin Pool
- todos |
9 |
See also various low-level TODOs in the source code. Try looking in |
284
by Martin Pool
- more TODO items |
10 |
the list archive or on gmane.org for previous discussion of these |
293
by Martin Pool
- todos |
11 |
issues. |
12 |
||
13 |
These are classified by approximate size: an hour or less, a day or |
|
14 |
less, and several days or more. |
|
15 |
||
282
by Martin Pool
- move all TODO items into ./TODO |
16 |
|
17 |
Small things
|
|
18 |
------------
|
|
19 |
||
991
by Martin Pool
todo |
20 |
* ``bzr status`` should show some description of patches that are merged but |
21 |
not yet committed. Preferably in a very compact format so that they |
|
22 |
just fit on a single line; this could also be used for say ``log
|
|
23 |
--tiny``:: |
|
24 |
||
25 |
2005-11-05 mbp Flowers for stephane
|
|
26 |
||
985
by Martin Pool
- update todo list |
27 |
* print a message at the end of running the tests telling them tht the
|
28 |
test log and output exists but can be removed |
|
975
by Martin Pool
- more checks for some operations in subdirectories |
29 |
|
30 |
* tests for running the commit editor, and fix problem of not passing in
|
|
31 |
multiple arguments |
|
32 |
||
852
by Martin Pool
todo |
33 |
* Merging add of a new file clashing with an existing file doesn't
|
34 |
work; add gets an error that it's already versioned and the merge |
|
932
by Martin Pool
- more diff TODOs |
35 |
aborts. |
852
by Martin Pool
todo |
36 |
|
850
by Martin Pool
- Merge merge updates from aaron |
37 |
* Merge should ignore the destination's working directory, otherwise
|
38 |
we get an error about the statcache when pulling from a remote |
|
39 |
branch. |
|
40 |
||
282
by Martin Pool
- move all TODO items into ./TODO |
41 |
* Add of a file that was present in the base revision should put back
|
42 |
the previous file-id. |
|
43 |
||
914
by Martin Pool
- fix up breakage of 'bzr log -v' by root_id patch |
44 |
* Not sure I'm happy with needing to pass a root id to EmptyTree;
|
45 |
comparing anything against an EmptyTree with no root should have the |
|
46 |
same effect(?) |
|
47 |
||
282
by Martin Pool
- move all TODO items into ./TODO |
48 |
* Handle diff of files which do not have a trailing newline; probably
|
49 |
requires patching difflib to get it exactly right, or otherwise |
|
50 |
calling out to GNU diff. |
|
51 |
||
902
by Martin Pool
todo |
52 |
* Should be able to copy files between branches to preserve their
|
53 |
file-id (and perhaps eventually parentage.) |
|
54 |
||
289
by Martin Pool
todo |
55 |
* -r option should take a revision-id as well as a revno.
|
56 |
||
903
by Martin Pool
todo |
57 |
* allow ``bzr st -r 300`` to show a summary of changes since then. |
58 |
||
322
by Martin Pool
- update todo list |
59 |
* ``bzr info`` should count only people with distinct email addresses as |
290
by Martin Pool
todo |
60 |
different committers. (Or perhaps only distinct userids?) |
61 |
||
322
by Martin Pool
- update todo list |
62 |
* On Windows, command-line arguments should be `glob-expanded`__, |
293
by Martin Pool
- todos |
63 |
because the shell doesn't do this. However, there are probably some |
64 |
commands where this shouldn't be done, such as 'bzr ignore', because |
|
65 |
we want to accept globs. |
|
66 |
||
329
by Martin Pool
- refactor command functions into command classes |
67 |
* ``bzr ignore`` command that just adds a line to the ``.bzrignore`` file |
68 |
and makes it versioned. Fix this to break symlinks. |
|
295
by Martin Pool
todo |
69 |
|
312
by Martin Pool
todo |
70 |
* Any useful sanity checks in 'bzr ignore'? Perhaps give a warning if
|
71 |
they try to add a single file which is already versioned, or if they |
|
72 |
add a pattern which already exists, or if it looks like they gave an |
|
73 |
unquoted glob. |
|
310
by Martin Pool
- new 'bzr ignored' command! |
74 |
|
325
by Martin Pool
- more revfile design notes |
75 |
__ http://mail.python.org/pipermail/python-list/2001-April/037847.html |
76 |
||
327
by Martin Pool
todo |
77 |
* Separate read and write version checks?
|
325
by Martin Pool
- more revfile design notes |
78 |
|
405
by Martin Pool
todo |
79 |
* ``bzr status DIR`` should give status on all files under that |
80 |
directory. |
|
81 |
||
484
by Martin Pool
todo |
82 |
* ``bzr log DIR`` should give changes to any files within DIR. |
83 |
||
329
by Martin Pool
- refactor command functions into command classes |
84 |
* ``bzr inventory -r REV`` and perhaps unify this with ``bzr ls``, |
85 |
giving options to display ids, types, etc. |
|
86 |
||
341
by Martin Pool
todo |
87 |
* Split BzrError into various more specific subclasses for different
|
88 |
errors people might want to catch. |
|
326
by Martin Pool
todo |
89 |
|
393
by Martin Pool
todo: export to tarball |
90 |
* If the export destination ends in '.tar', '.tar.gz', etc then create
|
91 |
a tarball instead of a directory. (Need to actually make a |
|
92 |
temporary directory and then tar that up.) |
|
93 |
||
94 |
http://www.gelato.unsw.edu.au/archives/git/0504/2194.html |
|
932
by Martin Pool
- more diff TODOs |
95 |
|
414
by Martin Pool
todo |
96 |
* RemoteBranch could maintain a cache either in memory or on disk. We
|
97 |
know more than an external cache might about which files are |
|
98 |
immutable and which can vary. On the other hand, it's much simpler |
|
99 |
to just use an external proxy cache. |
|
100 |
||
586
by Martin Pool
todo |
101 |
Perhaps ~/.bzr/http-cache. Baz has a fairly simple cache under |
102 |
~/.arch-cache, containing revision information encoded almost as a |
|
103 |
bunch of archives. Perhaps we could simply store full paths. |
|
104 |
||
487
by Martin Pool
todo |
105 |
* Maybe also store directories in the statcache so that we can quickly
|
106 |
identify that they still exist. |
|
107 |
||
570
by Martin Pool
doc |
108 |
* Diff should show timestamps; for files from the working directory we
|
109 |
can use the file itself; for files from a revision we should use the |
|
110 |
commit time of the revision. |
|
111 |
||
593
by Martin Pool
todo |
112 |
* Perhaps split command infrastructure from the actual command
|
113 |
definitions. |
|
114 |
||
115 |
* Cleaner support for negative boolean options like --no-recurse.
|
|
116 |
||
603
by Martin Pool
doc |
117 |
* Statcache should possibly map all file paths to / separators
|
118 |
||
642
by Martin Pool
- notes on patches for Windows |
119 |
* quotefn doubles all backslashes on Windows; this is probably not the
|
120 |
best thing to do. What would be a better way to safely represent |
|
121 |
filenames? Perhaps we could doublequote things containing spaces, |
|
122 |
on the principle that filenames containing quotes are unlikely? |
|
123 |
Nice for humans; less good for machine parsing. |
|
124 |
||
125 |
* Patches should probably use only forward slashes, even on Windows,
|
|
932
by Martin Pool
- more diff TODOs |
126 |
otherwise Unix patch can't apply them. (?) |
642
by Martin Pool
- notes on patches for Windows |
127 |
|
665
by Martin Pool
todo |
128 |
* Branch.update_revisions() inefficiently fetches revisions from the
|
129 |
remote server twice; once to find out what text and inventory they |
|
130 |
need and then again to actually get the thing. This is a bit |
|
932
by Martin Pool
- more diff TODOs |
131 |
inefficient. |
665
by Martin Pool
todo |
132 |
|
133 |
One complicating factor here is that we don't really want to have |
|
134 |
revisions present in the revision-store until all their constituent |
|
135 |
parts are also stored. |
|
136 |
||
137 |
The basic problem is that RemoteBranch.get_revision() and similar |
|
138 |
methods return object, but what we really want is the raw XML, which |
|
139 |
can be popped into our own store. That needs to be refactored. |
|
691
by Martin Pool
todo |
140 |
|
141 |
* ``bzr status FOO`` where foo is ignored should say so. |
|
724
by Martin Pool
- todo: bzr mkdir |
142 |
|
143 |
* ``bzr mkdir A...`` should just create and add A. |
|
818
by Martin Pool
- Clear pending-merge list when committing. |
144 |
|
145 |
* Guard against repeatedly merging any particular patch.
|
|
932
by Martin Pool
- more diff TODOs |
146 |
|
147 |
* More options for diff:
|
|
148 |
||
149 |
- diff two revisions of the same tree
|
|
150 |
||
151 |
- diff two different branches, optionally at different revisions
|
|
152 |
||
153 |
- diff a particular file in another tree against the corresponding
|
|
154 |
version in this tree (which should be the default if the second |
|
155 |
parameter is a tree root) |
|
156 |
||
157 |
- diff everything under a particular directory, in any of the above
|
|
158 |
ways |
|
159 |
||
160 |
- diff two files inside the same tree, even if they have different
|
|
161 |
ids |
|
162 |
||
163 |
- and, of course, tests for all this
|
|
164 |
||
960
by Martin Pool
todo |
165 |
* ``bzr add`` with no arguments should probably be the same as ``bzr add .`` |
932
by Martin Pool
- more diff TODOs |
166 |
|
967
by Martin Pool
todo |
167 |
* Reproducible performance benchmark to measure whether performance is
|
168 |
getting better or worse. |
|
665
by Martin Pool
todo |
169 |
|
414
by Martin Pool
todo |
170 |
|
282
by Martin Pool
- move all TODO items into ./TODO |
171 |
Medium things
|
172 |
-------------
|
|
173 |
||
174 |
* Merge revert patch.
|
|
175 |
||
329
by Martin Pool
- refactor command functions into command classes |
176 |
* ``bzr mv`` that does either rename or move as in Unix. |
282
by Martin Pool
- move all TODO items into ./TODO |
177 |
|
478
by Martin Pool
- put back support for running diff or status on |
178 |
* More efficient diff of only selected files. We should be able to
|
179 |
just get the id for the selected files, look up their location and |
|
180 |
diff just those files. No need to traverse the entire inventories. |
|
282
by Martin Pool
- move all TODO items into ./TODO |
181 |
|
479
by Martin Pool
todo |
182 |
* ``bzr status DIR`` or ``bzr diff DIR`` should report on all changes |
183 |
under that directory. |
|
184 |
||
282
by Martin Pool
- move all TODO items into ./TODO |
185 |
* Fix up Inventory objects to represent root object as an entry.
|
186 |
||
552
by Martin Pool
- update todo list |
187 |
* Don't convert entire entry from ElementTree to an object when it is
|
188 |
read in, but rather wait until the program actually wants to know |
|
189 |
about that node. |
|
282
by Martin Pool
- move all TODO items into ./TODO |
190 |
|
191 |
* Extract changes from one revision to the next to a text form
|
|
192 |
suitable for transmission over email. |
|
193 |
||
194 |
* More test cases.
|
|
195 |
||
491
by Martin Pool
- Selective commit! |
196 |
- Selected-file commit
|
197 |
||
198 |
- Impossible selected-file commit: adding things in non-versioned
|
|
199 |
directories, crossing renames, etc. |
|
200 |
||
282
by Martin Pool
- move all TODO items into ./TODO |
201 |
* Write a reproducible benchmark, perhaps importing various kernel versions.
|
202 |
||
203 |
* Directly import diffs! It seems a bit redundant to need to rescan
|
|
204 |
the directory to work out what files diff added/deleted/changed when |
|
205 |
all the information is there in the diff in the first place. |
|
206 |
Getting the exact behaviour for added/deleted subdirectories etc |
|
207 |
might be hard. |
|
208 |
||
209 |
At the very least we could run diffstat over the diff, or perhaps |
|
210 |
read the status output from patch. Just knowing which files might |
|
211 |
be modified would be enough to guide the add and commit. |
|
932
by Martin Pool
- more diff TODOs |
212 |
|
282
by Martin Pool
- move all TODO items into ./TODO |
213 |
Given this we might be able to import patches at 1/second or better. |
214 |
||
215 |
* Get branch over http.
|
|
216 |
||
217 |
* Pull pure updates over http.
|
|
218 |
||
219 |
* revfile compression.
|
|
220 |
||
221 |
* Split inventory into per-directory files.
|
|
222 |
||
284
by Martin Pool
- more TODO items |
223 |
* Fix ignore file parsing:
|
224 |
||
225 |
- fnmatch is not the same as unix patterns
|
|
226 |
||
227 |
- perhaps add extended globs from rsh/rsync
|
|
228 |
||
229 |
- perhaps a pattern that matches only directories or non-directories
|
|
230 |
||
312
by Martin Pool
todo |
231 |
* Consider using Python logging library as well as/instead of
|
232 |
bzrlib.trace. |
|
233 |
||
334
by Martin Pool
doc |
234 |
* Commands should give some progress indication by default.
|
235 |
||
236 |
- But quieten this with ``--silent``. |
|
237 |
||
312
by Martin Pool
todo |
238 |
* Change to using gettext message localization.
|
282
by Martin Pool
- move all TODO items into ./TODO |
239 |
|
315
by Martin Pool
todo |
240 |
* Make a clearer separation between internal and external bzrlib
|
241 |
interfaces. Make internal interfaces use protected names. Write at |
|
242 |
least some documentation for those APIs, probably as docstrings. |
|
243 |
||
244 |
Consider using ZopeInterface definitions for the external interface; |
|
245 |
I think these are already used in PyBaz. They allow automatic |
|
246 |
checking of the interface but may be unfamiliar to general Python |
|
321
by Martin Pool
doc: revfile storage and related things |
247 |
developers, so I'm not really keen. |
315
by Martin Pool
todo |
248 |
|
249 |
* Commands to dump out all command help into a manpage or HTML file or
|
|
250 |
whatever. |
|
251 |
||
326
by Martin Pool
todo |
252 |
* Handle symlinks in the working directory; at the very least it
|
253 |
should be possible for them to be present and ignored/unknown |
|
932
by Martin Pool
- more diff TODOs |
254 |
without causing assertion failures. |
326
by Martin Pool
todo |
255 |
|
256 |
Eventually symlinks should be versioned. |
|
257 |
||
329
by Martin Pool
- refactor command functions into command classes |
258 |
* Allow init in a subdirectory to create a nested repository, but only
|
259 |
if the subdirectory is not already versioned. Perhaps also require |
|
260 |
a ``--nested`` to protect against confusion.
|
|
261 |
||
932
by Martin Pool
- more diff TODOs |
262 |
* Branch names?
|
329
by Martin Pool
- refactor command functions into command classes |
263 |
|
339
by Martin Pool
many more diffs |
264 |
* More test framework:
|
265 |
||
266 |
- Class that describes the state of a working tree so we can just
|
|
267 |
assert it's equal. |
|
268 |
||
342
by Martin Pool
todo |
269 |
* There are too many methods on Branch() that really manipulate the
|
932
by Martin Pool
- more diff TODOs |
270 |
WorkingTree. They should be moved across. |
346
by Martin Pool
todo |
271 |
|
272 |
Also there are some methods which are duplicated on Tree and |
|
273 |
Inventory objects, and it should be made more clear which ones are |
|
274 |
proxies and which ones behave differently, and how. |
|
342
by Martin Pool
todo |
275 |
|
361
by Martin Pool
todo |
276 |
* Try using XSLT to add some formatting to REST-generated HTML. Or
|
277 |
maybe write a small Python program that specifies a header and foot |
|
278 |
for the pages and calls into the docutils libraries. |
|
279 |
||
366
by Martin Pool
todo |
280 |
* --format=xml for log, status and other commands.
|
329
by Martin Pool
- refactor command functions into command classes |
281 |
|
370
by Martin Pool
todo |
282 |
* Attempting to explicitly add a file that's already added should give
|
283 |
a warning; however there should be no warning for directories (since |
|
284 |
we scan for new children) or files encountered in a directory that's |
|
285 |
being scanned. |
|
329
by Martin Pool
- refactor command functions into command classes |
286 |
|
377
by Martin Pool
- todo notes on inventory |
287 |
* Better handling of possible collisions on case-losing filesystems;
|
288 |
make sure a single file does not get added twice under different |
|
289 |
names. |
|
290 |
||
291 |
* Clean up XML inventory:
|
|
292 |
||
293 |
- Use nesting rather than parent_id pointers.
|
|
294 |
||
295 |
- Hold the ElementTree in memory in the Inventory object and work
|
|
296 |
directly on that, rather than converting into Python objects every |
|
453
by Martin Pool
- Split WorkingTree into its own file |
297 |
time it is read in. Probably still exposoe it through some kind of |
377
by Martin Pool
- todo notes on inventory |
298 |
object interface though, but perhaps that should just be a proxy |
299 |
for the elements. |
|
300 |
||
932
by Martin Pool
- more diff TODOs |
301 |
- Less special cases for the root directory.
|
377
by Martin Pool
- todo notes on inventory |
302 |
|
388
by Martin Pool
doc |
303 |
* Perhaps inventories should remember the revision in which each file
|
304 |
was last changed, as well as its current state? This is a bit |
|
305 |
redundant but might often be interested to know. |
|
306 |
||
442
by Martin Pool
todo |
307 |
* stat cache should perhaps only stat files as necessary, rather than
|
308 |
doing them all up-front. On the other hand, that disallows the |
|
309 |
opimization of stating them in inode order. |
|
310 |
||
451
by Martin Pool
todo |
311 |
* It'd be nice to pipeline multiple HTTP requests. Often we can
|
312 |
predict what will be wanted in future: all revisions, or all texts |
|
932
by Martin Pool
- more diff TODOs |
313 |
in a particular revision, etc. |
451
by Martin Pool
todo |
314 |
|
315 |
urlgrabber's docs say they are working on batched downloads; we |
|
316 |
could perhaps ride on that or just create a background thread (ew). |
|
317 |
||
459
by Martin Pool
- diff now uses stat-cache -- much faster |
318 |
* Paranoid mode where we never trust SHA-1 matches.
|
319 |
||
502
by Martin Pool
todo |
320 |
* Don't commit if there are no changes unless forced.
|
321 |
||
522
by Martin Pool
todo |
322 |
* --dry-run mode for commit? (Or maybe just run with
|
323 |
check-command=false?) |
|
502
by Martin Pool
todo |
324 |
|
325 |
* Generally, be a bit more verbose unless --silent is specified.
|
|
326 |
||
522
by Martin Pool
todo |
327 |
* Function that finds all changes to files under a given directory;
|
328 |
perhaps log should use this if a directory is given. |
|
329 |
||
548
by Martin Pool
- Write statcache using \u style encoding to avoid |
330 |
* XML attributes might have trouble with filenames containing \n and
|
331 |
\r. Do we really want to support this? I think perhaps not. |
|
332 |
||
574
by Martin Pool
todo |
333 |
* Remember execute bits, so that exports will work OK.
|
334 |
||
579
by Martin Pool
todo |
335 |
* Unify smart_add and plain Branch.add(); perhaps smart_add should
|
336 |
just build a list of files to add and pass that to the regular add |
|
337 |
function. |
|
338 |
||
604
by Martin Pool
doc |
339 |
* Function to list a directory, saying in which revision each file was
|
340 |
last modified. Useful for web and gui interfaces, and slow to |
|
341 |
compute one file at a time. |
|
342 |
||
609
by Martin Pool
- cleanup test code |
343 |
* unittest is standard, but the results are kind of ugly; would be
|
344 |
nice to make it cleaner. |
|
345 |
||
627
by Martin Pool
todo |
346 |
* Check locking is correct during merge-related operations.
|
347 |
||
629
by Martin Pool
todo |
348 |
* Perhaps attempts to get locks should timeout after some period of
|
349 |
time, or at least display a progress message. |
|
350 |
||
702
by Martin Pool
todo: bzr upgrade |
351 |
* Split out upgrade functionality from check command into a separate
|
352 |
``bzr upgrade``.
|
|
353 |
||
738
by Martin Pool
- default plugin dir is ~/.bzr.conf/plugins |
354 |
* Don't pass around command classes but rather pass objects. This'd
|
355 |
make it cleaner to construct objects wrapping external commands. |
|
356 |
||
818
by Martin Pool
- Clear pending-merge list when committing. |
357 |
* Track all merged-in revisions in a versioned add-only metafile.
|
358 |
||
995
by Martin Pool
- todo: pull --clobber |
359 |
* ``pull --clobber`` should discard any local changes not present |
360 |
remotely. Not generally what you want, but possibly useful when |
|
361 |
you're just mirroring another branch and want to keep tracking it |
|
362 |
even when they e.g. uncommit or make similar non-forward movements. |
|
363 |
Also for push I suppose. Clobber may not be the best name, maybe |
|
364 |
``--destroy``?
|
|
365 |
||
315
by Martin Pool
todo |
366 |
|
282
by Martin Pool
- move all TODO items into ./TODO |
367 |
Large things
|
368 |
------------
|
|
369 |
||
321
by Martin Pool
doc: revfile storage and related things |
370 |
* Generate annotations from current file relative to previous
|
371 |
annotations. |
|
372 |
||
373 |
- Is it necessary to store any kind of annotation where data was
|
|
374 |
deleted? |
|
375 |
||
325
by Martin Pool
- more revfile design notes |
376 |
* Update revfile_ format and make it active:
|
321
by Martin Pool
doc: revfile storage and related things |
377 |
|
378 |
- Texts should be identified by something keyed on the revision, not
|
|
379 |
an individual text-id. This is much more useful for annotate I |
|
380 |
think; we want to map back to the revision that last changed it. |
|
381 |
||
382 |
- Access revfile revisions through the Tree/Store classes.
|
|
383 |
||
384 |
- Check them from check commands.
|
|
385 |
||
386 |
- Store annotations.
|
|
387 |
||
325
by Martin Pool
- more revfile design notes |
388 |
.. _revfile: revfile.html |
389 |
||
294
by Martin Pool
todo |
390 |
* Hooks for pre-commit, post-commit, etc.
|
391 |
||
392 |
Consider the security implications; probably should not enable hooks |
|
393 |
for remotely-fetched branches by default. |
|
394 |
||
395 |
* Pre-commit check. If this hook is defined, it needs to be handled
|
|
396 |
specially: create a temporary directory containing the tree as it |
|
397 |
will be after the commit. This means excluding any ignored/unknown |
|
398 |
files, and respecting selective commits. Run the pre-commit check |
|
399 |
(e.g. compile and run test suite) in there. |
|
400 |
||
519
by Martin Pool
- todo: discussion of pre-commit tests |
401 |
Possibly this should be done by splitting the commit function into |
402 |
several parts (under a single interface). It is already rather |
|
403 |
large. Decomposition: |
|
404 |
||
405 |
- find tree modifications and prepare in-memory inventory
|
|
406 |
||
407 |
- export that inventory to a temporary directory
|
|
408 |
||
409 |
- run the test in that temporary directory
|
|
410 |
||
411 |
- if that succeeded, continue to actually finish the commit
|
|
412 |
||
413 |
What should be done with the text of modified files while this is |
|
414 |
underway? I don't think we want to count on holding them in memory |
|
415 |
and we can't trust the working files to stay in one place so I |
|
416 |
suppose we need to move them into the text store, or otherwise into |
|
417 |
a temporary directory. |
|
418 |
||
419 |
If the commit does not actually complete, we would rather the |
|
932
by Martin Pool
- more diff TODOs |
420 |
content was not left behind in the stores. |
519
by Martin Pool
- todo: discussion of pre-commit tests |
421 |
|
282
by Martin Pool
- move all TODO items into ./TODO |
422 |
* Web interface
|
423 |
||
424 |
* GUI (maybe in Python GTK+?)
|
|
425 |
||
284
by Martin Pool
- more TODO items |
426 |
* C library interface
|
321
by Martin Pool
doc: revfile storage and related things |
427 |
|
428 |
* Expansion of $Id$ keywords within working files. Perhaps do this in
|
|
429 |
exports first as a simpler case because then we don't need to deal |
|
430 |
with removing the tags on the way back in. |
|
431 |
||
329
by Martin Pool
- refactor command functions into command classes |
432 |
* ``bzr find`` |