[Unison-hackers] patches for 2.10.2 and 2.12.0

[Jerome Vouillon]

On Mon, May 23, 2005 at 01:59:19PM -0400, Zvezdan Petkovic wrote:
> Can I safely assume that there's no harm even if I include those patches
> on a Unix platform like OpenBSD?

All patches can be included on all platforms.

> Namely, they are both in the beta 2.12.0 source.
> I think there should be no harm if stable 2.10.2 is patched the same.
> 
> Regarding the new patch for read-only problem above, the first change:
> 
>           `DIRECTORY ->
> -           Unix.chmod absolutePath 0o700;
> +           begin try
> +             Unix.chmod absolutePath 0o700
> +           with Unix.Unix_error _ -> () end;
> 
> has been applied to the beta 2.12.0, but the second change
> 
>         | `FILE ->
> -           if Util.osType <> `Unix then
> -             Unix.chmod absolutePath 0o600;
> +           if Util.osType <> `Unix then begin
> +             try
> +               Unix.chmod absolutePath 0o600;
> +             with Unix.Unix_error _ -> ()
> +           end;
> 
> has _not_ been applied to the beta 2.12.0 version.
> 
> Shouldn't 2.12.0 be also patched with this second set of lines only
> (see the attached patch).

Yes, it should also be patched.

-- Jerome