Maven 1.x Distribution Plugin

Make inclusion of site docs optional for binary distribution

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.7
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

The fix for MPDIST-8 unfortunately has made "site" a prereq for dist:build-bin. OK, the fix does make the behaviour predictable, but I would not say "right". IMHO the full project site docs don't belong in a binary distribution at all. What you need there is exactly what was bundled in RC2: the jar, and the javadoc to use it. The site docs don't belong in the source distribution either, as it is so easy to generate them from the source.

Not to break anything depending on the current behaviour, the site plugin would need a property like maven.dist.bin.includeSite=true/false, the default being true.

Activity

Hide
Geoffrey De Smet added a comment -

My project is for end-user (it's a game desktop app),
the kind of people that will freak out and turn into psychopats if they see complicated things like javadocs

So can you it a variable with 3 types:

  • all
  • javadocs
  • none
Show
Geoffrey De Smet added a comment - My project is for end-user (it's a game desktop app), the kind of people that will freak out and turn into psychopats if they see complicated things like javadocs So can you it a variable with 3 types:
  • all
  • javadocs
  • none
Hide
Geoffrey De Smet added a comment -

For performance it would also be nice that if "none" is selected, the site goal doesn't actually run

Show
Geoffrey De Smet added a comment - For performance it would also be nice that if "none" is selected, the site goal doesn't actually run
Hide
Lukas Theussl added a comment -

Added maven.dist.bin.include.site property.

Show
Lukas Theussl added a comment - Added maven.dist.bin.include.site property.

People

Vote (1)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: