~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to doc/developers/packrepo.txt

  • Committer: Vincent Ladeuil
  • Date: 2012-01-18 14:09:19 UTC
  • mto: This revision was merged to the branch mainline in revision 6468.
  • Revision ID: v.ladeuil+lp@free.fr-20120118140919-rlvdrhpc0nq1lbwi
Change set/remove to require a lock for the branch config files.

This means that tests (or any plugin for that matter) do not requires an
explicit lock on the branch anymore to change a single option. This also
means the optimisation becomes "opt-in" and as such won't be as
spectacular as it may be and/or harder to get right (nothing fails
anymore).

This reduces the diff by ~300 lines.

Code/tests that were updating more than one config option is still taking
a lock to at least avoid some IOs and demonstrate the benefits through
the decreased number of hpss calls.

The duplication between BranchStack and BranchOnlyStack will be removed
once the same sharing is in place for local config files, at which point
the Stack class itself may be able to host the changes.

Show diffs side-by-side

added added

removed removed

Lines of Context:
138
138
 
139
139
If you need any help or encounter any problems, please contact the developers
140
140
via the usual ways, i.e. chat to us on IRC or send a message to our mailing
141
 
list. See http://bazaar-vcs.org/BzrSupport for contact details.
 
141
list. See http://wiki.bazaar.canonical.com/BzrSupport for contact details.
142
142
 
143
143
 
144
144
Technical notes
250
250
 
251
251
The ``pack-names`` file gives the list of all finished non-obsolete
252
252
packs.  (This should always be the same as the list of files in the
253
 
``packs/`` directory, but the file is needed for readonly http clients
 
253
``packs/`` directory, but the file is needed for read-only HTTP clients
254
254
that can't easily list directories, and it includes other information.)
255
255
The constraint on the ``pack-names`` list is that every file mentioned
256
256
must exist in the ``packs/`` directory.