[PATCHv2 0/5] pathspec labels [WAS: submodule groups]

I am happy with the results of the label experiments. (Mainly writing the tests)

See the new tests what I mean. It cleared up for me as well when writing them.

TRUE and FALSE are solved issues in this series. However we still want funny characters not be used. What if we want to use them as special characters in the future? (e.g. 'label*' for 'labelA' or 'labelB' or label{A,B} for the same)

I think it is too late in the matching, so ideally we would check label names when they are assigned (i.e. while editing the .gitattributes file) That is not in our control, though.

And during use of match_pathspec it is already to late to die(...) for bad labels, because the labels may not be in control for the user (She just cloned the repository and wants to inspect certain files, which are labeled "$*&^".) So for now we just output a warning for these cases, in the hope that content creatos upstream respect the recommendation for labels.

Feedback welcome!

Thanks, Stefan

Stefan Beller (5):
  Documentation: fix a typo
  Documentation: correct typo in example for querying attributes
  pathspec: move long magic parsing out of prefix_pathspec
  pathspec: move prefix check out of the inner loop
  pathspec: record labels

 Documentation/gitattributes.txt               |  14 ++-
 Documentation/glossary-content.txt            |   4 +
 Documentation/technical/api-gitattributes.txt |   2 +-
 dir.c                                         |  50 ++++++++
 pathspec.c                                    | 110 +++++++++++------
 pathspec.h                                    |   1 +
 t/t6134-pathspec-with-labels.sh               | 164 ++++++++++++++++++++++++++
 7 files changed, 305 insertions(+), 40 deletions(-)
 create mode 100755 t/t6134-pathspec-with-labels.sh