[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#929452: marked as done (release.debian.org: [pre-approval] testing-proposed-updates for unicode changes)



Your message dated Sat, 25 May 2019 20:56:00 +0000
with message-id <9ffc15ce-ec8c-3da6-3080-5c1f1a404eee@thykier.net>
and subject line Re: Bug#929452: release.debian.org: [pre-approval] testing-proposed-updates for unicode changes
has caused the Debian Bug report #929452,
regarding release.debian.org: [pre-approval] testing-proposed-updates for unicode changes
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
929452: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=929452
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal

Hi all,

dur to unicode change, 2 nodejs packages require an update:
 - node-regenerate-unicode-properties
 - node-regexpu-core

These 2 packages have been updated in unstable so can no more be updated
using the normal way. The proposed changes are very few (patch to update
package.json + debian/control dependency update from node-unicode-11.0.0
to node-unicode-12.0.0).

These updates are required since they are both dependencies of node-rollup
which is a build dependency of ~20 packages.

Do you authorize me to upload these 2 +deb10u1 packages in
testing-proposed-updates? Packages are tested locally, build +
autopkgtest OK.

Sorry for the inconvenience.

Cheers,
Xavier

-- System Information:
Debian Release: 10.0
  APT prefers testing
  APT policy: (900, 'testing'), (500, 'testing-proposed-updates'), (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.19.0-4-amd64 (SMP w/8 CPU cores)
Kernel taint flags: TAINT_OOT_MODULE, TAINT_UNSIGNED_MODULE
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8), LANGUAGE= (charmap=UTF-8)
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled
diff --git a/debian/changelog b/debian/changelog
index 7389de9..66212c6 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+node-regenerate-unicode-properties (7.0.0+ds-1+deb10u1) UNRELEASED; urgency=medium
+
+  * Team upload
+  * Update node-unicode dependency to 12 (Closes: #929447)
+
+ -- Xavier Guimard <yadd@debian.org>  Thu, 23 May 2019 20:28:45 +0200
+
 node-regenerate-unicode-properties (7.0.0+ds-1) unstable; urgency=medium
 
   * New upstream release.
diff --git a/debian/control b/debian/control
index 22119fe..ace86b4 100644
--- a/debian/control
+++ b/debian/control
@@ -8,7 +8,7 @@ Build-Depends: debhelper (>= 11),
                node-jsesc (>= 2.5.1),
                node-regenerate (>= 1.3.2),
                node-unicode-canonical-property-names-ecmascript (>= 1.0.4),
-               node-unicode-11.0.0,
+               node-unicode-12.0.0,
                nodejs (>= 6)
 Standards-Version: 4.1.5
 Homepage: https://github.com/mathiasbynens/regenerate-unicode-properties
diff --git a/debian/patches/series b/debian/patches/series
new file mode 100644
index 0000000..0b7dbe1
--- /dev/null
+++ b/debian/patches/series
@@ -0,0 +1 @@
+unicode-12.diff
diff --git a/debian/patches/unicode-12.diff b/debian/patches/unicode-12.diff
new file mode 100644
index 0000000..10a50f8
--- /dev/null
+++ b/debian/patches/unicode-12.diff
@@ -0,0 +1,28 @@
+Description: update unicode to 12
+Author: Xavier Guimard <yadd@debian.org>
+Bug-Debian: https://bugs.debian.org/929447
+Forwarded: not-needed
+Last-Update: 2019-05-23
+
+--- a/build.js
++++ b/build.js
+@@ -4,7 +4,7 @@
+ const jsesc = require('jsesc');
+ const emptyDirSync = require('fs-extra').emptyDirSync;
+ const regenerate = require('regenerate');
+-const UNICODE_VERSION = '11.0.0';
++const UNICODE_VERSION = '12.0.0';
+ const unicode = require(`unicode-${ UNICODE_VERSION }`);
+ 
+ /*----------------------------------------------------------------------------*/
+--- a/package.json
++++ b/package.json
+@@ -38,7 +38,7 @@
+     "ava": "^0.25.0",
+     "fs-extra": "^6.0.1",
+     "jsesc": "^2.5.1",
+-    "unicode-11.0.0": "^0.7.7",
++    "unicode-12.0.0": "^0.7.7",
+     "unicode-canonical-property-names-ecmascript": "^1.0.4"
+   },
+   "scripts": {
diff --git a/debian/changelog b/debian/changelog
index 1440a50..bf94c5f 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+node-regexpu-core (4.4.0+ds-1+deb10u1) UNRELEASED; urgency=medium
+
+  * Team upload
+  * Update node-unicode dependency to 12 (Closes: #929448)
+
+ -- Xavier Guimard <yadd@debian.org>  Thu, 23 May 2019 20:48:12 +0200
+
 node-regexpu-core (4.4.0+ds-1) unstable; urgency=medium
 
   * New upstream release.
diff --git a/debian/control b/debian/control
index 043f294..15a710e 100644
--- a/debian/control
+++ b/debian/control
@@ -7,7 +7,7 @@ Build-Depends: debhelper (>= 11),
                node-jsesc (>= 2.5.1),
                node-lodash (>= 4.17.4),
                node-regenerate (>= 1.4.0),
-               node-unicode-11.0.0,
+               node-unicode-12.0.0,
                nodejs (>= 6)
 Standards-Version: 4.2.1
 Homepage: https://mths.be/regexpu
diff --git a/debian/patches/series b/debian/patches/series
new file mode 100644
index 0000000..1648c7a
--- /dev/null
+++ b/debian/patches/series
@@ -0,0 +1 @@
+unicode-12.patch
diff --git a/debian/patches/unicode-12.patch b/debian/patches/unicode-12.patch
new file mode 100644
index 0000000..483093c
--- /dev/null
+++ b/debian/patches/unicode-12.patch
@@ -0,0 +1,51 @@
+Description: update node-unicode dependency to 12
+Author: Xavier Guimard <yadd@debian.org>
+Bug-Debian: https://bugs.debian.org/929448
+Forwarded: not-needed
+Last-Update: 2019-05-23
+
+--- a/package.json
++++ b/package.json
+@@ -62,6 +62,6 @@
+ 		"lodash": "^4.17.10",
+ 		"mocha": "^5.2.0",
+ 		"regexpu-fixtures": "^2.1.2",
+-		"unicode-11.0.0": "^0.7.7"
++		"unicode-12.0.0": "^0.7.7"
+ 	}
+ }
+--- a/scripts/character-class-escape-sets.js
++++ b/scripts/character-class-escape-sets.js
+@@ -4,7 +4,7 @@
+ const jsesc = require('jsesc');
+ const regenerate = require('regenerate');
+ 
+-const Zs = require('unicode-11.0.0/General_Category/Space_Separator/code-points.js');
++const Zs = require('unicode-12.0.0/General_Category/Space_Separator/code-points.js');
+ 
+ const iuMappings = require('../data/iu-mappings.js');
+ 
+--- a/scripts/iu-mappings.js
++++ b/scripts/iu-mappings.js
+@@ -81,8 +81,8 @@
+ //  A. To do a simple case folding, use the mappings with status C + S.
+ //  B. To do a full case folding, use the mappings with status C + F.
+ 
+-const commonMappings = require('unicode-11.0.0/Case_Folding/C/code-points.js');
+-const simpleMappings = require('unicode-11.0.0/Case_Folding/S/code-points.js');
++const commonMappings = require('unicode-12.0.0/Case_Folding/C/code-points.js');
++const simpleMappings = require('unicode-12.0.0/Case_Folding/S/code-points.js');
+ 
+ // We want the `C` mappings in both directions (i.e. `A` should fold to `a`
+ // and `a` to `A`), and the `S` mappings in both directions (i.e. `ẞ` should
+--- a/tests/tests.js
++++ b/tests/tests.js
+@@ -498,7 +498,7 @@
+ ];
+ 
+ const getPropertyValuePattern = (path) => {
+-	const codePoints = require(`unicode-11.0.0/${
++	const codePoints = require(`unicode-12.0.0/${
+ 		path }/code-points.js`);
+ 	return {
+ 		'p': regenerate(codePoints).toString(),

--- End Message ---
--- Begin Message ---
Xavier:
> Control: tags -1 - moreinfo
> 
> Le 23/05/2019 à 22:25, Niels Thykier a écrit :
>> Control: tags -1 moreinfo
>>
>> Xavier Guimard:
>>> Package: release.debian.org
>>> Severity: normal
>>>
>>> Hi all,
>>>
>>> dur to unicode change, 2 nodejs packages require an update:
>>>  - node-regenerate-unicode-properties
>>>  - node-regexpu-core
>>>
>>> These 2 packages have been updated in unstable so can no more be updated
>>> using the normal way. The proposed changes are very few (patch to update
>>> package.json + debian/control dependency update from node-unicode-11.0.0
>>> to node-unicode-12.0.0).
>>>
>>> These updates are required since they are both dependencies of node-rollup
>>> which is a build dependency of ~20 packages.
>>>
>>> Do you authorize me to upload these 2 +deb10u1 packages in
>>> testing-proposed-updates? Packages are tested locally, build +
>>> autopkgtest OK.
>>>
>>> Sorry for the inconvenience.
>>>
>>> Cheers,
>>> Xavier
>>>
>>> [...]
>>>
>>
>> The versions in unstable: Could they be used / unblocked as-is (without
>> needing anything else)?
> 
> Hi,
> 
> yes they can be unblocked. Changes are not so big and all tests pass
> (even if upstream tests are not enabled here).
> 

Ok, then we might as well take the sid version given the tpu version
would not be considerably less risky compared to the effort.

Unblocked, thanks.
~Niels

--- End Message ---

Reply to: