Mailing-List: contact dev-help@ant.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Ant Developers List" <dev@ant.apache.org>
Received-SPF: pass (athena.apache.org: domain of xavier.hanin@gmail.com
 designates 74.125.92.144 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=message-id:date:from:to:subject:in-reply-to:mime-version
         :content-type:references;
        b=EBVz8kuvyCu4GWDf/WCBSVGmsDTjLUdYwv4BN9sYu4lnNP/NPfYcZXyghL5yDTIkg2
         9FDod753zbREoF/6OhKOVs43SffdF6RlizxZH56bFKF8y/9768z3siJli6zXbzVJpOMi
         u3LZIBww7W9rxD8QKY0aQaycmiyje7eMnRjS8=
Message-ID: <635a05060807160511m2eb15730ya560a4aacd608d51@mail.gmail.com>
Date: Wed, 16 Jul 2008 14:11:22 +0200
From: "Xavier Hanin" <xavier.hanin@gmail.com>
To: "Ant Developers List" <dev@ant.apache.org>
Subject: Re: Ivy svn breaks ant-contrib svn
In-Reply-To: <y1u7ibns5ux.fsf@v30161.1blu.de>
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----=_Part_19975_5403561.1216210282223"
References: <989542.37989.qm@web30803.mail.mud.yahoo.com>
	 <y1uprpg68ik.fsf@v30161.1blu.de> <y1utzer3afg.fsf@v30161.1blu.de>
	 <635a05060807150004n87b7f43h7a932be85aa155f5@mail.gmail.com>
	 <y1uprpf3913.fsf@v30161.1blu.de>
	 <635a05060807150044p1b3067aldcac0d508d0a8eca@mail.gmail.com>
	 <y1u7ibns5ux.fsf@v30161.1blu.de>

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

On Tue, Jul 15, 2008 at 2:13 PM, Stefan Bodewig <bodewig@apache.org> wrote:

> On Tue, 15 Jul 2008, Xavier Hanin <xavier.hanin@gmail.com> wrote:
>
> > On Tue, Jul 15, 2008 at 9:23 AM, Stefan Bodewig <bodewig@apache.org>
> wrote:
> >
> > > On Tue, 15 Jul 2008, Xavier Hanin <xavier.hanin@gmail.com> wrote:
> > >
> > > > On Tue, Jul 15, 2008 at 8:53 AM, Stefan Bodewig <bodewig@apache.org>
> > > wrote:
> > > >
> > > > > Ant-contrib also invokes setId() on the task, which works fine with
> > > > > Ivy 2.0.0beta2 but fails with trunk.
> > > > >
> > > > > Could you please re-add the setid method?
> > > >
> > > > setId is now setSettingsId on IvyConfigure, which is more in
> > > > conformance with Ant, since we are not setting the id of the
> > > > task, but of the underneath datatype.
> > >
> > > OK, what can ant-contrib do if it wants to compile against Ivy
> > > 2.0.0beta2 and trunk with the same codebase?
> >
> > This is not straightforward, since we broke the API.
>
> Right, that's why Gump finds it 8-)
>
> Maybe you could throw in a deprecated setId() method that delegated to
> setSettingsId()?  At least for the next beta so ant-contrib has a
> stable base to work from without resorting to reflection.
>
> I looked into the code to see what the id is used for.  It is later
> used as the argument for IvyCacheFileset.setSettingsRef - this
> wouldn't work with a reference to the task but would require a
> reference to the settings, which now would exactly be what
> setSettingsId creates, right?

Yes, I think setSettingsId is the good method to call, 2.0.0-beta2 was
broken in that sense. Introducing a setId method for backward compatiblity
with something broken in concept doesn't sound really nice IMO. We will
probably forget to remove it after, I don't like it too much. IMO antcontrib
should be built either against trunk, or against beta2, but not both.

But maybe others have a different opinion?

Xavier

>
>
> Stefan
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
> For additional commands, e-mail: dev-help@ant.apache.org
>
>


-- 
Xavier Hanin - Independent Java Consultant
http://xhab.blogspot.com/
http://ant.apache.org/ivy/
http://www.xoocode.org/

------=_Part_19975_5403561.1216210282223--