Summary: | repoman: linechecks: Mishandling of % characters from ebuilds | ||
---|---|---|---|
Product: | Portage Development | Reporter: | Arfrever Frehtes Taifersar Arahesis <arfrever.fta> |
Component: | Repoman | Assignee: | Portage team <dev-portage> |
Status: | RESOLVED FIXED | ||
Severity: | normal | Keywords: | InVCS |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | Patch |
Description
Arfrever Frehtes Taifersar Arahesis
2019-11-07 05:35:41 UTC
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/proj/portage.git/commit/?id=d156784fd7557e8f5ce8a6de5cd3a6a38a78e436 commit d156784fd7557e8f5ce8a6de5cd3a6a38a78e436 Author: Zac Medico <zmedico@gentoo.org> AuthorDate: 2019-11-08 04:52:42 +0000 Commit: Zac Medico <zmedico@gentoo.org> CommitDate: 2019-11-08 04:54:37 +0000 repoman: fix unsafe string interpolation (bug 699508) Reported-by: Arfrever Frehtes Taifersar Arahesis <Arfrever@Apache.Org> Bug: https://bugs.gentoo.org/699508 Signed-off-by: Zac Medico <zmedico@gentoo.org> repoman/lib/repoman/modules/linechecks/controller.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/proj/portage.git/commit/?id=d8879e74458935919a4303f4fd1d2d24eb92e094 commit d8879e74458935919a4303f4fd1d2d24eb92e094 Author: Zac Medico <zmedico@gentoo.org> AuthorDate: 2019-11-08 05:42:16 +0000 Commit: Zac Medico <zmedico@gentoo.org> CommitDate: 2019-11-08 05:43:26 +0000 Revert "repoman: fix unsafe string interpolation (bug 699508)" This reverts commit d156784fd7557e8f5ce8a6de5cd3a6a38a78e436. The 'on line: %d' string fails to match some messages. Reported-by: Arfrever Frehtes Taifersar Arahesis <Arfrever@Apache.Org> Bug: https://bugs.gentoo.org/699508 Signed-off-by: Zac Medico <zmedico@gentoo.org> repoman/lib/repoman/modules/linechecks/controller.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) Created attachment 595514 [details, diff]
Patch
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/proj/portage.git/commit/?id=bb6a2ca9ccc8d3d6f7feb7c9ac6b6c2b40ce9d28 commit bb6a2ca9ccc8d3d6f7feb7c9ac6b6c2b40ce9d28 Author: Arfrever Frehtes Taifersar Arahesis <Arfrever@Apache.Org> AuthorDate: 2019-11-09 03:39:53 +0000 Commit: Zac Medico <zmedico@gentoo.org> CommitDate: 2019-11-09 04:06:05 +0000 repoman: Fix unsafe string interpolation. "on line: %d" etc. is no longer included in messages returned from individual checks. "line %d: " is now consistently added by controller directly after ebuild path and precedes specific message. Example: app-misc/test/test-0.ebuild: line 5: please migrate from 'games' (no replacement) app-misc/test/test-0.ebuild: line 5: please migrate from 'versionator' to 'eapi7-ver (built-in since EAPI 7)' app-misc/test/test-0.ebuild: line 10: Useless blank line app-misc/test/test-0.ebuild: line 20: 'econf' call should be moved to src_configure Bug: https://bugs.gentoo.org/699508 Signed-off-by: Arfrever Frehtes Taifersar Arahesis <Arfrever@Apache.Org> Signed-off-by: Zac Medico <zmedico@gentoo.org> repoman/cnf/linechecks/linechecks.yaml | 46 +++++++++++----------- repoman/lib/repoman/modules/linechecks/base.py | 5 +-- .../lib/repoman/modules/linechecks/controller.py | 12 ++++-- .../modules/linechecks/deprecated/inherit.py | 18 +++------ repoman/lib/repoman/modules/linechecks/do/dosym.py | 2 +- .../lib/repoman/modules/linechecks/eapi/checks.py | 10 ++--- .../lib/repoman/modules/linechecks/emake/emake.py | 2 +- .../lib/repoman/modules/linechecks/phases/phase.py | 4 +- .../repoman/modules/linechecks/portage/internal.py | 7 +--- .../repoman/modules/linechecks/quotes/quoteda.py | 2 +- .../repoman/modules/linechecks/useless/dodoc.py | 2 +- .../repoman/modules/linechecks/whitespace/blank.py | 2 +- 12 files changed, 53 insertions(+), 59 deletions(-) The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2b90833cb908ee9a64c7293feadb16294ee5adff commit 2b90833cb908ee9a64c7293feadb16294ee5adff Author: Zac Medico <zmedico@gentoo.org> AuthorDate: 2019-11-08 21:58:22 -0800 Commit: Zac Medico <zmedico@gentoo.org> CommitDate: 2019-11-08 21:59:23 -0800 app-portage/repoman: Bump to version 2.3.18 #690786 Support metadata/layout.conf restrict-allowed #699514 Detect dosym absolute paths starting with ${D}, ${ED} etc. #699508 Fix unsafe string interpolation. Package-Manager: Portage-2.3.79, Repoman-2.3.18 Signed-off-by: Zac Medico <zmedico@gentoo.org> app-portage/repoman/Manifest | 1 + app-portage/repoman/repoman-2.3.18.ebuild | 63 +++++++++++++++++++++++++++++++ 2 files changed, 64 insertions(+) |