Issue Details (XML | Word | Printable)

Key: MNG-2609
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Paul Gier
Reporter: Gwyn Evans
Votes: 1
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
Maven 2

Mention 'activeByDefault' in the "Introduction to Build Profiles" guide

Created: 13/Oct/06 04:26 AM   Updated: 03/Sep/08 04:59 PM
Component/s: Documentation: Guides
Affects Version/s: None
Fix Version/s: 2.0.10, 2.1.0-M1

Time Tracking:
Not Specified

Complexity: Intermediate


 Description  « Hide
It appears to be that the primary use-case for someone considering profiles would be the requirement to modify an existing build in some way to accomodate a 'special-case'. As such, it seems to me that a mention of the activation option 'activeByDefault' should be added.
At present, the document implies that there either needs to be a change in the command line (e,g, "-Denv-prod") or some form of environment parsing to have the build run as before, whereas 'activeByDefault' show that's not so but is not mentioned.

 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Olav Reinert added a comment - 20/May/07 04:48 AM
Also, the current description of 'activeByDefault' in the POM reference (http://maven.apache.org/ref/current/maven-model/maven.html) is sort of misleading, I find. At least, it doesn't add any value beside what the tag name itself offers: "Flag specifying whether this profile is active by default."

Diving a bit deeper (i.e., reading the discussions in related bug reports, such as MNG-835 and MNG-2136) it becomes clear that the semantics of 'activeByDefault' are not entirely trivial. The tag makes sense to have, and it's useful, but unfortunately it remains largely undocumented...


Paul Gier added a comment - 15/May/08 06:35 PM
Added some notes about activeByDefault to the site, and added a better description of activeByDefault to the model.

John Casey added a comment - 03/Sep/08 04:59 PM
Adding fix-for for both 2.0.10 and 2.1.0-M1, since 2.1.0-M1 will actually be released first and may not incorporate all of the eventual issue fixes released in 2.0.10.