On 01/02/2007, at 3:57 PM, Edwin Punzalan wrote:


getArtifacts is transitive and getDependencies isn't.

Right - that's what I was sort of implying - does the affect the code in question? I would have thought that for this phase, it wouldn't - but I could be (probably am?) wrong.

How about a manual artifact resolution using the project builder ?

Yes good thinking - that's what I thought would be a possible alternative, but it's a bit ugly. Probably worth it though, as the failure is a pain either way.

- Brett



Brett Porter wrote:
I see - I suppose it is only failing when the artifacts are not present in the repository already, as it was working for me...

Can the code using getArtifacts() be changed to getDependencies() without changing the meaning? (Sorry, I haven't looked, I just know the dep resolution was problematic)

- Brett

On 01/02/2007, at 2:54 PM, Edwin Punzalan wrote:

Brett,

Its failing in phase check-dependency-snapshot phase... the phase uses project.getArtifacts() which is not populated when its missing


Brett Porter wrote:
Edwin - can we double check the requirement here. I recently removed this to prevent the problem of it failing if you haven't installed the project first.

What step does it fail at?

- Brett

On 01/02/2007, at 1:46 PM, [EMAIL PROTECTED] wrote:

Author: epunzalan
Date: Wed Jan 31 18:46:11 2007
New Revision: 502089

URL: http://svn.apache.org/viewvc?view=rev&rev=502089
Log:
require dependency resolution so snapshot dependency artifacts checking will work

Modified:
maven/release/trunk/maven-release-plugin/src/main/java/org/ apache/maven/plugins/release/PrepareReleaseMojo.java

Modified: maven/release/trunk/maven-release-plugin/src/main/ java/org/apache/maven/plugins/release/PrepareReleaseMojo.java URL: http://svn.apache.org/viewvc/maven/release/trunk/maven- release-plugin/src/main/java/org/apache/maven/plugins/release/ PrepareReleaseMojo.java?view=diff&rev=502089&r1=502088&r2=502089 ================================================================== ============ --- maven/release/trunk/maven-release-plugin/src/main/java/org/ apache/maven/plugins/release/PrepareReleaseMojo.java (original) +++ maven/release/trunk/maven-release-plugin/src/main/java/org/ apache/maven/plugins/release/PrepareReleaseMojo.java Wed Jan 31 18:46:11 2007
@@ -34,6 +34,7 @@
  * @author <a href="mailto:[EMAIL PROTECTED]">Brett Porter</a>
  * @version $Id$
  * @aggregator
+ * @requiresDependencyResolution test
  * @goal prepare
  * @todo [!] check how this works with version ranges
  */


------------------------------------------------------------------- --
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-------------------------------------------------------------------- -
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to