Hi,

Unfortunately, the documentation doesn't state that it **is** a separate module - which is precisely my point. Spring is so ubiquitous now that it would not be a wild assumption that Tapestry would come with Spring support out of the box. I think its reasonable enough to have Spring as a 'second' option to Tapestry IOC, if that is Howard's preference, but my point is the documentation (I followed the 'snippets' link on the HowTo on the wiki) emphasises just how little effort it takes to incorporate Spring - unfortunately it misses out some vital information i.e. "This bit of code has dependencies" - it would not harm the document to put the dependency markup right at the top.

David.



SergeEby wrote:
Hi,

It seems obvious to me that you need to add any self-contained module such
as tapestry-spring, tapestry-hibernate or tapestry-upload to your
application only if you use it.

/Serge


TapestryBeast wrote:
Hi,

Just to say that I think the documentation here:

http://tapestry.apache.org/tapestry5/tapestry-spring/

needs updating.


Nowhere does it state that a dependency on tapestry-spring is needed in pom.xml as such:

        <dependency>
            <groupId>org.apache.tapestry</groupId>
            <artifactId>tapestry-spring</artifactId>
            <version>${tapestry-release-version}</version>
        </dependency>

I know this might seem obvious, but I for one assumed that this would come with the Tapestry-core dependency files. This cost me 3 hours with Tomcat "startup failed due to previous errors" errors whenever spring was added to web.xml until I remembered to look in the full tomcat log to find out what the problem was.

Perhaps the same sort of update is also needed for the file upload page on the Tapestry web site as well.

Its probably difficult to decided what to do when half of users are still downloading the full gzip file and the other half are doing this stuff using Maven but I think this is definitely something worth adding.


David.


---------------------------------------------------------------------
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]

Reply via email to