[RFC/PATCHv10 00/11] git notes

Hi,

Here is the 10th iteration of the git-notes series. Changes in this iteration are as follows:

Changes to existing patches:

- Rebased onto 488bdf2... (Fix crasher on encountering SHA1-like non-note
  in notes tree) (There are no conflicts resolved by this, but a test in
  this series crashes without the crash fix in 488bdf2...).

- Yet another rewrite of the fast-import patch. After trying to implement
  Shawn's suggestions, I found that hacking the mode bits of note entries
  could not work, since they are stripped by a mktree()/load_tree()
  round-trip. The new versions treats _all_ entries with note-compatible
  paths (40 hex chars, not including directory separators) as note entries
  (to be subjected to fanout adjustments) in branches where there are note
  activity (one or more 'N' commands).
  Branches without note activity are not touched, of course.

- Otherwise, Shawn's suggestions to the previous iteration have been
  incorporated.

- Extended t9301 tests to verify that non-notes residing in a notes tree
  are not clobbered by the fast-import patch.

- Fix t9301 test #12 to not "cheat" (by 'deleteall' followed by a few
  additions). Instead, remove notes one-by-one to verify correct
  fanout consolidation.

- Minor cleanups here and there

If Shawn is OK with the fast-import patch, I believe that at least patches #1 - #3 (and possibly #4 - #5) are ready for 'next'.

Patches #6 - #11 drastically extend the notes API. Since there are currently no users of that API, and it has not been discussed much on the list (although these patches have already been present in a few iterations), I would still consider them RFC quality.

TODO:
- Builtin-ify git-notes shell script to take advantage of notes API
- Garbage collect notes whose referenced object is unreachable (gc_notes())
- Handle note objects that are not blobs, but trees

Have fun! :)

...Johan

Johan Herland (11):
  fast-import: Proper notes tree manipulation
  Rename t9301 to t9350, to make room for more fast-import tests
  Add more testcases to test fast-import of notes
  Minor style fixes to notes.c
  Notes API: get_commit_notes() -> format_note() + remove the commit restriction
  Notes API: init_notes(): Initialize the notes tree from the given notes ref
  Notes API: add_note(): Add note objects to the internal notes tree structure
  Notes API: get_note(): Return the note annotating the given object
  Notes API: for_each_note(): Traverse the entire notes tree with a callback
  Notes API: Allow multiple concurrent notes trees with new struct notes_tree
  Refactor notes concatenation into a flexible interface for combining notes

 fast-import.c                                    |  134 +++++-
 notes.c                                          |  345 +++++++++----
 notes.h                                          |  114 ++++-
 pretty.c                                         |    9 +-
 t/t9300-fast-import.sh                           |  156 +++++-
 t/t9301-fast-import-notes.sh                     |  623 ++++++++++++++++++++++
 t/{t9301-fast-export.sh => t9350-fast-export.sh} |    0
 7 files changed, 1259 insertions(+), 122 deletions(-)
 create mode 100755 t/t9301-fast-import-notes.sh
 rename t/{t9301-fast-export.sh => t9350-fast-export.sh} (100%)