Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 612702 - sci-libs/vtk-7 bundled six conflicts with dev-python/six
Summary: sci-libs/vtk-7 bundled six conflicts with dev-python/six
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gentoo Science Related Packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 637646
  Show dependency tree
 
Reported: 2017-03-15 11:16 UTC by Gert Wollny
Modified: 2017-11-17 18:46 UTC (History)
2 users (show)

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


Attachments
patch against ebuild to force use of dev-pythion/six (vtk-7.1.0-systemsix.patch,663 bytes, patch)
2017-03-15 11:18 UTC, Gert Wollny
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gert Wollny 2017-03-15 11:16:51 UTC
VTK bundles six and the ebuild doesn't enable the system version. Hence with an installed dev-python/six a file collision occurs. 

Reproducible: Always

Steps to Reproduce:
1. install dev-python/six
2. install sci-libs/vtk-7.1.0  

Actual Results:  
Detected file collision(s):
 * 
 * 	/usr/lib64/python2.7/site-packages/six.py
 * 
 * Searching all installed packages for file collisions...
 * 
 * Press Ctrl-C to Stop
 * 
 * dev-python/six-1.10.0:0::gentoo


Expected Results:  
clean installation
Comment 1 Gert Wollny 2017-03-15 11:18:01 UTC
Created attachment 467118 [details, diff]
patch against ebuild to force use of dev-pythion/six
Comment 2 Gert Wollny 2017-03-15 11:20:39 UTC
See also: 

https://github.com/gerddie/gentoo-imaging/issues/7
Comment 3 Tamas Jantvik 2017-07-06 08:22:16 UTC
The patch seems to be malformed. But this works:

( cd "/usr/portage/sci-libs/vtk" && [[ -f "vtk-7.1.0.ebuild" ]] && wget -qO- 'https://612702.bugs.gentoo.org/attachment.cgi?id=467118' | sed '3,6d' | patch -s && repoman manifest )

Which replaces the ebuild. Could be run in postsync.
Comment 4 Alexander 2017-09-03 09:35:01 UTC
I test it this different settings of `python` and `web` USE-flags, and pull commit to GH https://github.com/gentoo/gentoo/pull/5606
Comment 5 Guilherme Amadio gentoo-dev 2017-09-04 09:22:29 UTC
The (In reply to Alexander from comment #4)
> I test it this different settings of `python` and `web` USE-flags, and pull
> commit to GH https://github.com/gentoo/gentoo/pull/5606

The patch by Gert Wollny is better in the sense that it does not move the dependencies from web to the python USE flag. I think that we need a combination of the two things, in which the six dependency is added to the right place (need to test if USE=python requrires six or only web requires it), and keep the other deps intact, so PR 5606 above cannot be merged as is.
Comment 6 Alexander 2017-09-06 14:57:05 UTC
(In reply to Guilherme Amadio from comment #5)
> The (In reply to Alexander from comment #4)
> > I test it this different settings of `python` and `web` USE-flags, and pull
> > commit to GH https://github.com/gentoo/gentoo/pull/5606
> 
> The patch by Gert Wollny is better in the sense that it does not move the
> dependencies from web to the python USE flag. I think that we need a
> combination of the two things, in which the six dependency is added to the
> right place (need to test if USE=python requrires six or only web requires
> it), and keep the other deps intact, so PR 5606 above cannot be merged as is.

Hi.
The only thing, than I was guided when move dependences and makeopts, is that the `USE=web` depends on the `USE=python`:
```shell
REQUIRED_USE="
	all-modules? ( python xdmf2 boost )
	java? ( qt5 )
	python? ( ${PYTHON_REQUIRED_USE} )
	tcl? ( rendering )
	test? ( python )
	tk? ( tcl )
	web? ( python )
	^^ ( X aqua offscreen )
	"
```
Best regards, alex.
Comment 7 Larry the Git Cow gentoo-dev 2017-11-17 18:46:53 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2c9f39647f523d4757bd0d7d8cf9d2abcf5648f1

commit 2c9f39647f523d4757bd0d7d8cf9d2abcf5648f1
Author:     Bernd Waibel <waebbl@gmail.com>
AuthorDate: 2017-11-16 07:04:07 +0000
Commit:     David Seifert <soap@gentoo.org>
CommitDate: 2017-11-17 18:39:23 +0000

    sci-libs/vtk: patch to use dev-python/twisted
    
    Depending on dev-python/twisted-core and dev-python/autobahn produces
    a block, because autobahn depends on dev-python/twisted.
    
    Closes: https://bugs.gentoo.org/637646
    Closes: https://bugs.gentoo.org/612702
    Package-Manager: Portage-2.3.14, Repoman-2.3.5
    Closes: https://github.com/gentoo/gentoo/pull/6200

 sci-libs/vtk/vtk-7.1.0.ebuild | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)