Issue Details (XML | Word | Printable)

Key: MPDIST-2
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Brett Porter
Reporter: Martin Lambert
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Maven 1.x Distribution Plugin

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

Created: 29/Sep/03 06:21 AM   Updated: 13/Apr/04 06:53 PM   Resolved: 13/Apr/04 06:53 PM
Component/s: None
Affects Version/s: None
Fix Version/s: None

Time Tracking:
Not Specified

File Attachments: 1. Text File dist.patch (3 kB)

Environment: Sun Solaris v5.8


 Description  « Hide

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.



Arnaud Heritier added a comment - 13/Apr/04 09:47 AM

same thing for me under HPX11.


Arnaud Heritier added a comment - 13/Apr/04 09:57 AM

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


Brett Porter added a comment - 13/Apr/04 06:53 PM

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