comparison tests/test-hgignore.t @ 49603:3eda36e9b3d6 stable

matcher: fix issues regex flag contained in pattern (issue6759)

Python 3.11 is now enforcing that flag must be at the beginning of the regex

This creates a serious regression for people using Python 3.11 with an hgignore
using flag in a "relre" pattern.

We now detect any flags in such pattern and "prepend" our ".*" pattern after them.

In addition, we now insert the flag in the regexp to only affect the pattern we
are rewriting. Otherwise, the regex built from the combined pattern would these
flags in the middle of it anyway.

As a side effect of this last change, we fix a bug… before this change regex
flag in a pattern would affect all combined patterns. That was bad and is not
longer the case.

The Rust code needs to be updated to fix that very bug, but we will do it in
another changeset.

49602:4e3d86565327

  $ echo "*.o" > .hgignore
  $ hg status
  abort: $TESTTMP/ignorerepo/.hgignore: invalid pattern (relre): *.o (glob)
  [255]

  $ echo 're:^(?!a).*\.o$' > .hgignore
  $ hg status
  A dir/b.o
  ? .hgignore

49603:3eda36e9b3d6

  $ echo "*.o" > .hgignore
  $ hg status
  abort: $TESTTMP/ignorerepo/.hgignore: invalid pattern (relre): *.o (glob)
  [255]

Test relre with flags (issue6759)
---------------------------------

regexp with flag is the first one

  $ echo 're:(?i)\.O$' > .hgignore
  $ echo 're:.hgignore' >> .hgignore
  $ hg status
  A dir/b.o
  ? a.c
  ? syntax

regex with flag is not the first one

  $ echo 're:.hgignore' > .hgignore
  $ echo 're:(?i)\.O$' >> .hgignore
  $ hg status
  A dir/b.o
  ? a.c
  ? syntax

flag in a pattern should affect that pattern only

  $ echo 're:(?i)\.O$' > .hgignore
  $ echo 're:.HGIGNORE' >> .hgignore
  $ hg status
  A dir/b.o
  ? .hgignore (no-rust !)
  ? .hgignore (rust missing-correct-output !)
  ? a.c
  ? syntax

  $ echo 're:.HGIGNORE' > .hgignore
  $ echo 're:(?i)\.O$' >> .hgignore
  $ hg status
  A dir/b.o
  ? .hgignore
  ? a.c
  ? syntax


further testing
---------------

  $ echo 're:^(?!a).*\.o$' > .hgignore
  $ hg status
  A dir/b.o
  ? .hgignore