~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to TODO

  • Committer: Martin Pool
  • Date: 2005-07-27 14:19:23 UTC
  • Revision ID: mbp@sourcefrog.net-20050727141923-9f4f14835da8a2db
- clean up code that loads modules to test, taking advantage of less broken
  testLoader in python 2.4

Show diffs side-by-side

added added

removed removed

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