As gentoo is now doing ssl by defaut, we should be able to specify https://www.gentoo.org/dtd/metadata.dtd instead of http://www.gentoo.org/dtd/metadata.dtd in metadata.xml and thus repoman should at least except both if not correct to https silently.
Could we please get this fixed?
Pyhton 2.7: repoman full --xmlparse -d RepoMan scours the neighborhood... >>> Creating Manifest for /local/home/justin/tree/media-libs/simage metadata.bad [fatal] 1 media-libs/simage/metadata.xml: DOCTYPE: SYSTEM should refer to 'http://www.gentoo.org/dtd/metadata.dtd', not 'https://www.gentoo.org/dtd/metadata.dtd' repo.eapi.deprecated 1 media-libs/simage/simage-1.7.0.ebuild: 2 Please fix these important QA issues first. RepoMan sez: "Make your QA payment on time and you'll never see the likes of me." Python 3.4: justin@paul ~/tree/media-libs/simage $ (1) repoman full --xmlparse -d RepoMan scours the neighborhood... >>> Creating Manifest for /local/home/justin/tree/media-libs/simage repo.eapi.deprecated 1 media-libs/simage/simage-1.7.0.ebuild: 2 RepoMan sez: "You're only giving me a partial QA payment? I'll take it this time, but I'm not happy."
(In reply to Justin Lecher from comment #2) It seems you found a bug in the XML parsing code under python3. I would suggest reverting your mass-update until repoman has actually been updated to accommodate it.
With QA's blessing, I reverted the http -> https conversion in metadata.xml for now.
repoman patch sent for review.
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/proj/portage.git/commit/?id=3950d76df63cf0f3f6502014dd61b8aa9c260e36 commit 3950d76df63cf0f3f6502014dd61b8aa9c260e36 Author: Michał Górny <mgorny@gentoo.org> AuthorDate: 2018-02-04 13:34:33 +0000 Commit: Michał Górny <mgorny@gentoo.org> CommitDate: 2018-02-05 18:45:38 +0000 repoman: Update metadata.dtd URI to allow https:// Update the default metadata.dtd URI used in repoman and Portage tests to use https://. However, allow also http:// form for the migration period. The http:// compat can be removed once the Gentoo repository is updated to use https:// everywhere. Bug: https://bugs.gentoo.org/552720 Closes: https://github.com/gentoo/portage/pull/255 Reviewed-by: Zac Medico <zmedico@gentoo.org> pym/portage/tests/resolver/ResolverPlayground.py | 2 +- repoman/pym/repoman/metadata.py | 2 +- repoman/pym/repoman/modules/scan/metadata/pkgmetadata.py | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-)}
Can this bug be closed? AFAICS, the fix was released with repoman-2.3.7 (and 2.3.9 went stable in June 2018).
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/proj/gentoo-syntax.git/commit/?id=50344cc0ce9c3f430de883b3d8d3966933d249d0 commit 50344cc0ce9c3f430de883b3d8d3966933d249d0 Author: Mike Gilbert <floppym@gentoo.org> AuthorDate: 2021-09-11 13:54:37 +0000 Commit: Mike Gilbert <floppym@gentoo.org> CommitDate: 2021-09-11 13:55:17 +0000 newmetadata: use https in DOCTYPE URI Bug: https://bugs.gentoo.org/552720 Signed-off-by: Mike Gilbert <floppym@gentoo.org> plugin/newmetadata.vim | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/proj/devmanual.git/commit/?id=ee6c5a78e4eb2a2051f0ce6ef9856dd10c3f4b14 commit ee6c5a78e4eb2a2051f0ce6ef9856dd10c3f4b14 Author: Ulrich Müller <ulm@gentoo.org> AuthorDate: 2021-09-11 10:44:55 +0000 Commit: Ulrich Müller <ulm@gentoo.org> CommitDate: 2021-09-11 14:09:23 +0000 ebuild-writing/misc-files/metadata: Replace http by https Bug: https://bugs.gentoo.org/552720 Signed-off-by: Ulrich Müller <ulm@gentoo.org> ebuild-writing/misc-files/metadata/text.xml | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/proj/portage.git/commit/?id=0ec131cc84893f429da2c4486545a6c93d3447c0 commit 0ec131cc84893f429da2c4486545a6c93d3447c0 Author: Ulrich Müller <ulm@gentoo.org> AuthorDate: 2022-01-25 16:41:18 +0000 Commit: Ulrich Müller <ulm@gentoo.org> CommitDate: 2022-01-25 16:41:18 +0000 repoman: Remove http compatibility code for metadata DTD Commit 3950d76df says: "The http:// compat can be removed once the Gentoo repository is updated to use https:// everywhere." Bug: https://bugs.gentoo.org/552720 Signed-off-by: Ulrich Müller <ulm@gentoo.org> repoman/lib/repoman/modules/scan/metadata/pkgmetadata.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
