Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 7 Dec 2013 11:05:45 -0800
From:      Greg Lewis <glewis@eyesbeyond.com>
To:        Albert Shih <Albert.Shih@obspm.fr>
Cc:        freebsd-java@freebsd.org
Subject:   Re: 7u45 update - amd64 testers
Message-ID:  <20131207190545.GA87007@misty.eyesbeyond.com>
In-Reply-To: <20131207092231.GA51378@pcjas.obspm.fr>
References:  <20131207064044.GA79133@misty.eyesbeyond.com> <20131207092231.GA51378@pcjas.obspm.fr>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Dec 07, 2013 at 10:22:31AM +0100, Albert Shih wrote:
>  Le 06/12/2013 ? 22:40:44-0800, Greg Lewis a ?crit
> > Can someone please download
> > 
> > http://people.freebsd.org/~glewis/openjdk7-7u45.diff
> > 
> > and see if it compiles on amd64?  If it does I'll go ahead and commit it.
> > Already tested on i386 locally.
> 
> No that's not compiles.
> 
> On :
> 
>     9.2-STABLE FreeBSD 9.2-STABLE #11 r258722: Thu Nov 28 23:37:54 CET 2013 
> 
>     kernel  /usr/obj/usr/src/sys/GENERIC  amd64
> 
> all ports is up2date.
> 
> Here the logs.
> 
> Thanks for your work.

Thanks Albert.  That actually looks like a 8.x vs. 9.x problem.  It needs
to be fixed regardless.

On 8.x:

typedef struct sem *    sem_t;

On 10.x (and presumably 9.x as well based on the error)

typedef struct _sem     sem_t;

I think the solution is to vary the os_semaphore_t definition in os_bsd.cpp
based on the FreeBSD version.  Let me test compiling that on 8.x and I'll
send it out.

-- 
Greg Lewis                          Email   : glewis@eyesbeyond.com
Eyes Beyond                         Web     : http://www.eyesbeyond.com
Information Technology              FreeBSD : glewis@FreeBSD.org



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20131207190545.GA87007>