Return-Path: Delivered-To: apmail-ant-dev-archive@www.apache.org Received: (qmail 14037 invoked from network); 13 Feb 2008 09:12:54 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 13 Feb 2008 09:12:54 -0000 Received: (qmail 85703 invoked by uid 500); 13 Feb 2008 09:12:47 -0000 Delivered-To: apmail-ant-dev-archive@ant.apache.org Received: (qmail 85661 invoked by uid 500); 13 Feb 2008 09:12:47 -0000 Mailing-List: contact dev-help@ant.apache.org; run by ezmlm Precedence: bulk List-Unsubscribe: List-Help: List-Post: List-Id: "Ant Developers List" Reply-To: "Ant Developers List" Delivered-To: mailing list dev@ant.apache.org Received: (qmail 85650 invoked by uid 99); 13 Feb 2008 09:12:47 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 13 Feb 2008 01:12:47 -0800 X-ASF-Spam-Status: No, hits=2.0 required=10.0 tests=HTML_MESSAGE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of xavier.hanin@gmail.com designates 64.233.178.241 as permitted sender) Received: from [64.233.178.241] (HELO hs-out-0708.google.com) (64.233.178.241) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 13 Feb 2008 09:12:15 +0000 Received: by hs-out-0708.google.com with SMTP id n78so969512hsc.12 for ; Wed, 13 Feb 2008 01:12:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references; bh=ji/DJB5ZXWhJwETISHxWDzoMjRpa5O0Pqc6LHfr9OLs=; b=f2Gh0M5DHuaFPNOm9O90Rw/4OUq8wUiGUednUaN+PgAfWhJfCbzvvuDeGhiY4CC9Jacb0lxLM0l+4F2XUPNQUFT3dKnmpcL++k4OdxsL2L33w+9b7JmsyUtFSVUeuZRoBl0hwbAnYn1/MvtaKVdKXQGOl0ZCrwYB8ImLVo6+r1k= 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=iUEjQPdWdupaBJau6Oj+gA1ap0PmhL7kkijUQCY+DdsnRZZIKeHoaIjvI2gbiy0c2DS/3Rq4AgFUqQFVtEUutISdhC9LJwSAoi7FbLPXJ0uS04jmNpI0AI44fhRElBF3DHy3fTy0Tu7e0A0pisT/iEquIFHzNgjk2k/mpa61qd0= Received: by 10.140.126.14 with SMTP id y14mr1675134rvc.59.1202893941907; Wed, 13 Feb 2008 01:12:21 -0800 (PST) Received: by 10.141.171.19 with HTTP; Wed, 13 Feb 2008 01:12:21 -0800 (PST) Message-ID: <635a05060802130112s46ea0998uc6b89d436d41be30@mail.gmail.com> Date: Wed, 13 Feb 2008 10:12:21 +0100 From: "Xavier Hanin" To: "Ant Developers List" Subject: Re: Store pom dependency Managment in extra data In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_28222_22276878.1202893941906" References: X-Virus-Checked: Checked by ClamAV on apache.org ------=_Part_28222_22276878.1202893941906 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline On Feb 13, 2008 9:10 AM, Gilles Scokart wrote: > In the rewrite of the pom parser, I would like to store the maven > dependency > managment (and maybe also the properties) into the extra data of the > moduleRevisionId. > > That way, the corresponding ivy module descriptor would contains all the > info coming from the pom. I think it might be usefull when it is > serialized > are reused from the cache (thought I'm not sure if it's the original pom > or > the serialized ivy.xml that is used in that case). > > However, I have noticed that the extra attributes are used to compare > moduleRevisionId, and I fear that might bring some problem. Does it? Probably. > And > is it normal to use the extra attributes in this comparison? I think so, many time people use extra attributes in the info tag to add extra identification attributes. I don't really think putty meta information such as the dependency management information in extra attributes is a good fit. But I like the idea of keeping data from dependency management section. Maybe a good way to do it would be to add a Map of properties to the module descriptor. This would be another area of flexibility for Ivy users, and I'd even see usage for Ivy files, if we actually make them usable as properties in the Ivy file. Something like: WDYT? Xavier > > > -- > Gilles Scokart > -- Xavier Hanin - Independent Java Consultant http://xhab.blogspot.com/ http://ant.apache.org/ivy/ http://www.xoocode.org/ ------=_Part_28222_22276878.1202893941906--