Change 20100401-bargull-TPw by barg...@bargull02 on 2010-04-01 23:44:11
in /home/anba/src/svn/openlaszlo/trunk
for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: tweak build process for windows

Bugs Fixed: LPP-8858 (Reference: make reference build work on windows machines)

Technical Reviewer: ptw
QA Reviewer: ?
Doc Reviewer: (pending)

Overview:
Building the docs on windows should be working now


Details:
Comment.java:
- split comment lines in an os-independent way, that means don't use the "line.separator" system property, but a regular expression to detect the line ending - this change is necessary, because some files used solely \n to break lines, so applying the windows line separator \r\n didn't detect an EOL

build.xml:
Both path transformations are performed on the caller side, because
1) too much logic makes xsl even more awkward to read
2) and I wanted to avoid duplicate code in the ruby scripts
- target "dbk.examples.preprocess"
-- replace any backslash characters with normal slashes, this is necessary because the "base.dir" param (see dbkpreprocessexamples.xsl) is a part of a file-protocol URI (file:///..)
- target "reference.frameset.generate"
-- normalize path to use slash characters, apparently ruby gets confused when you mix / and \ in a single path -- if toccheck fails it shouldn't make the complete build fail when running the QUICK version (QUICK uses only a small subset of all classes, therefore toccheck will always fail in this mode)

js2doc-verbose.xsl:
- re-order elements, xls:attribute must appear before any output elements (xsl:message is considered as an output element!), otherwise the xslt engine reports a warning ("attribute xyz outside of element")

js2doc-comment2dbk.xsl:
- <xalanredirect:write> doesn't work for me (no idea why), I had to add <redirect:write> and the corresponding xml namespace declaration for "redirect". - next curiosity: for <redirect:write>, I had to prefix the path with "build/reference/" (no idea why)

dbkpreprocessexamples.xsl:
- changed programsource variable to use the file-protocol to refer to local files (programsource is an absolute path and on windows absolute paths start with the drive letter. But the document() function expects a valid URI, therefore it interprets the drive letter as the protocol name, this approach won't succeed since there is no valid protocol named "C" or "D" or on whatever drive OpenLaszlo is installed)


build-opt.xml:
- windows restriction is no longer necessary if this patch works


This patch works with my local Windows 7 plus cygwin environment, but it should be tested on other Windows environments. Is anybody out there using Windows, so he can perform the necessary QA?


Tests:
cd docs/src
ant clean build

Files:
M WEB-INF/lps/server/src/org/openlaszlo/js2doc/Comment.java
M docs/src/build.xml
M docs/src/xsl/js2doc-verbose.xsl
M docs/src/xsl/js2doc-comment2dbk.xsl
M docs/src/xsl/dbkpreprocessexamples.xsl
M build-tools/build-opt.xml

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20100401-bargull-TPw.tar

Reply via email to