Re: unison: warn about OCaml version incompatibilities

2014-10-16 Thread Giovanni Bechis
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 10/16/14 13:38, Christopher Zimmermann wrote:
 Hi,
 
 I keep being asked why unison fails to work since the upgrade to OCaml
 4.02. Therefore I'd like to add this piece of information to the 
 install message.
what about putting the MESSAGE in a README file ?

 The patch is just for my convenience when debugging. Is it ok to commit 
 this patch, too?
ok giovanni@

 Unison comes in stable and snapshot versions. The snapshot version is 
 outdated and broken. Can it be removed? Should it be updated?
 
I am not using unison,-snapshot, it is unhooked since a lot, I think it can be 
removed.
 Cheers
  Giovanni
-BEGIN PGP SIGNATURE-
Version: GnuPG v1

iQEcBAEBAgAGBQJUP7pqAAoJEI7Sc79XDlKKWs0H/ijO67vLmT/ScguY92EHqWB3
Qi0386frakxCylEBhN7hJJflYOQTYNlWVxa98tfUyFQZJrtguzxrG6NK36KYNQKS
5Y+93tzS9a+wSG9mwhgVNoKrYeHJNQniji7H84WpBMwdquvBQrHNQrShPP6Dwm7y
tt9ZnAjCZJxB6SlJI/9Xk2AwRbtsOn01ElC04xHDeOIW+18bOLIajkSN+dHfwvAm
mMRbUCXkn/CuzqFlsz0+PkJZ1WtCdwhgcLzky8OSYuf+0VNjDJUngoh7BBm8Yjad
MPctiUqYOpaHt95aFEWO7s2mkwk/T9bVD/UxtRRryD07PnNrfOscL7/TBFM4cJ8=
=cD0V
-END PGP SIGNATURE-



Re: unison: warn about OCaml version incompatibilities

2014-10-16 Thread David Coppa
On Thu, Oct 16, 2014 at 2:30 PM, Giovanni Bechis giova...@paclan.it wrote:

 I am not using unison,-snapshot, it is unhooked since a lot, I think it can 
 be removed.

+1

Ciao!
David
-- 
If you try a few times and give up, you'll never get there. But if
you keep at it... There's a lot of problems in the world which can
really be solved by applying two or three times the persistence that
other people will.
-- Stewart Nelson



Re: unison: warn about OCaml version incompatibilities

2014-10-16 Thread Stuart Henderson
On 2014/10/16 14:32, David Coppa wrote:
 On Thu, Oct 16, 2014 at 2:30 PM, Giovanni Bechis giova...@paclan.it wrote:
 
  I am not using unison,-snapshot, it is unhooked since a lot, I think it can 
  be removed.
 
 +1

Probably makes sense to move net/unison/stable/* to net/unison/* as well
(this will need @pkgpath net/unison/stable adding to plist).



Re: unison: warn about OCaml version incompatibilities

2014-10-16 Thread Stuart Henderson
On 2014/10/16 13:38, Christopher Zimmermann wrote:
 Hi,
 
 I keep being asked why unison fails to work since the upgrade to OCaml
 4.02. Therefore I'd like to add this piece of information to the 
 install message.

Ugh!

In a previous version, they had this ...

Update to 2.40.102
fixes an incompatibility between a server compiled with ocaml3 and a client
compiled with ocaml4

Could something like this be done instead?



Re: unison: warn about OCaml version incompatibilities

2014-10-16 Thread David Coppa
On Thu, Oct 16, 2014 at 2:40 PM, Stuart Henderson st...@openbsd.org wrote:
 On 2014/10/16 14:32, David Coppa wrote:
 On Thu, Oct 16, 2014 at 2:30 PM, Giovanni Bechis giova...@paclan.it wrote:

  I am not using unison,-snapshot, it is unhooked since a lot, I think it 
  can be removed.

 +1

 Probably makes sense to move net/unison/stable/* to net/unison/* as well
 (this will need @pkgpath net/unison/stable adding to plist).

Yes, like you did with mutt.



Re: unison: warn about OCaml version incompatibilities

2014-10-16 Thread Christopher Zimmermann
On Thu, 16 Oct 2014 14:41:37 +0200 David Coppa dco...@gmail.com wrote:

 On Thu, Oct 16, 2014 at 2:40 PM, Stuart Henderson st...@openbsd.org
 wrote:
  On 2014/10/16 14:32, David Coppa wrote:
  On Thu, Oct 16, 2014 at 2:30 PM, Giovanni Bechis
  giova...@paclan.it wrote:
 
   I am not using unison,-snapshot, it is unhooked since a lot, I
   think it can be removed.
 
  +1
 
  Probably makes sense to move net/unison/stable/* to net/unison/* as
  well (this will need @pkgpath net/unison/stable adding to plist).
 
 Yes, like you did with mutt.

Thanks for the hints! So is the following diff OK? Someone told me CVS
will clean up the empty directories on its own, doesn't it?

Christopher


Index: Makefile
===
RCS file: /cvs/ports/net/Makefile,v
retrieving revision 1.865
diff -u -p -r1.865 Makefile
--- Makefile4 Oct 2014 18:50:55 -   1.865
+++ Makefile16 Oct 2014 13:06:27 -
@@ -609,6 +609,7 @@
  SUBDIR += uhttpmock
  SUBDIR += unifi
  SUBDIR += unison
+ SUBDIR += unison,no_x11
  SUBDIR += unworkable
  SUBDIR += ushare
  SUBDIR += uucp
Index: unison/Makefile
===
RCS file: /cvs/ports/net/unison/Makefile,v
retrieving revision 1.7
diff -u -p -r1.7 Makefile
--- unison/Makefile 1 Feb 2010 17:00:14 -   1.7
+++ unison/Makefile 16 Oct 2014 13:06:27 -
@@ -1,9 +1,65 @@
-# $OpenBSD: Makefile,v 1.7 2010/02/01 17:00:14 espie Exp $
+# $OpenBSD: Makefile.inc,v 1.24 2014/06/28 21:08:09 kili Exp $
 
-SUBDIR =
-# SUBDIR +=snapshot
-# SUBDIR +=snapshot,no_x11
-SUBDIR +=  stable
-SUBDIR +=  stable,no_x11
+COMMENT=   multi-platform file synchronization tool
+CATEGORIES=net
 
-.include bsd.port.subdir.mk
+V= 2.40.102
+REVISION = 5
+DISTNAME=  unison-${V}
+MASTER_SITES=  ${HOMEPAGE}download/releases/stable/
+
+HOMEPAGE=  http://www.cis.upenn.edu/~bcpierce/unison/
+MAINTAINER=Giovanni Bechis giova...@openbsd.org
+
+# GPLv3
+PERMIT_PACKAGE_CDROM=  Yes
+
+WANTLIB=   c m util
+
+MODULES=   lang/ocaml
+
+USE_GMAKE= Yes
+
+FLAVORS=   no_x11
+FLAVOR?=
+
+FLAVOR_COMMA=   ${FLAVOR_EXT:S/-/,/g}
+SUBST_VARS= FLAVOR_COMMA
+
+.if ${FLAVOR:Mno_x11}
+MAKE_FLAGS=UISTYLE=text
+.else
+MODULES += devel/gettext
+BUILD_DEPENDS+=x11/lablgtk2
+LIB_DEPENDS=   x11/gtk+2
+MAKE_FLAGS=UISTYLE=gtk2
+WANTLIB += X11 Xcomposite Xcursor Xdamage Xext Xfixes Xi Xinerama
+WANTLIB += Xrandr Xrender fontconfig freetype
+WANTLIB += gio-2.0 glib-2.0 gobject-2.0 pango-1.0
+WANTLIB += pangocairo-1.0 pangoft2-1.0 gdk_pixbuf-2.0
+WANTLIB += gdk-x11-2.0 gtk-x11-2.0 atk-1.0 cairo z
+WANTLIB += pthread
+.endif
+
+# CFLAGS _must_ be empty.  This is an OCaml compiler.
+MAKE_FLAGS+=   NATIVE=${MODOCAML_NATIVE:S/Yes/true/:S/No/false/} \
+   OCAMLOPT=ocamlopt.opt \
+   CFLAGS=
+
+TEST_TARGET=   selftest
+PORTHOME=  ${WRKDIR}
+
+# Avoid multiple all:: entries and use the real target entry.
+ALL_TARGET=unison
+
+DOCS=  BUGS.txt NEWS README
+
+# Avoid the nightmare of their Makefile install target.
+# Do not use INSTALL_PROGRAM, as the bytecode version must not be stripped!
+do-install:
+   ${INSTALL_SCRIPT} ${WRKSRC}/unison ${PREFIX}/bin
+   ${INSTALL_DATA_DIR} ${PREFIX}/share/doc/unison
+   @cd ${WRKSRC}  ${INSTALL_DATA} ${DOCS} ${PREFIX}/share/doc/unison
+
+
+.include bsd.port.mk
Index: unison/Makefile.inc
===
RCS file: unison/Makefile.inc
diff -N unison/Makefile.inc
--- unison/Makefile.inc 28 Jun 2014 21:08:09 -  1.24
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,54 +0,0 @@
-# $OpenBSD: Makefile.inc,v 1.24 2014/06/28 21:08:09 kili Exp $
-
-DISTNAME=  unison-$V
-CATEGORIES=net
-
-HOMEPAGE=  http://www.cis.upenn.edu/~bcpierce/unison/
-MAINTAINER=Giovanni Bechis giova...@openbsd.org
-
-# GPLv3
-PERMIT_PACKAGE_CDROM=  Yes
-
-WANTLIB=   c m util
-
-MODULES=   lang/ocaml
-
-USE_GMAKE= Yes
-
-FLAVORS=   no_x11
-FLAVOR?=
-
-.if ${FLAVOR:Mno_x11}
-MAKE_FLAGS=UISTYLE=text
-.else
-MODULES += devel/gettext
-BUILD_DEPENDS+=x11/lablgtk2
-LIB_DEPENDS=   x11/gtk+2
-MAKE_FLAGS=UISTYLE=gtk2
-WANTLIB += X11 Xau Xcomposite Xcursor Xdamage Xdmcp Xext Xfixes Xi Xinerama
-WANTLIB += Xrandr Xrender expat fontconfig freetype pixman-1 pthread-stubs
-WANTLIB += gio-2.0 glib-2.0 gmodule-2.0 gobject-2.0 pango-1.0
-WANTLIB += pangocairo-1.0 pangoft2-1.0 gdk_pixbuf-2.0
-WANTLIB += gdk-x11-2.0 gtk-x11-2.0 atk-1.0 cairo png xcb z
-WANTLIB += GL Xxf86vm drm gthread-2.0 pthread xcb-render xcb-shm
-.endif
-
-# CFLAGS _must_ be empty.  This is an OCaml compiler.
-MAKE_FLAGS+=   NATIVE=${MODOCAML_NATIVE:S/Yes/true/:S/No/false/} \
-   OCAMLOPT=ocamlopt.opt \
-   CFLAGS=
-
-TEST_TARGET=   selftest
-PORTHOME=  ${WRKDIR}
-
-# Avoid multiple 

Re: unison: warn about OCaml version incompatibilities

2014-10-16 Thread Christopher Zimmermann
On Thu, 16 Oct 2014 13:41:43 +0100 Stuart Henderson st...@openbsd.org
wrote:

 On 2014/10/16 13:38, Christopher Zimmermann wrote:
  Hi,
  
  I keep being asked why unison fails to work since the upgrade to
  OCaml 4.02. Therefore I'd like to add this piece of information to
  the install message.
 
 Ugh!
 
 In a previous version, they had this ...
 
 Update to 2.40.102
 fixes an incompatibility between a server compiled with ocaml3 and a
 client compiled with ocaml4
 
 Could something like this be done instead?


I guess this was about this svn commit:

 * Use hash function from OCaml 3.x for comparing archives, even when
   compiled with OCaml 4.x


 git-svn-id: https://webdav.seas.upenn.edu/svn/unison/trunk@514 
4bce34ff-96ee-0310-b826-95abcfff6e26


This does not seem to be about marshalling, but is also a case of relying on
OCaml stdlib functionality that is not meant to be included in portable
protocols.
Sorry to say, but I'm not motivated to fix this design bug of unison.
(would take me about a days work).


Christopher

-- 
http://gmerlin.de
OpenPGP: http://gmerlin.de/christopher.pub
F190 D013 8F01 AA53 E080  3F3C F17F B0A1 D44E 4FEE

signature.asc
Description: PGP signature


Re: unison: warn about OCaml version incompatibilities

2014-10-16 Thread Stuart Henderson
On 2014/10/16 15:15, Christopher Zimmermann wrote:
 On Thu, 16 Oct 2014 13:41:43 +0100 Stuart Henderson st...@openbsd.org
 wrote:
 
  On 2014/10/16 13:38, Christopher Zimmermann wrote:
   Hi,
   
   I keep being asked why unison fails to work since the upgrade to
   OCaml 4.02. Therefore I'd like to add this piece of information to
   the install message.
  
  Ugh!
  
  In a previous version, they had this ...
  
  Update to 2.40.102
  fixes an incompatibility between a server compiled with ocaml3 and a
  client compiled with ocaml4
  
  Could something like this be done instead?
 
 
 I guess this was about this svn commit:
 
  * Use hash function from OCaml 3.x for comparing archives, even when
compiled with OCaml 4.x
 
 
  git-svn-id: https://webdav.seas.upenn.edu/svn/unison/trunk@514 
 4bce34ff-96ee-0310-b826-95abcfff6e26
 
 
 This does not seem to be about marshalling, but is also a case of relying on
 OCaml stdlib functionality that is not meant to be included in portable
 protocols.
 Sorry to say, but I'm not motivated to fix this design bug of unison.
 (would take me about a days work).
 
 
 Christopher
 
 -- 
 http://gmerlin.de
 OpenPGP: http://gmerlin.de/christopher.pub
 F190 D013 8F01 AA53 E080  3F3C F17F B0A1 D44E 4FEE



Ah, OK.



Re: unison: warn about OCaml version incompatibilities

2014-10-16 Thread Stuart Henderson
On 2014/10/16 15:08, Christopher Zimmermann wrote:
 Thanks for the hints! So is the following diff OK?

yep.

 Someone told me CVS will clean up the empty directories on its own, doesn't 
 it?

It cleans them at checkout/update time as long as you use -p,
there's no real way (short of messing with the repository) to
remove them.



Re: unison: warn about OCaml version incompatibilities

2014-10-16 Thread Giovanni Bechis
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 10/16/14 15:08, Christopher Zimmermann wrote:

 Thanks for the hints! So is the following diff OK?
ok giovanni@

 Someone told me CVS
 will clean up the empty directories on its own, doesn't it?
 
sthen@ has been exhaustive.
 Cheers
  Giovanni
-BEGIN PGP SIGNATURE-
Version: GnuPG v1

iQEcBAEBAgAGBQJUP+Y3AAoJEI7Sc79XDlKKBwkH/0cvnzbOuiVBWaFhYEG/8pQ+
UdOwBR/ssU8x8TVbTTY4wBrfLXhQeLKKwd3+kzgjs0aVrTJ/87z4TK+6f7tzUl/C
DX/OcWZMAjFGFTIBxr3Yt8ENGYEJaqdp6rGvVYvCC5QupllEtluWye8UGOk6XP0g
+zcxGQfMhTAaPSuPfv83O+pNDDelX/jeLWKcHDlo0tzTc4ozyUahMRGX/dGhPgqQ
EXjD7c4GLpXucxkJJoFHHZmBdRjQmMSpmj5z7N/a4EfzljruoInSkbshzzwFr/7Z
oX6rJoLe0sdWBwIWvsGyuFIrunZDkLNpEltV8/EEgzW4QWDdXsvTkEJQk4SaxWg=
=mjP3
-END PGP SIGNATURE-