~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to TODO

  • Committer: mbp at sourcefrog
  • Date: 2005-03-09 06:49:00 UTC
  • Revision ID: mbp@sourcefrog.net-20050309064900-74935ffb7350b24b
import more files from baz

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