Maven 1.x Distribution Plugin
  1. Maven 1.x Distribution Plugin
  2. MPDIST-19

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
    • 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

          • Assignee:
            Lukas Theussl
            Reporter:
            Anders Westlund
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: