Comments inline...
Alan Coopersmith wrote:
>> The CSW* packages come from blastwave.org
>
> blastwave.org or opencsw.org? (The CSW project recently forked into two
> separate projects.)
>
>> CSWexpect
>> CSWtcl
>> CSWtk
>
> Why are you using these instead of the versions already included in
> Solaris / OpenSolaris?
>
Sorry, this information in the 20 Questions is no longer correct. These
packages were previously included when we were supporting Solaris 10 on
x64 platforms, but we've now restricted that to just OpenSolaris, so the
above packages are no longer required.
>> SUNWreadline
>
> I don't see this in your Solaris_package_files.txt - are you shipping this
> in /usr/lib with headers in /usr/include? Are you warning users at all that
> any software they build that calls this may become subject to the GPL?
>
This was a recent addition as we've added support for SAGE. GNU
Readline is going through the OSR process (#10492). We have not yet
received legal approval for this component.
>> 8.1.1 For Solaris components, what Solaris release(s) does it run on
>> or work with?
>> OpenSolaris 2008.11
>
> Really? Not on Solaris 10 or earlier at all?
We only support OpenSolaris 2008.11. It may work on other versions, but
we will not officially support it (except on SPARC). It was requested
by the PAC to limit the number of supported OSes.
> How do you support SPARC then?
The limited set of components on the SPARC distribution are supported on
Solaris 10 U6.
> Why do you modify dtlogin configuration,
> which doesn't exist on Linux or OpenSolaris?
>
Sorry, this is also an artifact of when we were supporting Solaris 10
x64. On OpenSolaris and Linux we modify gdm.
>> * changes the default $PATH and $LD_LIBRARY_PATH for all users
>
> How does it change the default $PATH?
> How does it set $LD_LIBRARY_PATH without breaking other software?
We modify the /etc/{bashrc,csh.cshrc} to change the default environment
(for all users).
>
> Suggested TCR: If it is absolutely impossible to avoid setting
> LD_LIBRARY_PATH, LD_LIBRARY_PATH_32 & LD_LIBRARY_PATH_64 must
> be used to avoid breaking 32 vs. 64 bit applications.
>
On Solaris we use LD_LIBRARY_PATH_64 (only 64 bit applications are
supported), but this env. variable doesn't seem to exist on Linux so we
use LD_LIBRARY_PATH.
>> cr_start.sh name & arguments Committed Scalable Viz
>> User's Guide
>> dmx_start.sh name & arguments Committed Scalable Viz
>> User's Guide
>> paraview.sh name & arguments Committed Paraview docs
>> gui_start.sh name & arguments Committed Scalable Viz
>> User's Guide
>
> Why do your command names all end in *.sh? Will you rename them if in future
> versions they change from shell scripts to perl, python or C code?
>
They are all Bash scripts.
> What shell is used for these scripts? If it's /bin/sh, have you verified
> they
> work with both classic Solaris /bin/sh and ksh93 (which OpenSolaris uses as
> /bin/sh)?
>
>> We edit:
>> * /etc/dt/config/Xconfig to set Dtlogin*grabServer: False
>
> As much as it pains me to say it, given the security-reducing change you make:
> TCR: Perform equivalent modifications to gdm configuration as made to dtlogin
> configuration. (gdm is used on OpenSolaris as dtlogin is removed, and
> gdm may be used on Solaris 10 if accessibility is required or the site
> admin otherwise prefers it.)
>
The changes are being made to gdm on OpenSolaris and Linux.
>> ./SUNWdmx/reloc/dmx/include/X11/*
>> ./SUNWdmx/reloc/dmx/lib/X11/*
>
> You appear to be shipping a large portion of the X Window System, including
> many libraries and headers which would already be present on the system, and
> which if included in the global LD_LIBRARY_PATH you set would break other
> applications.
>
> TCR: Stop that!
>
Are you saying DMX is available on OpenSolaris -- we could not find it
the last time we looked.
Thanks for the feedback,
Steve
