Bug 312869 – dev-libs/Ice-3.4.0 new release ebuild+patch file

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Philip Kovacs 2010-04-02 19:22:02 UTC]

Attached is the patch to Ice-3.3.1.ebuild for new release 3.4.0, as well as a new patch file for the Gentoo standard locations.  The ebuild is essentially the same with some dependency bumps.  Tested here on ~x86.

Reproducible: Always

[Comment 1 Philip Kovacs 2010-04-02 19:23:17 UTC]

Created attachment 226311 [details, diff]
dev-cpp/Ice-3.4.0.ebuild.diff

diff patch from Ice-3.3.1.ebuild

[Comment 2 Philip Kovacs 2010-04-02 19:25:29 UTC]

Created attachment 226313 [details, diff]
files/Ice-3.4.0-Makefile.patch

new patch file for the sources

[Comment 3 Lars Wendler (Polynomial-C) (RETIRED) 2010-04-03 14:07:00 UTC]

I just want to note that this version is also affected by bug #312023

Philip thanks for the diffs, especially the Makefile patch which I failed to create myself :)

[Comment 4 Philip Kovacs 2010-04-03 20:53:59 UTC]

Created attachment 226467 [details, diff]
files/Ice-3.4.0-openssl.patch

I posted a revised openssl patch upstream at zeroc.com as well, as mine is better. :p

[Comment 5 Philip Kovacs 2010-04-03 20:55:28 UTC]

Created attachment 226469 [details, diff]
dev-cpp/Ice-3.4.0.ebuild.diff

Adds the aforementioned openssl patch.

[Comment 6 Philip Kovacs 2010-04-04 00:25:30 UTC]

Created attachment 226493 [details, diff]
files/Ice-3.4.0-stream1.patch

For the sake of completeness, I am adding the stream-related patches posted by the Ice devs for 3.4.0.

[Comment 7 Philip Kovacs 2010-04-04 00:26:27 UTC]

Created attachment 226495 [details, diff]
files/Ice-3.4.0-stream2.patch

[Comment 8 Philip Kovacs 2010-04-04 00:27:24 UTC]

Created attachment 226499 [details, diff]
dev-cpp/Ice-3.4.0.ebuild.diff

and finally a revised ebuild to apply all four patches.

[Comment 9 Tiziano Müller (RETIRED) 2010-06-09 20:24:49 UTC]

Thanks a lot for the patches. Done.

[Comment 10 Philip Kovacs 2010-06-10 01:52:30 UTC]

There is an error in the Ice-3.4.0.ebuild that was posted to portage.  That ebuild manually sets DB_FLAGS, but it indicated db 4.6 instead of 4.8.

[Comment 11 Philip Kovacs 2010-06-10 01:53:29 UTC]

Created attachment 234799 [details, diff]
dev-libs/Ice/Ice-3.4.0.ebuild.diff

this attachment should fix it.

[Comment 12 Philip Kovacs 2010-06-10 02:01:09 UTC]

As a follow-up I should mention that the ebuild uses this RDEPEND:

>=sys-libs/db-4.8.30:4.8[-nocxx]

while hard-coding DB_FLAGS to version 4.8.

This is problematic of course as it leaves a latent bug:
if the version db should go above 4.8, the ebuild will break again.

Just a heads-up.

[Comment 13 Philip Kovacs 2010-06-10 02:07:01 UTC]

Created attachment 234803 [details, diff]
dev-libs/Ice/Ice-3.4.0.ebuild.diff

DB_LIBS also needs to be bumped to 4.8.

[Comment 14 Lars Wendler (Polynomial-C) (RETIRED) 2010-06-10 19:07:38 UTC]

(In reply to comment #12)
> As a follow-up I should mention that the ebuild uses this RDEPEND:
> 
> >=sys-libs/db-4.8.30:4.8[-nocxx]
> 
> while hard-coding DB_FLAGS to version 4.8.
> 
> This is problematic of course as it leaves a latent bug:
> if the version db should go above 4.8, the ebuild will break again.
> 
> Just a heads-up.
> 

The dependency requires any db version from slot 4.8 which will never have any other version than 4.8.x

(In reply to comment #13)
> Created an attachment (id=234803) [details]
> dev-libs/Ice/Ice-3.4.0.ebuild.diff
> 
> DB_LIBS also needs to be bumped to 4.8.
> 

+  10 Jun 2010; Lars Wendler <[e-mail]> Ice-3.4.0.ebuild:
+  non-maintainer commit: Fixing compilation against sys-libs/db:4.8 (see bug
+  #312869)
+