Maven 1

Running Maven in a directory not containing a POM file always results in 'Build Successful'.

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.0.2, 1.1-beta-1
  • Fix Version/s: 1.1-beta-3
  • Component/s: core
  • Labels:
    None
  • Environment:
    Not of importance.
  • Number of attachments :
    0

Description

In maven 1.0.2 and 1.1-beta-1, running 'maven' in a directory that doesn't contain a POM file always results in 'Build Succesful'.

======================================================
$.\maven.bat
__ __

\/ __ Apache_ ___
  \/ / ` \ V / -) ' \ ~ intelligent projects ~
_   _,_ _/___ _ _ v. 1.1-beta-1

BUILD SUCCESSFUL
Total time : 1 seconds
Finished at : vrijdag 8 juli 2005 13:32:35 CEST
======================================================

This could be confusing for novice users of Maven and it would be better to generate an error stating
that no POM file was found.

Activity

Hide
Davy Toch added a comment -

Just a small correction : an error should be generated when no POM is found and when no maven.xml is found, because it is sometimes useful to have a Maven project with just a maven.xml script. The latter is similar to an ANT project with just a build.xml file but has the additional advantage that Jelly scripting is available (thus avoiding stupid restrictions in ANT like : no variables, no logical constructs like if, foreach, ...).

Show
Davy Toch added a comment - Just a small correction : an error should be generated when no POM is found and when no maven.xml is found, because it is sometimes useful to have a Maven project with just a maven.xml script. The latter is similar to an ANT project with just a build.xml file but has the additional advantage that Jelly scripting is available (thus avoiding stupid restrictions in ANT like : no variables, no logical constructs like if, foreach, ...).
Hide
Lukas Theussl added a comment -

Just issuing a warning message should be enough.

Show
Lukas Theussl added a comment - Just issuing a warning message should be enough.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: