Bug 1750281: Bump compare-locales from 8.1.0 to 8.2.1 r=eemeli

Resolves race condition on creating parent directories when run in parallel. Differential Revision: https://phabricator.services.mozilla.com/D136278
Brli · Jan 18, 2022 · a8e8758 · a8e8758
1 parent a7c1adf
commit a8e8758
Show file tree

Hide file tree

Showing 13 changed files with 121 additions and 167 deletions.
diff --git a/third_party/python/compare_locales/compare_locales-8.1.0.dist-info/DESCRIPTION.rst b/third_party/python/compare_locales/compare_locales-8.1.0.dist-info/DESCRIPTION.rst
diff --git a/third_party/python/compare_locales/compare_locales-8.1.0.dist-info/metadata.json b/third_party/python/compare_locales/compare_locales-8.1.0.dist-info/metadata.json
diff --git a/.../compare_locales-8.1.0.dist-info/METADATA → .../compare_locales-8.2.1.dist-info/METADATA b/.../compare_locales-8.1.0.dist-info/METADATA → .../compare_locales-8.2.1.dist-info/METADATA
@@ -1,6 +1,6 @@
-Metadata-Version: 2.0
+Metadata-Version: 2.1
 Name: compare-locales
-Version: 8.1.0
+Version: 8.2.1
 Summary: Lint Mozilla localizations
 Home-page: UNKNOWN
 Author: Axel Hecht
@@ -18,6 +18,8 @@ Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.5
 Classifier: Programming Language :: Python :: 3.6
 Classifier: Programming Language :: Python :: 3.7
+Classifier: Programming Language :: Python :: 3.8
+Classifier: Programming Language :: Python :: 3.9
 Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Classifier: Topic :: Software Development :: Localization
 Classifier: Topic :: Software Development :: Testing
@@ -27,7 +29,7 @@ Requires-Dist: fluent.syntax (<0.19,>=0.18.0)
 Requires-Dist: pytoml
 Requires-Dist: six
 
-[![Build Status](https://travis-ci.org/Pike/compare-locales.svg?branch=master)](https://travis-ci.org/Pike/compare-locales)
+![Build tests](https://github.com/mozilla/compare-locales/workflows/test/badge.svg)
 # compare-locales
 Lint Mozilla localizations
 

diff --git a/...es/compare_locales-8.1.0.dist-info/RECORD → ...es/compare_locales-8.2.1.dist-info/RECORD b/...es/compare_locales-8.1.0.dist-info/RECORD → ...es/compare_locales-8.2.1.dist-info/RECORD
diff --git a/...les/compare_locales-8.1.0.dist-info/WHEEL → ...les/compare_locales-8.2.1.dist-info/WHEEL b/...les/compare_locales-8.1.0.dist-info/WHEEL → ...les/compare_locales-8.2.1.dist-info/WHEEL
@@ -1,5 +1,5 @@
 Wheel-Version: 1.0
-Generator: bdist_wheel (0.30.0)
+Generator: bdist_wheel (0.37.1)
 Root-Is-Purelib: true
 Tag: py2-none-any
 Tag: py3-none-any

diff --git a/..._locales-8.1.0.dist-info/entry_points.txt → ..._locales-8.2.1.dist-info/entry_points.txt b/..._locales-8.1.0.dist-info/entry_points.txt → ..._locales-8.2.1.dist-info/entry_points.txt
diff --git a/...are_locales-8.1.0.dist-info/top_level.txt → ...are_locales-8.2.1.dist-info/top_level.txt b/...are_locales-8.1.0.dist-info/top_level.txt → ...are_locales-8.2.1.dist-info/top_level.txt
diff --git a/third_party/python/compare_locales/compare_locales/__init__.py b/third_party/python/compare_locales/compare_locales/__init__.py
@@ -1 +1 @@
-version = "8.1.0"
+version = "8.2.1"
diff --git a/third_party/python/compare_locales/compare_locales/checks/android.py b/third_party/python/compare_locales/compare_locales/checks/android.py
@@ -251,3 +251,15 @@ def check_params(params, string):
                 "Mismatching formatter",
                 "android"
             )
+    # All parameters used in the reference are expected to be included.
+    # Warn if this isn't the case.
+    for order in params:
+        if order not in sorted(lparams):
+            yield (
+                "warning",
+                0,
+                "Formatter %{}${} not found in translation".format(
+                    order, params[order]
+                ),
+                "android",
+            )
diff --git a/third_party/python/compare_locales/compare_locales/compare/content.py b/third_party/python/compare_locales/compare_locales/compare/content.py
@@ -34,8 +34,7 @@ def __init__(self, quiet=0):
 
     def create_merge_dir(self, merge_file):
         outdir = mozpath.dirname(merge_file)
-        if not os.path.isdir(outdir):
-            os.makedirs(outdir)
+        os.makedirs(outdir, exist_ok=True)
 
     def merge(self, ref_entities, ref_file, l10n_file, merge_file,
               missing, skips, ctx, capabilities, encoding):

diff --git a/third_party/python/compare_locales/compare_locales/plurals.py b/third_party/python/compare_locales/compare_locales/plurals.py
@@ -167,6 +167,7 @@
     'sat': 1,
     'sc': 1,
     'scn': 1,
+    'sco': 1,
     'si': 1,
     'sk': 8,
     'sl': 10,
@@ -181,6 +182,7 @@
     'ta': 1,
     'ta': 1,
     'te': 1,
+    'tg': 1,
     'th': 0,
     'tl': 1,
     'tn': 1,

diff --git a/third_party/python/requirements.in b/third_party/python/requirements.in
@@ -6,7 +6,7 @@ cbor2==4.0.1
 # define it here because it's needed by other dependencies on
 # Windows systems.
 colorama==0.4.4
-compare-locales==8.1.0
+compare-locales==8.2.1
 cookies==2.2.1
 coverage==5.1
 cram==0.7

diff --git a/third_party/python/requirements.txt b/third_party/python/requirements.txt
@@ -90,9 +90,9 @@ colorama==0.4.4 \
     --hash=sha256:5941b2b48a20143d2267e95b1c2a7603ce057ee39fd88e7329b0c292aa16869b \
     --hash=sha256:9f47eda37229f68eee03b24b9748937c7dc3868f906e8ba69fbcbdd3bc5dc3e2
     # via -r requirements-mach-vendor-python.in
-compare-locales==8.1.0 \
-    --hash=sha256:286270797ce64f7a2f25e734bb437870661409884a4f0971c0bb94fdad6c1f35 \
-    --hash=sha256:3d374ff959d5de2cfd5b94caf6b0fa61445f1d8ede5af384002cb3542aacad3a
+compare-locales==8.2.1 \
+    --hash=sha256:470d50d96c68f8e147daa3d70f29a7b750adefea450c5fa07e0f666c8083d854 \
+    --hash=sha256:e6a1610151d357e74ee6c1f5e944f1868e449f83e478c84d92f7b86132f721d7
     # via
     #   -r requirements-mach-vendor-python.in
     #   fluent.migrate