On 18 Mar 07, at 5:52 PM 18 Mar 07, Brett Porter wrote:
What was your justification for removing them?
Actually, as a follow up to show why what we have currently as not
being ideal is the XhtmlSink. This is something I started and was a
mistake in the form that I did it because I did merge in the concept
of site. What we ended up with is a coupling in the RenderingContext
about physical locations of sites. The XhtmlSink should purely render
out Xhtml so that it would work in a system producing dynamic content
where something like sitemesh would do the decorating. The XhtmlSink
could have easily been subclassed to allow for site capabilities, but
the coupling happened because the concerns of pure output were not
separated from the site concept.
Jason.
I have no objection to splitting the site plugin into some maven
site related tools, but I don't think these belong in there.
- Brett
On 19/03/2007, at 5:54 AM, [EMAIL PROTECTED] wrote:
Author: jvanzyl
Date: Sun Mar 18 11:54:20 2007
New Revision: 519667
URL: http://svn.apache.org/viewvc?view=rev&rev=519667
Log:
o site specific modules moved to sitetools
Removed:
maven/doxia/trunk/doxia-decoration-model/
maven/doxia/trunk/doxia-doc-renderer/
maven/doxia/trunk/doxia-site-renderer/
Modified:
maven/doxia/trunk/pom.xml
Modified: maven/doxia/trunk/pom.xml
URL: http://svn.apache.org/viewvc/maven/doxia/trunk/pom.xml?
view=diff&rev=519667&r1=519666&r2=519667
=====================================================================
=========
--- maven/doxia/trunk/pom.xml (original)
+++ maven/doxia/trunk/pom.xml Sun Mar 18 11:54:20 2007
@@ -146,9 +146,6 @@
<modules>
<module>doxia-core</module>
<module>doxia-sink-api</module>
- <module>doxia-site-renderer</module>
- <module>doxia-decoration-model</module>
<module>doxia-modules</module>
- <module>doxia-doc-renderer</module>
</modules>
</project>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
