Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 625908 - app-doc/kicad-doc: invalid dep on app-text/texlive
Summary: app-doc/kicad-doc: invalid dep on app-text/texlive
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Zoltan Puskas
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-07-21 21:24 UTC by Michał Górny
Modified: 2018-02-22 00:04 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2017-07-21 21:24:11 UTC
app-text/texlive:=[l10n_en?,l10n_fr?,l10n_it?,l10n_ja?,l10n_nl?,l10n_pl?]

This dep is wrong. app-text/texlive is just a metapackage for user's convenience. You need to depend on specific texlive packages that your ebuild actually needs.
Comment 1 Zoltan Puskas 2017-10-07 04:44:08 UTC
Addressed for kicad-doc-4.0.6 in PR 4991.
Comment 2 Zoltan Puskas 2017-10-07 05:07:29 UTC
Except that makes the ebuild fail on a bunch of languages. I need texlive to be installed with the language flag enabled that is also used for this ebuild.

Examining the texlive ebuild I can see that the language packs are dependant on the texlive version and there is no other texlive package I can pull it that has language flags. This will have to stay.

Unless someone has a suggestion how to solve it, I will mark this bug as a WONTFIX and there is no other solution to it.
Comment 3 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2017-10-07 10:54:00 UTC
You have to depend directly on the exact packages your package needs.
Comment 4 Zoltan Puskas 2017-10-07 19:02:30 UTC
Okay, if that's the policy. I have update the PR with exact dependencies.
Comment 5 Alexis Ballier gentoo-dev 2017-10-08 10:15:52 UTC
(In reply to Michał Górny from comment #0)
> app-text/texlive:=[l10n_en?,l10n_fr?,l10n_it?,l10n_ja?,l10n_nl?,l10n_pl?]
> 
> This dep is wrong. app-text/texlive is just a metapackage for user's
> convenience. You need to depend on specific texlive packages that your
> ebuild actually needs.

Note that it is sometimes ok and expected for ebuilds to depend on the metapackage: Think about a latex GUI with a 'compile' button. Compilation will fail if user builds a document using packages that are not installed. In that case, we expect ebuilds to depend on app-text/texlive to specify "anything tex-related"  and users to set proper useflags on app-text/texlive for their intended use. If your package needs specific latex packages that are known in advance then you should depend on them directly in order to avoid users having to fine tune package.use for app-text/texlive when a simpler direct dep in the ebuild would have had the same effect.

A good rule of thumb could be that 'app-text/texlive' deps are ok but usedeps on it are forbidden.