~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to TODO

  • Committer: Martin Pool
  • Date: 2005-07-23 13:59:30 UTC
  • Revision ID: mbp@sourcefrog.net-20050723135930-d81530c82c925cb0
- less dodgy is_inside function

Show diffs side-by-side

added added

removed removed

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