~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to TODO

  • Committer: mbp at sourcefrog
  • Date: 2005-03-29 08:01:30 UTC
  • Revision ID: mbp@sourcefrog.net-20050329080130-89b78af279b196b3
match ignore patterns like ./config.h

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