Summary: | >=dev-libs/gobject-introspection-1.62.0 lost USE=test dependency on dev-python/markdown | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Paolo Pedroni <paolo.pedroni> |
Component: | Current packages | Assignee: | Gentoo Linux Gnome Desktop Team <gnome> |
Status: | RESOLVED OBSOLETE | ||
Severity: | normal | CC: | sam |
Priority: | Normal | Keywords: | PullRequest |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: |
https://github.com/gentoo/gentoo/pull/15501 https://bugs.gentoo.org/show_bug.cgi?id=733668 |
||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Paolo Pedroni
2020-04-24 14:52:12 UTC
The dep is already there for USE=doctool (rdepend). If there's a problem, then it's about doctool deps being needed on tests even with -Ddoctool=false being passed to meson configuration The dep on markdown was added in version 1.60.2-r1 in bug #693408, and it fixed tests with the use case USE="-doctool test". Is it really a problem to add it back to 1.62.0 and 1.64.0? This is still an issue in 1.72.0. If you have tests enabled, test phase will fail because markdown can't be found. Can the ebuild at least be updated so that if testing is enabled the right dependency gets pulled in? Or perhaps, skip that test when USE="-doctools" please? (In reply to unhappy-ending from comment #3) > This is still an issue in 1.72.0. If you have tests enabled, test phase will > fail because markdown can't be found. Can the ebuild at least be updated so > that if testing is enabled the right dependency gets pulled in? Or perhaps, > skip that test when USE="-doctools" please? That doesn't make sense to me. So: option('doctool', type: 'feature', value : 'auto', description: 'Install g-ir-doc-tool and run related tests' ) ... and https://gitlab.gnome.org/GNOME/gobject-introspection/-/commit/4e1ada2070a2626c82bd510f5165e5557c7d0608 is curious. It implies it's supposed to do the right thing already. Furthermore, we actually have the markdown dep unconditionally for tests right now anyway? > $(python_gen_cond_dep ' > dev-python/mako[${PYTHON_USEDEP}] > dev-python/markdown[${PYTHON_USEDEP}] > ') (In reply to unhappy-ending from comment #3) > This is still an issue in 1.72.0. If you have tests enabled, test phase will > fail because markdown can't be found. Can the ebuild at least be updated so > that if testing is enabled the right dependency gets pulled in? Or perhaps, > skip that test when USE="-doctools" please? If you're having an issue, please file a new bug with the full log and emerge --info. It does not appear to be the same issue as this one, as explained above. It's possible it's something like the doctools stuff doesn't get built but one of the tests still expects it, but I don't see how right now. Weird, I've had gobject-introspection on my system and markdown wasn't pulled in when I went to rebuild it, I had to manually do it. I just forcefully (emerge -C) both markdown and gobject-introspection and did a re-emerge and it's pulling in correctly now. Maybe I missed something along the way, likely user error. Sorry :) |