Seema Alevoor wrote:
>
> Oh... overlooked "/usr" in your path.
> 
> We want to make it easier for the users to use these
> additional configurations. I think moving configuration related files
> outside /etc would cause more confusion.

Trying to summarize..

#1: First there are upstream distro samples, these live currently
under /etc/apache2/2.2/original/.  These are just samples in the most
generic sense of the word. They don't necessarily even work as
delivered (though they may), they just demonstrate various
possibilities. They are delivered as-is from the upstream sources.
Early in web stack project life there was a discussion of whether they
go in /etc or /usr.

#2: Then there's the OpenSolaris-tuned samples, like your ssl.conf
here.  This category contains configuration that has already been
customized for the needs of these SUNWapch22* packages on
OpenSolaris. It is configuration that we can say is tested to work as
given, if suitably enabled, but which can't be enabled out of the box
because not everyone needs it. Like in this case, not every apache22
install is using SSL.

With this patch, looks like the same files are duplicated in
/etc/apache2/2.2/original/ and /etc/apache2/2.2/samples-conf.d/ with
the only(?) difference that ssl.conf is slightly different. That's a
lot of duplication! Can we eliminate so much duplication?

Earlier you had suggested /etc/apache2/2.2/conf.d/ssl.conf.disabled
(or similar), what made you decide move it to samples-conf.d/ instead?

How about eliminating the duplicates under original/extra/ and
delivering only OpenSolaris-tuned versions [which for many files might
be identical, but doesn't have to be] in a suitable location. Whether
that's conf.d/*.conf.disabled or samples-conf.d?


-- 
Jyri J. Virkki - jyri.virkki at sun.com - Sun Microsystems

Reply via email to