Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 59762

Summary: net-misc/unison-2.9.1: Fixes for compilation with ocaml-3.08
Product: Gentoo Linux Reporter: Bardur Arantsson <bugs-gentoo.org>
Component: New packagesAssignee: Gentoo Team for the ML programming language family <ml>
Status: RESOLVED FIXED    
Severity: normal    
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: unison-2.9.1-r1.ebuild
ocaml-3.08-extnames.patch

Description Bardur Arantsson 2004-08-08 05:26:46 UTC
The current stable version of unison (2.9.1) doesn't compile with ocaml 3.08 (which was just added to portage) because some exported symbols have been renamed.

I've attached a small patch to unison which fixes the compilation problems and an updated ebuild which automatically patches the source if ocaml-3.08 (or later) is detected.
Comment 1 Bardur Arantsson 2004-08-08 05:28:03 UTC
Created attachment 37017 [details]
unison-2.9.1-r1.ebuild
Comment 2 Bardur Arantsson 2004-08-08 05:28:47 UTC
Created attachment 37018 [details, diff]
ocaml-3.08-extnames.patch
Comment 3 Matthieu Sozeau (RETIRED) gentoo-dev 2004-08-09 03:41:43 UTC
Fixed in CVS, thanks for your help.
Comment 4 Florian Pressler 2004-08-14 01:57:57 UTC
Hello!

Could it be that the patch is the reason for the trouble I have compiling the new ebuild (unison-2.9.1-r1)? After having a working unison v2.9.1, the compilation of  the patched ebuild fails with

ocamlopt: uigtk.ml ---> uigtk.cmx
ocamlopt -I lwt -I ubase -I /usr/lib/ocaml/lablgtk -c /var/tmp/portage/unison-2.9.1-r1/work/unison-2.9.1/uigtk.ml
The files /usr/lib/ocaml/pervasives.cmi and /usr/lib/ocaml/lablgtk/gdk.cmi
make inconsistent assumptions over interface Pervasives
make: *** [uigtk.cmx] Error 2

!!! ERROR: net-misc/unison-2.9.1-r1 failed.
!!! Function src_compile, Line 44, Exitcode 2
!!! (no error message)

regards,
Florian
Comment 5 Florian Pressler 2004-08-14 02:00:54 UTC
PS: Just saw that there is already Bugzilla Bug 59957 regarding this issue... ignore my previous comment...