Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@harmony.apache.org
Received-SPF: pass (athena.apache.org: domain of vladimir.k.beliaev@gmail.com
 designates 64.233.166.177 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=beta;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references;
        b=BgX6A6vUYDffOdQ9LzIoZT2gzkTYjdQquLNxqQ52G1ypRoUeOO/xgKvNQC/DiyoShH9shBKee081vfilz3KjZHIpnbqG7n8jST0aOa19BdIHGvy258dXLjGI/P7aEaRGJqUrABdieqJOBoq9c+DrDhtoVK3waRK6VrmFwKCz1D8=
Message-ID: <587698b80710190958t4069b132vaeac28a53473e353@mail.gmail.com>
Date: Fri, 19 Oct 2007 20:58:43 +0400
From: "Vladimir Beliaev" <vladimir.k.beliaev@gmail.com>
To: dev@harmony.apache.org
Subject: Re: [build] jdktools skips a fetch-depends for federated build
In-Reply-To: <200710191421.l9JELTTn011609@d06av04.portsmouth.uk.ibm.com>
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----=_Part_9987_29488115.1192813123913"
References: <587698b80710190106t23b40b17mc016668fac133c74@mail.gmail.com>
	 <20071019083648.5FAA2724960@athena.apache.org>
	 <587698b80710190422h147d77c2t8c801d793822c3c1@mail.gmail.com>
	 <200710191421.l9JELTTn011609@d06av04.portsmouth.uk.ibm.com>

------=_Part_9987_29488115.1192813123913
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

2007/10/19, Mark Hindess <mark.hindess@googlemail.com>:
>
>
> On 19 October 2007 at 15:22, "Vladimir Beliaev" <
> vladimir.k.beliaev@gmail.com>
> wrote:
> >
> > Hello, Mark,
> >
> > > ant -Dauto.fetch=true
> > >
> > > It should work if you do this?
> >
> > I've missed this. Thanks a lot for the note.
> >
> > > seem overly complicated and probably can be replaced by a single
> > > target:
> >
> > Yes, until this "inheritall="false"" is inserted not just for fun...
>
> Sorry, I don't understand.  I suggested replacing two:
>
> <ant inheritall="false" .../>
>
> calls with a single:
>
> <ant inheritall="false" .../>
>
> call so I don't see the problem.  Can you explain what you think I've
> missed?


I do not see the problem too. I've missed inheritall="false" in first tag.

Looking at the similar calls for classlib_fetch_depends I see that
> what is actually wrong is that the "fetch_jdktools_libs" target is not
> supposed to have the if="auto.fetch" attribute.  (It is supposed to be
> unconditional so a user can invoke it.)  This is an excellent example of
> why it is helpful to see which tasks are supposed to be called by a user
> and which are not.
>
> > > For example adopting the convention of beginning non-api/-user
> > > targets with hyphen so it is easier to read
> >
> > Yes, code clean up work is really fun especially for one who is able
> > to commit it immediately...
>
> I'll reply to this in another note but, assuming I understand you
> correctly, the short version is "I strongly disagree".


I believe your assumption is not correct. Looks like my phrase is confusing.
Sorry.

Thanks
Vladimir

Regards,
Mark.

> 2007/10/19, Mark Hindess <mark.hindess@googlemail.com>:
> >
> >
> > For licensing reasons, fetch depends is only executed if you run:
> >
> > ant -Dauto.fetch=true
> >
> > It should work if you do this?
> >
> > Having said that I think there is room for improvement in the build.xml.
> > For example, these targets:
> >
> >    <target name="auto_fetch_jdktools_libs" if="auto.fetch">
> >        <ant target="fetch_jdktools_libs" inheritall="false" >
> >            <property name="hy.cfg" value="${hy.cfg}"/>
> >        </ant>
> >    </target>
> >
> >    <target name="fetch_jdktools_libs" if="auto.fetch">
> >        <ant antfile="working_jdktools/build.xml" target="fetch-depends"
> > inheritall="false" >
> >            <property name="hy.cfg" value="${hy.cfg}"/>
> >        </ant>
> >    </target>
> >
> > seem overly complicated and probably can be replaced by a single target:
> >
> >    <target name="auto_fetch_jdktools_libs" if="auto.fetch">
> >        <ant antfile="working_jdktools/build.xml"
> >             target="fetch-depends" inheritall="false" >
> >            <property name="hy.cfg" value="${hy.cfg}"/>
> >        </ant>
> >    </target>
> >
> > Unless I'm missing something?  This build.xml really needs some cleanup.
> > (For example adopting the convention of beginning non-api/-user targets
> > with hyphen so it is easier to read.)  Maybe I'll take a look later.
> >
> > Regards,
> > Mark.
> >
> > On 19 October 2007 at 12:06, "Vladimir Beliaev" <
> > vladimir.k.beliaev@gmail.com> wrote:
> > >
> > > Hello,
> > >
> > > I come across the same issue twice - the federated build skips
fetching
> > > dependencies for jdktools.
> > >
> > > I mean I did a build just by simple 'ant' command from the root
> > directory of
> > > federated workspace:
> > >
> > > /tmp/trunk $ pwd
> > >     /tmp/trunk
> > >  /tmp/trunk $ ls -1
> > >     working_classlib/
> > >     working_jdktools/
> > >     working_vm/
> > >     <skipped>
> > >  /tmp/trunk $ ant
> > >      <skipped>
> > >     BUILD FAILED...
> > >      <skipped>
> > >
> > > And this can be fixed by:
> > >
> > >  /tmp/trunk $ cd working_jdktools
> > >  /tmp/trunk/working_jdktools $ ant fetch-depends
> > >
> > > Is it a known issue?
> > >
> > > I double check trunk/build.xml - looks like all tags for jdttools's
> > > fetch-depends are in place... So it may be my local issue... That's
why
> > > I ask.
> > >
> > > --
> > > Vladimir Beliaev
> > > Intel Middleware Products Division


-- 
Vladimir Beliaev
Intel Middleware Products Division

------=_Part_9987_29488115.1192813123913--