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

Bug#989522: unblock: trscripts/1.18+nmu2 xfonts-bolkhov/1.1.20001007-8.2



Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock

Please unblock package trscripts and xfonts-bolkhov

[ Reason ]
The awk script generated by trscripts used a non deterministic for-in
loop resulting in the russian letter 'у' displayed as latin u with the
xfonts-bolkhov-misc font. This was reported as #979599 and #979710.

[ Impact ]
Font rendering would be wrong without the patch.

[ Tests ]
run: xfontsel -sampleUCS у -pattern "-rfx-*" and look at the displayed
symbol.

[ Risks ]
The change in trscripts is minimal, just using a different for loop
style. xfonts-bolkhov is a no change rebuild, just bumping the
dependency on trscripts.

[ Checklist ]
  [X] all changes are documented in the d/changelog
  [X] I reviewed all changes and I approve them
  [X] attach debdiff against the package in testing

unblock trscripts/1.18+nmu2
unblock xfonts-bolkhov/1.1.20001007-8.2
diff -Nru trscripts-1.18+nmu1/debian/changelog trscripts-1.18+nmu2/debian/changelog
--- trscripts-1.18+nmu1/debian/changelog	2021-01-07 15:01:30.000000000 +0100
+++ trscripts-1.18+nmu2/debian/changelog	2021-06-05 20:08:15.000000000 +0200
@@ -1,3 +1,12 @@
+trscripts (1.18+nmu2) unstable; urgency=medium
+
+  * Non-maintainer upload.
+  * Make trbdf awk script portable (Closes: #979599).
+    POSIX awk does not specify the order in a for(i in array) loop, so
+    switching to a for loop with an increment.
+
+ -- Jochen Sprickerhof <jspricke@debian.org>  Sat, 05 Jun 2021 20:08:15 +0200
+
 trscripts (1.18+nmu1) unstable; urgency=medium
 
   * Non maintainer upload by the Reproducible Builds team.
diff -Nru trscripts-1.18+nmu1/gen_trbdf trscripts-1.18+nmu2/gen_trbdf
--- trscripts-1.18+nmu1/gen_trbdf	2009-05-02 12:43:11.000000000 +0200
+++ trscripts-1.18+nmu2/gen_trbdf	2021-06-05 20:08:15.000000000 +0200
@@ -312,15 +312,15 @@
 EOF
 
 if [ "$usefb" = yes ]; then
-    printf "	split(tu[i] \" \" alt1[tu[i]] \" \" alt2[tu[i]], a);\n"
+    printf "	an = split(tu[i] \" \" alt1[tu[i]] \" \" alt2[tu[i]], a);\n"
     printf "	split(0 \" \" weight1[tu[i]] \" \" weight2[tu[i]], w);\n"
 else
-    printf "	split(tu[i] \" \" alt1[tu[i]], a);\n"
+    printf "	an = split(tu[i] \" \" alt1[tu[i]], a);\n"
     printf "	split(0 \" \" weight1[tu[i]], w);\n"
 fi
 
 cat <<"EOF"
-	for(j in a)
+	for(j=1; j <= an; ++j)
 	  {
 	    if(ut[a[j]]!="")
 	      {
@@ -339,7 +339,7 @@
 	      }
 	  }
 	k=0;
-	for(j in a)
+	for(j=1; j <= an; ++j)
 	  {
 	    if(ut[a[j]]!="")
 	      {
@@ -356,7 +356,7 @@
 	    printf "\";\n";
 	  }
 	k=0;
-	for(j in a)
+	for(j=1; j <= an; ++j)
 	  {
 	    if(ut[a[j]]!="")
 	      {
diff -u xfonts-bolkhov-1.1.20001007/debian/changelog xfonts-bolkhov-1.1.20001007/debian/changelog
--- xfonts-bolkhov-1.1.20001007/debian/changelog
+++ xfonts-bolkhov-1.1.20001007/debian/changelog
@@ -1,3 +1,11 @@
+xfonts-bolkhov (1.1.20001007-8.2) unstable; urgency=medium
+
+  * Non-maintainer upload.
+  * Bump dependency on trscripts to fix generated fonts when using mawk, cf.
+    #979599.
+
+ -- Jochen Sprickerhof <jspricke@debian.org>  Sat, 05 Jun 2021 23:47:31 +0200
+
 xfonts-bolkhov (1.1.20001007-8.1) unstable; urgency=medium
 
   * Non maintainer upload by the Reproducible Builds team.
diff -u xfonts-bolkhov-1.1.20001007/debian/control xfonts-bolkhov-1.1.20001007/debian/control
--- xfonts-bolkhov-1.1.20001007/debian/control
+++ xfonts-bolkhov-1.1.20001007/debian/control
@@ -3,7 +3,7 @@
 Section: fonts
 Priority: optional
 Standards-Version: 3.6.2
-Build-Depends: debhelper (>=9~), trscripts (>= 1.13), xfonts-utils
+Build-Depends: debhelper (>=9~), trscripts (>= 1.18+nmu2), xfonts-utils
 
 Package: xfonts-bolkhov-75dpi
 Architecture: all

Reply to: