Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@harmony.apache.org
Received-SPF: neutral (nike.apache.org: 195.212.17.164 is neither permitted
 nor denied by domain of mark.hindess@googlemail.com)
Message-Id: <201004301134.o3UBYx38002804@d12av03.megacenter.de.ibm.com>
In-reply-to: <20100321161815.772A8816018@nike.apache.org>
References: <3b3f27c61003210858qd0f0fd4lbc8065b0c23c8362@mail.gmail.com>
 <20100321161815.772A8816018@nike.apache.org>
Comments: In-reply-to Mark Hindess <mark.hindess@googlemail.com>
   message dated "Sun, 21 Mar 2010 16:17:23 +0000."
From: Mark Hindess <mark.hindess@googlemail.com>
To: dev@harmony.apache.org
Subject: Re: zlib in drlvm
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Date: Fri, 30 Apr 2010 12:34:59 +0100


In message <20100321161815.772A8816018@nike.apache.org>, Mark Hindess writes:
>
> In message <3b3f27c61003210858qd0f0fd4lbc8065b0c23c8362@mail.gmail.com>,
> Nathan Beyer writes:
> >
> > While trying to perform a fresh build, zlib 1.2.3 wasn't downloading
> > and it looks like zlib.net has released 1.2.4. While trying to update
> > zlib, i noticed a few things:
> > * The pre-built binary is only used for Windows - i assume we're using
> > a shared zlib on other OSes, but do we keep the version controlled?
> 
> I think the zlib api is pretty stable.  So I'm not overly concerned
> about version control.  However, I suspect it is moot since hyzlib
> is loaded first and almost certainly takes priority over the system
> version.  (Unless you set -Dhy.local.zlib=true to pick up the system
> version instead of hyzlib [unix only].)
> 
> > * The pre-built binary is only used for x86 Windows - we're
> > downloading an 'x86_64' version from a JIRA attachment [1]. Shouldn't
> > we put this into SVN, like the other pre-built dependencies?
> 
> We should find a way to remove it and use only hyzlib instead.
> 
> > * Anyone know how the x86_64 binary was built?
> 
> No.  I don't know enough about windows dll loading but perhaps it is
> overriden by hyzlib anyway?
> 
> > -Nathan
> > 
> > [1] https://issues.apache.org/jira/browse/HARMONY-3196

In commit r939640, I fixed drlvm to use hyzlib on windows as the
download was broken again and there didn't seem to be a new equivalent.
Still need to:

  1) Remove depends.properties entries.
  2) Use hyzlib during link on unix (when hy.local.zlib=true is not set).
  3) Upgrade hyzlib in classlib to use zlib 1.2.5

Regards,
 Mark.