[icedtea-web] RFC: support configure's --program-suffix option

[Deepak Bhole]

* Dr Andrew John Hughes <ahughes at redhat.com> [2011-04-28 12:21]:
> On 11:36 Thu 28 Apr     , Omair Majid wrote:
> > On 04/28/2011 11:29 AM, Deepak Bhole wrote:
> > > * Omair Majid<omajid at redhat.com>  [2011-04-28 11:05]:
> > >> Hi,
> > >>
> > >> The attached patch makes icedtea-web support configure's
> > >> --program-suffix option.
> > >>
> > >> A number of distributions use alternatives to manage
> > >> /usr/bin/javaws. Using a prefix of /usr makes IcedTea-Web's javaws
> > >> conflict with the alternatives-managed /usr/bin/javaws. Using a
> > >> --program-suffix is one way to work around this problem and the
> > >> patch makes this option work.
> > >>
> > >> This patch does not modify the names of desktop files, but does fix
> > >> the binary names they point to.
> > >>
> > >> I would like to add this to HEAD and 1.1. Does anyone have any
> > >> thoughts or comments?
> > >>
> > >
> > > Changes look fine to me. However I am not sure if itweb-settings needs
> > > to be changed too. It is IcedTea-Web specific and quite unlikely to have
> > > alternatives at any point in the future.
> > >
> > 
> > I agree that we should not see problems with itweb-settings. On the 
> > other hand, --program-suffix will allow you to install two different 
> > versions of itweb-settings with different names (say itweb-settings11 
> > and itweb-settings12) side by side (though I dont see why someone would 
> > want to do this).
> > 
> > Also applying the value of --program-suffix to only selected binaries 
> > breaks the meaning of --program-suffix. What do you think of having a 
> > special --javaws-suffix option?
> > 
> 
> I think the original patch is fine and both should support the option.
> There's no harm in providing the option to do it even if you personally
> can't think of a reason to.  The facility is there if someone wants to
> use it.  As Omair says, installing 1.1 and 2.0 would be one such case.
> 

Fair enough then. I am okay with this patch for HEAD and 1.1.

Cheers,
Deepak

> 
> -- 
> Andrew :)
> 
> Free Java Software Engineer
> Red Hat, Inc. (http://www.redhat.com)
> 
> Support Free Java!
> Contribute to GNU Classpath and IcedTea
> http://www.gnu.org/software/classpath
> http://icedtea.classpath.org
> PGP Key: F5862A37 (https://keys.indymedia.org/)
> Fingerprint = EA30 D855 D50F 90CD F54D  0698 0713 C3ED F586 2A37