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

tar command 'U' option not supported on Solaris OS.

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Environment:
      Sun Solaris v5.8
    • Number of attachments :
      1

      Description

      Unlike the site plugin that uses a property to set the tar program used (maven.site.tar.executable), the dist plugin uses the hard coded string 'tar' to run the tar command. This is a problem on Solaris OS where the 'U' option is not supported by the standard tar program.

      Below is the snippet of plugin.jelly script in question (dist:deploy goal) -

      <deploy:artifact
      artifact="distributions.tar.gz"
      type="distributions"
      assureDirectoryCommand="mkdir -p"
      siteCommand="cd @deployDirectory@;gunzip distributions.tar.gz;tar xUvf distributions.tar;chmod -R g+u *;rm distributions.tar"
      />

      It would seem appropriate/consistent to replace the hard coded string with a 'maven.dist.tar.executable' to allow a different tar program, i.e. GNU tar, that supports the 'U' to be used. This would default to a value of 'tar' and therefore be transparent in behaviour for most people.

      This bug was also present in Beta 10.

      Cheers,

      Martin.

      1. dist.patch
        3 kB
        Arnaud Heritier

        Activity

        Hide
        Arnaud Heritier added a comment -

        same thing for me under HPX11.

        Show
        Arnaud Heritier added a comment - same thing for me under HPX11.
        Hide
        Arnaud Heritier added a comment -

        This patch add the properties maven.dist.tar.executable and maven.dist.gunzip.executable as in the site plugin.

        Show
        Arnaud Heritier added a comment - This patch add the properties maven.dist.tar.executable and maven.dist.gunzip.executable as in the site plugin.
        Hide
        Brett Porter added a comment -

        thanks, but as coincidence would have it I did just this today already

        Show
        Brett Porter added a comment - thanks, but as coincidence would have it I did just this today already

          People

          • Assignee:
            Brett Porter
            Reporter:
            Martin Lambert
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: