Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 834447 - app-editors/vim[minimal] File collision when installing app-editors/vim-core[minimal]
Summary: app-editors/vim[minimal] File collision when installing app-editors/vim-core[...
Status: RESOLVED NEEDINFO
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: Normal normal (vote)
Assignee: Vim Maintainers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-03-01 18:29 UTC by enlightened
Modified: 2022-03-08 15:55 UTC (History)
3 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 enlightened 2022-03-01 18:29:03 UTC
Repro steps:
Install app-editors/vim-8.2.4328 with "minimal" flag
Install app-editors/vim-core-8.2.4328 with "minimal" flag

Error:
 * Detected file collision(s):
 * 
 *      /build/eve/usr/share/vim/vim82/defaults.vim
 * 
 * Searching all installed packages for file collisions...
 * 
 * Press Ctrl-C to Stop
 * 
 * app-editors/vim-8.2.4328:0::portage-stable
 *      /build/eve/usr/share/vim/vim82/defaults.vim
Comment 1 Mike Gilbert gentoo-dev 2022-03-01 18:42:25 UTC
Please include the full package category/name in the summary when creating bug reports.
Comment 2 Mike Gilbert gentoo-dev 2022-03-01 18:44:05 UTC
I assume you are using some forked version of the Gentoo repository. Please ensure you are using the latest ebuild from Gentoo.

Also, please provide emerge --info and attach a build log.
Comment 3 Brian Norris 2022-03-07 23:43:39 UTC
(Yes, this report is coming from Chromium OS.)

But the problem comes from the fact that the [[ -f ${vimfiles}/... ]] logic was not respecting SYSROOTs, nor was it handling the file-move logic that portage can normally do post-install, if you have your blockers right [1].

But it looks like you've completely given up on USE=minimal here?

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0816769373e7ee62685158b65f7cfde287e5d936

I guess that solves the problem technically, but we actually do use USE=minimal, so it's probably be nice to reintroduce it. I think it would just need a blocker, like:

## vim.ebuild
RDEPEND="minimal? ( !app-editors/vim-core )"

## vim-core.ebuild
RDEPEND="!app-editors/vim-core[minimal]"

or similar.

[1] https://devmanual.gentoo.org/general-concepts/dependencies/#blockers
Comment 4 Mike Gilbert gentoo-dev 2022-03-08 00:01:31 UTC
I don't really see the point/benefit.
Comment 5 Brian Norris 2022-03-08 00:16:21 UTC
(In reply to Mike Gilbert from comment #4)
> I don't really see the point/benefit.

Is vim-core[minimal] equivalently "small" compared to vim[minimal] (modulo perhaps some docs, etc., which can be stripped via other means)? If so, then indeed there's not much point.
Comment 6 Mike Gilbert gentoo-dev 2022-03-08 15:55:36 UTC
It looks like USE=minimal cuts down the installed size of app-editors/vim-core from 36.1 MiB to 15.5 MiB. Perhaps someone could look into reducing that further.

On app-editors/vim, USE=minimal reduces the size from 3.6 MiB to 1.5 MiB.