Issue 112741 - filter: fails to build with system saxon
Summary: filter: fails to build with system saxon
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: DEV300m83
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: Mathias_Bauer
QA Contact: issues@framework
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2010-06-28 08:31 UTC by rene
Modified: 2011-02-28 13:46 UTC (History)
6 users (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description rene 2010-06-28 08:31:50 UTC 
with

XSLTransformer.java:81: package net.sf.saxon does not exist
import net.sf.saxon.FeatureKeys;
                   ^
XSLTransformer.java:298: cannot find symbol
symbol: variable FeatureKeys
                           
tfactory.setAttribute(FeatureKeys.ALLOW_EXTERNAL_FUNCTIONS, new Boolean(true));
                                                  ^
Note: XSLTransformer.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
2 errors

Trivial patch is

--- a/filter/source/xsltfilter/makefile.mk
+++ b/filter/source/xsltfilter/makefile.mk
@@ -66,7 +66,7 @@
 JARTARGET		= $(TARGET).jar
 
 .IF "$(SYSTEM_SAXON)" == "YES"
-XCLASSPATH+=$(SAXON_JAR)
+XCLASSPATH:=$(XCLASSPATH)$(PATH_SEPERATOR)$(SAXON_JAR)
 .ELSE
 JARFILES += saxon9.jar
 .ENDIF

hr/rt: can we masterfix this?
Comment 1 rene 2010-06-28 08:36:56 UTC 
was introduced by http://qa.openoffice.org/issues/show_bug.cgi?id=110136
integrated in m77 (cmcfixes73) -> regression
Comment 2 jens-heiner.rechtien 2010-06-28 11:21:55 UTC 
@obo: please apply patch as masterfix.
Comment 3 oliver.bolte 2010-06-28 11:41:34 UTC 
Fixed in DEV300_m84 with revision: c3ffaa0d55a2
Comment 4 Mathias_Bauer 2011-02-09 12:08:55 UTC 
@rene: I assume that for obvious reasons you don't want to verify the fix. Just
let me know if this assumption is right.
Comment 5 Mathias_Bauer 2011-02-28 13:46:28 UTC 
closing