Both media-gfx/argyllcms-1.4.0 and media-gfx/dispcalgui-0.9.9.1 install /lib/udev/rules.d/55-Argyll.rules. This results in a merge conflict on Gentoo stable.
Created attachment 328056 [details] media-gfx/argyllcms-1.4.0 failure log
Ugh that is ugly Probably need to remove the file from dispcalgui in src_prepare or whatever. Let argyllcms install the udev rules instead. Marios what do you think?
I am on it. Since argyllcms is a dependency of dispcalgui and a diff between the 2 rules files doesn't show any differences, I think we can safely remove it from the dispcalgui ebuild. I will attach fixed ebuilds for both stable and testing arches in a bit. What is funny, is that on my computer, argyllcms installs its rules in /usr/lib/udev/rules.d while dispcalgui installs its rules in /lib/udev/rules.d.
Maybe this is due to the recent udev.eclass changes?
> Maybe this is due to the recent udev.eclass changes? I don't know but since we won't be installing any udev rules anymore, it doesn't matter I think.
(In reply to comment #5) > > Maybe this is due to the recent udev.eclass changes? > I don't know but since we won't be installing any udev rules anymore, it > doesn't matter I think. Yes I think it does not matter either.
Created attachment 328090 [details] dispcalGUI 0.9.9.1 fixed ebuild This fix can be considered a hack. I use bash (sed) to enable a configuration option that could be passed to `setup.py install`. Unfortunately my knowledge on this is limited and after a hour of searching, I gave up and turned to good old bash. If anyone is interested in fixing it, the `setup.py install` command should be run with `--skip-instrument-configuration-files`. Then my sed directive should be removed from the ebuild.
Created attachment 328092 [details] dispcalGUI 1.1.0.0 fixed ebuild The same "hack logic" as in the previous comment applies. Please remove media-gfx/dispcalgui-1.0.7.7 from portage.
Isn't this a bit dirty? Wouldn't it be better to do something like this src_install() { distutils_src_install # remove udev files rm ${D}/blabla } I don't mind either way just saying.
Created attachment 328094 [details] dispcalGUI 1.1.2.1 ebuild It is a chance to update dispcalgui to the latest version too. It is the same ebuild as dispcalgui-1.1.0.0-r1 renamed. As usual, wget http://dispcalgui.hoech.net/download/dispcalGUI.tar.gz mv dispcalGUI.tar.gz dispcalGUI-1.1.2.1.tar.gz Thanks Richard for reporting and Markos for taking the time to submit and manage the ebuilds and distfiles in portage. :)
Let me check and I will post updated ebuilds if it works. ;) (In reply to comment #9) > Isn't this a bit dirty? > > Wouldn't it be better to do something like this > > src_install() { > distutils_src_install > # remove udev files > rm ${D}/blabla > } > > I don't mind either way just saying.
Created attachment 328098 [details] dispcalGUI 0.9.9.1 updated ebuild This version removes udev files as you suggested instead of sed'ing setup.py.
Created attachment 328100 [details] dispcalGUI 1.1.0.0 updated ebuild
Created attachment 328102 [details] dispcalGUI 1.1.2.1 updated ebuild Hope everything is alright now. Thanks!
Fixed the following problems as well ebuild.minorsyn 9 media-gfx/dispcalgui/dispcalgui-0.9.9.1.ebuild: Ebuild contains leading spaces on line: 52 media-gfx/dispcalgui/dispcalgui-0.9.9.1.ebuild: Ebuild contains leading spaces on line: 54 media-gfx/dispcalgui/dispcalgui-0.9.9.1.ebuild: Unquoted Variable on line: 54 media-gfx/dispcalgui/dispcalgui-1.1.0.0.ebuild: Ebuild contains leading spaces on line: 59 media-gfx/dispcalgui/dispcalgui-1.1.0.0.ebuild: Ebuild contains leading spaces on line: 61 media-gfx/dispcalgui/dispcalgui-1.1.0.0.ebuild: Unquoted Variable on line: 61 media-gfx/dispcalgui/dispcalgui-1.1.2.1.ebuild: Ebuild contains leading spaces on line: 59 media-gfx/dispcalgui/dispcalgui-1.1.2.1.ebuild: Ebuild contains leading spaces on line: 61 media-gfx/dispcalgui/dispcalgui-1.1.2.1.ebuild: Unquoted Variable on line: 6 (you should have seen them if you used "repoman full") ;-) Also added || die to the 'rm' command in src_install +*dispcalgui-1.1.2.1 (02 Nov 2012) + + 02 Nov 2012; Markos Chandras <hwoarang@gentoo.org> +dispcalgui-1.1.2.1.ebuild, + -dispcalgui-1.0.7.7.ebuild, dispcalgui-0.9.9.1.ebuild, + dispcalgui-1.1.0.0.ebuild: + Version bump. Fix old ebuilds to not install udev rules in order to avoid + collisions with media-gfx/argyllcms. Thanks to Marios Andreopoulos + <opensource@andmarios.com>. Bug #440932. Remove old +
Thanks! I will use repoman from now on. (In reply to comment #15) > Fixed the following problems as well > ebuild.minorsyn 9 >... > (you should have seen them if you used "repoman full") ;-) > > Also added || die to the 'rm' command in src_install >... >
*** Bug 445024 has been marked as a duplicate of this bug. ***
in dispcalgui-1.1.2.9 the rule is called 45-Argyll.rules not 55-Argyll.rules and the build now fail
(In reply to comment #18) > in dispcalgui-1.1.2.9 the rule is called 45-Argyll.rules not 55-Argyll.rules > and the build now fail Please do NOT reply to closed bugs. Search bugzilla for open bugs (such as 449302) or open a new one
