[ 
http://jira.codehaus.org/browse/MERCURY-48?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=157957#action_157957
 ] 

Herve Boutemy commented on MERCURY-48:
--------------------------------------

here are my feelings about the proposed syntax:

- {{config}}
. to be consistent with 
[settings|http://maven.apache.org/ref/current/maven-settings/settings.html], I 
think {{localRepository}} and {{repository}} would be better than {{repo}}
. and {{layout}} instead of {{type}}, exactly for the same reason
. why no {{settings}} instead of {{config}}?

- {{dep}}
. {{dependencies}} would be better than {{dep}} IMHO, to be consistent with 
[pom|http://maven.apache.org/ref/current/maven-model/maven.html]
. I think you meant {{dep pom="${basedir}/pom.xml"}} instead of {{dependency 
pom=...}}: {{pom}} and {{dependency}} are mutually exclusive
. I don't understand the meaning of {{xmlns}} attribute of {{dependency}}
. adding classic {{groupId}}/{{artifactId}}/{{version}}/{{type}}/{{scope}} 
coordinates, mutually exlusive with {{name}}, would let people choose the 
syntax they prefer

- {{resolve}}
. ok with {{pathid}} instead of {{pathId}}: this is consistent with Ant 
conventions

- {{write}}
. sould support {{pom}} attribute, mutually exclusive with {{name}}
. I'd prefer {{file}} instead of {{bin}}, since it's the name used by Maven Ant 
Tasks and I don't find {{bin}} really expressive
. I suppose {{attach}} is scheduled for a later update

> Document and vet the design
> ---------------------------
>
>                 Key: MERCURY-48
>                 URL: http://jira.codehaus.org/browse/MERCURY-48
>             Project: Mercury
>          Issue Type: Sub-task
>          Components: Ant tasks
>            Reporter: Oleg Gusakov
>            Assignee: Oleg Gusakov
>   Original Estimate: 2 days
>          Time Spent: 1 day
>  Remaining Estimate: 1 day
>


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to