~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to TODO

  • Committer: Martin Pool
  • Date: 2005-06-29 02:55:33 UTC
  • Revision ID: mbp@sourcefrog.net-20050629025533-c8fb62423361c8a2
Patch from John:

- StringIO mixes poorly with difflib

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