[PATCH 0/3] check new.tags for invalid tags
Jani Nikula writes: > Clearly broken. This series fixes the issue at the cli > level. (Forbidding empty tags at the lib level is slightly more > complicated, as we would still have to ensure old dump files can be > restored.) pushed, with the 4th fixup patch. d
Re: [PATCH 0/3] check new.tags for invalid tags
Jani Nikula j...@nikula.org writes: Clearly broken. This series fixes the issue at the cli level. (Forbidding empty tags at the lib level is slightly more complicated, as we would still have to ensure old dump files can be restored.) pushed, with the 4th fixup patch. d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
[PATCH 0/3] check new.tags for invalid tags
On Sun, Feb 23 2014, Jani Nikula wrote: > On Sun, 23 Feb 2014, Rob Browning wrote: >> In the [new] section, "tags=;" will cause notmuch to create empty tags >> that are fairly hard to remove from the command line. > > Clearly broken. This series fixes the issue at the cli > level. (Forbidding empty tags at the lib level is slightly more > complicated, as we would still have to ensure old dump files can be > restored.) > >> After some help on #bup, here's what I came up with to remove them, >> though it assumes that the empty tag "+ " will always be first in dump's >> output: >> >> notmuch dump --format=batch-tag 'tag:""' | perl -pe 's/^\+ //' \ >> | notmuch restore --format=batch-tag >> >> And note that you have to use restore, "notmuch tag --batch" doesn't >> appear to accept "- " as a tag, even though dump will produce "+ ". > > I didn't check this further, but the regular, non-batch notmuch tag > should still work for removal of empty tags. LGTM. $ notmuch tag + -- id:edc2bc900f75bb2e72be2037e2df9105be7f0273.1393174108.git.jani at nikula.org Error: empty tag forbiddenzsh: exit 1 notmuch tag + -- $ notmuch restore --accumulate + id:edc2bc900f75bb2e72be2037e2df9105be7f0273.1393174108.git.jani at nikula.org -- notmuch search shows space after ( to inform there is empty tag. $ notmuch tag - -- id:edc2bc900f75bb2e72be2037e2df9105be7f0273.1393174108.git.jani at nikula.org notmuch search no longer shows space after ( Also, SomeBody(tm) should add '\n' to the fprintf() in tag-util.c:175 (line number after applying these patches). Tomi > > BR, > Jani. > > Jani Nikula (3): > cli: export function for illegal tag checking > cli: make sure notmuch new and insert don't add invalid tags > test: add tests for invalid new.tags > > notmuch-insert.c| 9 + > notmuch-new.c | 14 +- > tag-util.c | 9 + > tag-util.h | 12 > test/T050-new.sh| 17 + > test/T070-insert.sh | 19 +++ > 6 files changed, 71 insertions(+), 9 deletions(-) > > -- > 1.8.5.3 > > ___ > notmuch mailing list > notmuch at notmuchmail.org > http://notmuchmail.org/mailman/listinfo/notmuch
[PATCH 0/3] check new.tags for invalid tags
On Sun, 23 Feb 2014, Rob Browning wrote: > In the [new] section, "tags=;" will cause notmuch to create empty tags > that are fairly hard to remove from the command line. Clearly broken. This series fixes the issue at the cli level. (Forbidding empty tags at the lib level is slightly more complicated, as we would still have to ensure old dump files can be restored.) > After some help on #bup, here's what I came up with to remove them, > though it assumes that the empty tag "+ " will always be first in dump's > output: > > notmuch dump --format=batch-tag 'tag:""' | perl -pe 's/^\+ //' \ > | notmuch restore --format=batch-tag > > And note that you have to use restore, "notmuch tag --batch" doesn't > appear to accept "- " as a tag, even though dump will produce "+ ". I didn't check this further, but the regular, non-batch notmuch tag should still work for removal of empty tags. BR, Jani. Jani Nikula (3): cli: export function for illegal tag checking cli: make sure notmuch new and insert don't add invalid tags test: add tests for invalid new.tags notmuch-insert.c| 9 + notmuch-new.c | 14 +- tag-util.c | 9 + tag-util.h | 12 test/T050-new.sh| 17 + test/T070-insert.sh | 19 +++ 6 files changed, 71 insertions(+), 9 deletions(-) -- 1.8.5.3
[PATCH 0/3] check new.tags for invalid tags
On Sun, 23 Feb 2014, Rob Browning r...@defaultvalue.org wrote: In the [new] section, tags=; will cause notmuch to create empty tags that are fairly hard to remove from the command line. Clearly broken. This series fixes the issue at the cli level. (Forbidding empty tags at the lib level is slightly more complicated, as we would still have to ensure old dump files can be restored.) After some help on #bup, here's what I came up with to remove them, though it assumes that the empty tag + will always be first in dump's output: notmuch dump --format=batch-tag 'tag:' | perl -pe 's/^\+ //' \ | notmuch restore --format=batch-tag And note that you have to use restore, notmuch tag --batch doesn't appear to accept - as a tag, even though dump will produce + . I didn't check this further, but the regular, non-batch notmuch tag should still work for removal of empty tags. BR, Jani. Jani Nikula (3): cli: export function for illegal tag checking cli: make sure notmuch new and insert don't add invalid tags test: add tests for invalid new.tags notmuch-insert.c| 9 + notmuch-new.c | 14 +- tag-util.c | 9 + tag-util.h | 12 test/T050-new.sh| 17 + test/T070-insert.sh | 19 +++ 6 files changed, 71 insertions(+), 9 deletions(-) -- 1.8.5.3 ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH 0/3] check new.tags for invalid tags
On Sun, Feb 23 2014, Jani Nikula j...@nikula.org wrote: On Sun, 23 Feb 2014, Rob Browning r...@defaultvalue.org wrote: In the [new] section, tags=; will cause notmuch to create empty tags that are fairly hard to remove from the command line. Clearly broken. This series fixes the issue at the cli level. (Forbidding empty tags at the lib level is slightly more complicated, as we would still have to ensure old dump files can be restored.) After some help on #bup, here's what I came up with to remove them, though it assumes that the empty tag + will always be first in dump's output: notmuch dump --format=batch-tag 'tag:' | perl -pe 's/^\+ //' \ | notmuch restore --format=batch-tag And note that you have to use restore, notmuch tag --batch doesn't appear to accept - as a tag, even though dump will produce + . I didn't check this further, but the regular, non-batch notmuch tag should still work for removal of empty tags. LGTM. $ notmuch tag + -- id:edc2bc900f75bb2e72be2037e2df9105be7f0273.1393174108.git.j...@nikula.org Error: empty tag forbiddenzsh: exit 1 notmuch tag + -- $ notmuch restore --accumulate + id:edc2bc900f75bb2e72be2037e2df9105be7f0273.1393174108.git.j...@nikula.org ctrl-d -- exit value 0 -- notmuch search shows space after ( to inform there is empty tag. $ notmuch tag - -- id:edc2bc900f75bb2e72be2037e2df9105be7f0273.1393174108.git.j...@nikula.org no output -- exit value 0 notmuch search no longer shows space after ( Also, SomeBody(tm) should add '\n' to the fprintf() in tag-util.c:175 (line number after applying these patches). Tomi BR, Jani. Jani Nikula (3): cli: export function for illegal tag checking cli: make sure notmuch new and insert don't add invalid tags test: add tests for invalid new.tags notmuch-insert.c| 9 + notmuch-new.c | 14 +- tag-util.c | 9 + tag-util.h | 12 test/T050-new.sh| 17 + test/T070-insert.sh | 19 +++ 6 files changed, 71 insertions(+), 9 deletions(-) -- 1.8.5.3 ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch