[15090] in athena10
Bug#873966: config-package-dev: transform operation error
daemon@ATHENA.MIT.EDU (Bruno Maitre)
Fri Sep 1 13:27:18 2017
Reply-To: Bruno Maitre <bruno.maitre@gmail.com>, 873966@bugs.debian.org
Resent-From: Bruno Maitre <bruno.maitre@gmail.com>
Resent-To: debian-bugs-dist@lists.debian.org
MIME-Version: 1.0
From: Bruno Maitre <bruno.maitre@gmail.com>
Date: Fri, 1 Sep 2017 19:22:26 +0200
Message-ID: <CAAjYQmMptfowp0q_KQoGkZkLeKpHdEud18Ne2tQqe7T4EWXP3Q@mail.gmail.com>
To: submit@bugs.debian.org
Content-Type: multipart/mixed; boundary="f403045fb750a7d9c9055824032b"
--f403045fb750a7d9c9055824032b
Content-Type: multipart/alternative; boundary="f403045fb750a7d9c50558240329"
--f403045fb750a7d9c50558240329
Content-Type: text/plain; charset="UTF-8"
Package: config-package-dev
Version: 5.2
Severity: normal
Tags: patch
Dear Maintainer,
When using the transform operation dh_configpackage will output that kind
of error:
Can't use string ("/ARRAY(0x55f37e2c3080)") as an ARRAY ref while
"strict refs" in use at /usr/bin/dh_configpackage line 394.
This error has been introduced by the correction of:
#803962 : config-package-dev: Requires leading slashes un debian/*.displace
For this correction we iterrate through the differents operation arrays to
add a leading slash if needed.
The problem is that @transformfiles is an array of array since it's created
with filedoublearray function and not the filearray function like the others
operation arrays (which are simples arrays).
This lead to an add of a slash in front of the reference array ARRAY(0xXXX...)
as can be seen in the error message.
Subsequent processing of transformfiles results in an error.
I've attached a patch with a way to fix this issue by removing transformfiles
from the leading slashes verification loop and by checking leading slashes in
the treatment of transformfiles itself as it was done before.
Note that I've triggered the error in a Debian testing environment and
the stable version is not affected.
Best regards,
Bruno
-- System Information:
Debian Release: 9.1
APT prefers stable
APT policy: (500, 'stable')
Architecture: amd64 (x86_64)
Kernel: Linux 4.9.0-3-amd64 (SMP w/2 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8),
LANGUAGE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
--f403045fb750a7d9c50558240329
Content-Type: text/html; charset="UTF-8"
<div dir="ltr"><pre class="gmail-aLF-aPX-K0-aPE">Package: config-package-dev
Version: 5.2
Severity: normal
Tags: patch
Dear Maintainer,
When using the transform operation dh_configpackage will output that kind
of error:
Can't use string ("/ARRAY(0x55f37e2c3080)") as an ARRAY ref while "strict refs" in use at /usr/bin/dh_configpackage line 394.
This error has been introduced by the correction of:
#803962 : config-package-dev: Requires leading slashes un debian/*.displace
For this correction we iterrate through the differents operation arrays to
add a leading slash if needed.
The problem is that @transformfiles is an array of array since it's created
with filedoublearray function and not the filearray function like the others
operation arrays (which are simples arrays).
This lead to an add of a slash in front of the reference array ARRAY(0xXXX...)
as can be seen in the error message.
Subsequent processing of transformfiles results in an error.
I've attached a patch with a way to fix this issue by removing transformfiles
from the leading slashes verification loop and by checking leading slashes in
the treatment of transformfiles itself as it was done before.
Note that I've triggered the error in a Debian testing environment and
the stable version is not affected.
Best regards,<br>Bruno<br>
-- System Information:
Debian Release: 9.1
APT prefers stable
APT policy: (500, 'stable')
Architecture: amd64 (x86_64)
Kernel: Linux 4.9.0-3-amd64 (SMP w/2 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8), LANGUAGE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)</pre></div>
--f403045fb750a7d9c50558240329--
--f403045fb750a7d9c9055824032b
Content-Type: text/x-diff; charset="US-ASCII";
name="0001-dh_configpackage-Fix-leading-slashes-check-for-the-t.patch"
Content-Disposition: attachment;
filename="0001-dh_configpackage-Fix-leading-slashes-check-for-the-t.patch"
Content-Transfer-Encoding: base64
X-Attachment-Id: f_j7257e1b0
RnJvbSA0MWJkNDExYjQxMDQ5ODJmMDVkNjNiN2UxNzM4NTkxYmEzNTdhYjdhIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBCcnVubyBNYWl0cmUgPGJydW5vLm1haXRyZUBnbWFpbC5jb20+
CkRhdGU6IEZyaSwgMSBTZXAgMjAxNyAxNzoyNTowOSArMDIwMApTdWJqZWN0OiBbUEFUQ0hdIGRo
X2NvbmZpZ3BhY2thZ2U6IEZpeCBsZWFkaW5nIHNsYXNoZXMgY2hlY2sgZm9yIHRoZSB0cmFuc2Zv
cm0KIG9wZXJhdGlvbgoKLS0tCiBkaF9jb25maWdwYWNrYWdlIHwgMyArKy0KIDEgZmlsZSBjaGFu
Z2VkLCAyIGluc2VydGlvbnMoKyksIDEgZGVsZXRpb24oLSkKCmRpZmYgLS1naXQgYS9kaF9jb25m
aWdwYWNrYWdlIGIvZGhfY29uZmlncGFja2FnZQppbmRleCBiYTFiMDYwLi5hYjZmMDY3IDEwMDc1
NQotLS0gYS9kaF9jb25maWdwYWNrYWdlCisrKyBiL2RoX2NvbmZpZ3BhY2thZ2UKQEAgLTM3Niw3
ICszNzYsNyBAQCBmb3JlYWNoIG15ICRwYWNrYWdlIChAeyRkaHtET1BBQ0tBR0VTfX0pIHsKICAg
ICAgICAgQHRyYW5zZm9ybWZpbGVzID0gZmlsZWRvdWJsZWFycmF5KCR0cmFuc2Zvcm1maWxlKSBp
ZiAkdHJhbnNmb3JtZmlsZTsKICAgICB9CiAKLSAgICBmb3JlYWNoIG15ICRsaXN0cmVmIChcQGRp
c3BsYWNlZmlsZXMsIFxAaGlkZWZpbGVzLCBcQHVuZGlzcGxhY2VmaWxlcywgXEB1bmhpZGVmaWxl
cywgXEB0cmFuc2Zvcm1maWxlcykgeworICAgIGZvcmVhY2ggbXkgJGxpc3RyZWYgKFxAZGlzcGxh
Y2VmaWxlcywgXEBoaWRlZmlsZXMsIFxAdW5kaXNwbGFjZWZpbGVzLCBcQHVuaGlkZWZpbGVzKSB7
CiAgICAgICAgIGZvcmVhY2ggbXkgJGZpbGUgKEAkbGlzdHJlZikgewogICAgICAgICAgICAgJGZp
bGUgPX4gc3xeLz98L3w7CiAgICAgICAgIH0KQEAgLTM5Miw2ICszOTIsNyBAQCBmb3JlYWNoIG15
ICRwYWNrYWdlIChAeyRkaHtET1BBQ0tBR0VTfX0pIHsKIAogICAgIGZvcmVhY2ggbXkgJGxpbmUg
KEB0cmFuc2Zvcm1maWxlcykgewogICAgICAgICBteSAkZmlsZSA9IHNoaWZ0IEAkbGluZTsKKyAg
ICAgICAgJGZpbGUgPX4gc3xeLz98L3w7CiAgICAgICAgIG15ICRzb3VyY2U7CiAgICAgICAgIG15
ICRzb3VyY2VfaXNfbG9jYWwgPSAwOwogICAgICAgICBpZiAoQCRsaW5lWzBdID1+IC9ePC8pIHsK
LS0gCjIuMTEuMAoK
--f403045fb750a7d9c9055824032b--
