Summary: | media-libs/freetype does unseemly things with autotools-utils.eclass (was for a while: move remove_libtool_files from autotools-utils.eclass into eutils.eclass) | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Michał Górny <mgorny> |
Component: | Current packages | Assignee: | Gentoo's Team for Core System packages <base-system> |
Status: | RESOLVED FIXED | ||
Severity: | major | CC: | fonts, realnc, zima |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 392063 |
Description
Michał Górny
2011-11-27 10:15:11 UTC
I have an install failure due to this issue. The eqawarn function (called from remove_libtool_files) returns nonzero status (since I do not use QA logs), causing the whole src_install to fail. Maybe it is specific to paludis. Do you need more info? The only thing we use is remove_libtool_files(). I really don't feel like using autotools-utils here, so it'd be nice if it were split out into eutils or something so we don't have to start duplicating code. @base-system: Can we move remove_libtool_files from autotools-utils.eclass into eutils.eclass? i'm not entirely convinced of its useful/correct-ness. i don't think there should be a "remove all" option, and generally i think package maintainers should be understanding the libraries they install and what is appropriate for removal. although you could say that even if people understand what's going on, they'd still want to use the func out of laziness ... (In reply to comment #4) > i'm not entirely convinced of its useful/correct-ness. i don't think there > should be a "remove all" option, and generally i think package maintainers > should be understanding the libraries they install and what is appropriate for > removal. I agree; if we move it, there should be only one mode of operation. > although you could say that even if people understand what's going on, they'd > still want to use the func out of laziness ... It's still better than people do now. Because now, people usually copy random pieces of code like 'use static-libs ||' and randomly install useless .la files anyway. (In reply to comment #4) > although you could say that even if people understand what's going on, they'd > still want to use the func out of laziness ... Or to avoid code duplication and not have to update every one of their ebuilds when policies change. Well, I need to bump freetype and don't feel like inheriting an eclass that I have no control over that changes every week. So I'm just ripping any static lib logic out of the ebuild. Patches welcome. Fixed in 2.4.9. This version will be stable in a few days, and I'll drop the offending versions. The change gives rise to bug 407663. How do we resolve the blocker? |