1879120 - Remove all python deprecation warning: invalid escape sequence

Status: RESOLVED FIXED

(Firefox Build System :: General, task)

Description

[[:sergesanspaille]]

Attachment: literal checker

The script attached can be used to scan existing python code for invalid escape sequence. It's more systematic than the best-effort approach we currently have.

Comment 1

[[:sergesanspaille]]

Running

git ls-tree -r HEAD --name-only | while read line; do case $line in ; *.py) ~/sources/litlint/litlint $line ;; esac; done 2>&1 | grep -v -E '^third_party/' | tee litlint.log

And cleaning up the logfile with

grep invalid\ escape litlint.log | grep -v -E '^third_party/' -c

yield 146 entries on current tree

Comment 2

[[:sergesanspaille]]

Attachment: Bug 1879120 - Remove all python deprecation warning: invalid escape sequence r=ahochheiden! UPGRADE_NSPR_RELEASE

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9efb6f330567
Remove all python deprecation warning: invalid escape sequence r=ahochheiden,webdriver-reviewers,perftest-reviewers,afinder UPGRADE_NSPR_RELEASE

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/44782 for changes under testing/web-platform/tests

Comment 5

[pstanciu]

https://hg.mozilla.org/mozilla-central/rev/9efb6f330567

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot

Comment 7

[Kai Engert (:KaiE:)]

Hello. Please do not use UPGRADE_NSPR_RELEASE for allowing individual commits in the nsprpub directory. The barrier to add that tag is intended to let developers know that individual changes need to be added to the separate NSPR project, and that the nsprpub directory should be upgraded as a whole, only. All individual commits in the nsprpub directory are already overwritten the next time a NSPR release is uplifted.