Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 469414 - vim-spell.eclass should not DEPEND on app-editors/{,g}vim
Summary: vim-spell.eclass should not DEPEND on app-editors/{,g}vim
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Vim Maintainers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-05-11 11:28 UTC by Roman Žilka
Modified: 2017-09-12 20:51 UTC (History)
0 users

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 Roman Žilka 2013-05-11 11:28:18 UTC
vim-spell.eclass includes (g)vim in DEPEND, but the eclass seems to only RDEPEND on vim. By removing vim from DEPEND, all the vim-spell-* pkgs would not be needlessly remerged with --rebuild-if-* when vim upgrades.

No ebuild that inherits vim-spell relies on vim for building either, and vim-spell.eclass doesn't inherit vim-doc.eclass (which needs vim for build time).

By the way, vim-doc.eclass calls vim during build time, but relies on another eclass / the ebuild to include vim in DEPEND. Structurally that's unclean, with a tiny probability of future issues. Just FYI.

Reproducible: Always
Comment 1 Patrice Clement gentoo-dev 2017-09-12 20:51:28 UTC
commit 907c3adc19d8e5010dcb2e010858907e6d390027 (HEAD -> master, origin/master, origin/HEAD)
Author:     Patrice Clement <monsieurp@gentoo.org>
AuthorDate: Tue Sep 12 22:40:44 2017 +0200
Commit:     Patrice Clement <monsieurp@gentoo.org>
CommitDate: Tue Sep 12 22:50:31 2017 +0200

vim-spell.eclass: do not DEPEND on app-editors/{,g}vim.

Gentoo-Bug: https://bugs.gentoo.org/469414

eclass/vim-spell.eclass | 4 ----
1 file changed, 4 deletions(-)

Thank you for holding out for so long. We've updated the eclass accordingly.