submodule trailing slash improvements
Get series with b4
b4 am [email protected]Versions newest first
- v2 Latest4 patches 5 repliesIn review
Changes in v2 from the cover letter
* Improvements to existing pathspec code to use is_dir_sep instead of comparing against '/' and handle multiple trailing slashes
* Remove calls to read_cache() made redundant by a new call in builtin/reset.c::parse_args()
Patches in v2 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/4 | pathspec: use is_dir_sep() to check for trailing slashespathspec.c | +3 | 1 reply | |
| 2/4 | pathspec: strip multiple trailing slashes from submodulespathspec.c | +17 | 4 replies | |
| 3/4 | rm: re-use parse_pathspec's trailing-slash removalbuiltin/ | +4 | 0 replies | |
| 4/4 | reset: handle submodule with trailing slashbuiltin/ t/ | +10 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.
Cover letter [PATCH v2 0/4] submodule trailing slash improvements
Changes since v1:
* Improvements to existing pathspec code to use is_dir_sep instead of comparing against '/' and handle multiple trailing slashes
* Remove calls to read_cache() made redundant by a new call in builtin/reset.c::parse_args()
John Keeping (4):
pathspec: use is_dir_sep() to check for trailing slashes
pathspec: strip multiple trailing slashes from submodules
rm: re-use parse_pathspec's trailing-slash removal
reset: handle submodule with trailing slash
builtin/reset.c | 8 ++++++-- builtin/rm.c | 20 ++++---------------- pathspec.c | 30 +++++++++++++++++++----------- t/t7400-submodule-basic.sh | 6 ++++-- 4 files changed, 33 insertions(+), 31 deletions(-)
Signature · 1 line
1.8.4.277.gfbd6843.dirty