Bug#293989: xsl stylesheet does not work anymore

2005-02-07 Thread Wichert Akkerman
Package: db2latex-xsl
Version: 0.8pre1-2
Severity: grave

I am suddenly getting this on every run:

[typhoon;..nTest/trunk/newdoc]-100 make manual.tex
xsltproc --output manual.tex manual.xsl manual.xml
compilation error: file 
file:///usr/share/xml/docbook/stylesheet/db2latex/latex/qandaset.mod.xsl line 
366 element template
xsl:template: error duplicate name 'question.answer.label'
make: *** [manual.tex] Error 5
zsh: exit 2 make manual.tex

Wichert.


-- System Information:
Debian Release: 3.1
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing')
Architecture: i386 (i686)
Kernel: Linux 2.6.9
Locale: LANG=C, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)

-- no debconf information


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#293989: xsl stylesheet does not work anymore

2005-02-07 Thread Michael Wiedmann
Wichert Akkerman [EMAIL PROTECTED] wrote:

 Package: db2latex-xsl
 Version: 0.8pre1-2
 Severity: grave
 
 I am suddenly getting this on every run:
..
 xsl:template: error duplicate name 'question.answer.label'

A few weeks ago there was an identical bug report (289183) reported by Nikolai
Prokoschenko [EMAIL PROTECTED]. Together we figured out that this was
due to changed handling of duplicate templates in the newer versions of
xsltproc. 
So I guess you are using xsltproc from experimental?

A workaround for the time being is to use xsltprc from sarge. 
A general solution requires upstream modifications. Unfortunately up to now I
didn't get any response from upstream about this issue :-(

Michael
-- 


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#293989: xsl stylesheet does not work anymore

2005-02-07 Thread Wichert Akkerman
reassign 293989 libxslt1.1
title 293989 changed handling of duplicate templates breaks applications
thanks

Previously Michael Wiedmann wrote:
 So I guess you are using xsltproc from experimental?

No, it seems Mike Hommey uploaded the experimental version to unstable
last weekend. 

 A workaround for the time being is to use xsltprc from sarge.

And this bugreport should prevent the now broken version in unstable
from entering testing. To be precise, the package to be downgraded is
libxslt1.1, not xsltproc.

Wichert.

-- 
Wichert Akkerman [EMAIL PROTECTED]It is simple to make things.
http://www.wiggy.net/   It is hard to make things simple.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#293989: [xml/sgml-pkgs] Bug#293989: xsl stylesheet does not work anymore

2005-02-07 Thread Mike Hommey
reassign 293989 db2latex-xsl
retitle 293989 improper use of duplicate named templates
thanks

On Mon, Feb 07, 2005 at 12:33:46PM +0100, Wichert Akkerman [EMAIL PROTECTED] 
wrote:
 Package: db2latex-xsl
 Version: 0.8pre1-2
 Severity: grave
 
 I am suddenly getting this on every run:
 
 [typhoon;..nTest/trunk/newdoc]-100 make manual.tex
 xsltproc --output manual.tex manual.xsl manual.xml
 compilation error: file 
 file:///usr/share/xml/docbook/stylesheet/db2latex/latex/qandaset.mod.xsl line 
 366 element template
 xsl:template: error duplicate name 'question.answer.label'
 make: *** [manual.tex] Error 5
 zsh: exit 2 make manual.tex

Taken from:
http://www.w3.org/TR/xslt#named-templates

It is an error if a stylesheet contains more than one template with the
same name and same import precedence.

Thus, libxslt changed its broken behaviour to actually follow the XSLT
specification. Yes, it breaks broken stylesheets, but the breakage is
not in libxslt.

Reassigning to db2latex-xsl.

Mike


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#293989: [xml/sgml-pkgs] Bug#293989: xsl stylesheet does not work anymore

2005-02-07 Thread Wichert Akkerman
Previously Mike Hommey wrote:
 Thus, libxslt changed its broken behaviour to actually follow the XSLT
 specification. Yes, it breaks broken stylesheets, but the breakage is
 not in libxslt.

Can we keep a bug on libxslt1.1 as well to prevent db2latex from
breaking in testing? It would be somewhat unfortunate to have stricter
checking in libxslt1.1 break db2latex while there is no fix for it yet.

Alternatively, is there a hinting option to do that instead?

Wichert.

-- 
Wichert Akkerman [EMAIL PROTECTED]It is simple to make things.
http://www.wiggy.net/   It is hard to make things simple.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#293989: [xml/sgml-pkgs] Bug#293989: xsl stylesheet does not work anymore

2005-02-07 Thread Michael Wiedmann
Mike Hommey [EMAIL PROTECTED] wrote:

 Taken from:
 http://www.w3.org/TR/xslt#named-templates
 
 It is an error if a stylesheet contains more than one template with the
 same name and same import precedence.

But see also:
http://www.w3.org/TR/xslt#conflict

It is possible for a source node to match more than one template rule. The
template rule to be used is determined as follows:...

It is an error if this leaves more than one matching template rule. An XSLT
processor may signal the error; ...

I understand that xsltproc/libxslt acts according to this (may signal...).
In previous versions xsltproc/libxslt recovered from such a situation. It
would be good to know the reason why Daniel changed this.

Michael
-- 


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#293989: [xml/sgml-pkgs] Bug#293989: xsl stylesheet does not work anymore

2005-02-07 Thread Mike Hommey
On Mon, Feb 07, 2005 at 03:46:45PM +0100, Wichert Akkerman [EMAIL PROTECTED] 
wrote:
 Previously Mike Hommey wrote:
  Thus, libxslt changed its broken behaviour to actually follow the XSLT
  specification. Yes, it breaks broken stylesheets, but the breakage is
  not in libxslt.
 
 Can we keep a bug on libxslt1.1 as well to prevent db2latex from
 breaking in testing? It would be somewhat unfortunate to have stricter
 checking in libxslt1.1 break db2latex while there is no fix for it yet.
 
 Alternatively, is there a hinting option to do that instead?

This version of libxslt is bound to the latest version of libxml2 in sid,
and the previous one breaks with this latest version of libxml2. Thus if you
want to prevent libxslt from entering into testing, you have to do the
same for libxml2, which is another story, considering the reverse
dependencies on it.
The current version of libxml2 in sarge is not recommended for release for
several reasons (leaks, bugs...), fixed in the version currently in sid.
On the other hand, I see no huge reverse dependency list on db2latex-xsl.
While having a broken package is unfortunate, why blocking others
instead of trying to fix it ? (If you'd provide some more informations,
i'd come with a fix quite easily, i think it's a matter of replacing
some xsl:include by xsl:import).

Mike


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#293989: [xml/sgml-pkgs] Bug#293989: xsl stylesheet does not work anymore

2005-02-07 Thread Mike Hommey
On Mon, Feb 07, 2005 at 04:27:56PM +0100, Mike Hommey [EMAIL PROTECTED] wrote:
 (If you'd provide some more informations, i'd come with a fix quite easily,
 i think it's a matter of replacing some xsl:include by xsl:import).

So, taking a quick look into it shows that the occurence of
question.answer.label in common/common.xsl seems removeable. It is the
same as the one in qandaset.mod.xsl, there's only a difference of
fallback variable, but it seems that the one in common.xsl is useless
and is only here because common.xsl is taken from docbook-xsl.
So, i'd say there are 2 solutions :
- removing this templace in common/common.xsl
- replace the xsl:include by a xsl:import for the common/common.xsl file
  in docbook.xsl.

Wichert, if you could provide your test files so that i could give it a
try...

Mike


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#293989: [xml/sgml-pkgs] Bug#293989: xsl stylesheet does not work anymore

2005-02-07 Thread Mike Hommey
On Mon, Feb 07, 2005 at 09:11:06PM +0100, Michael Wiedmann [EMAIL PROTECTED] 
wrote:
 * Mike Hommey [EMAIL PROTECTED] wrote [050207 17:28]:
 ...
  So, i'd say there are 2 solutions :
  - removing this templace in common/common.xsl
  - replace the xsl:include by a xsl:import for the common/common.xsl file
in docbook.xsl.
 
 If done this already in my local CVS copy and it works.
 Unfortunately the upstream author James Devenish didn't respond to my
 email for already some months now.

It would be nice to put this fix in the debian package even if upstream
doesn't take it for now. Best would be to provide your patch, so that
Ardo or Wolfgang can upload a fixed version ;)

Thanks

Mike


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#293989: [xml/sgml-pkgs] Bug#293989: xsl stylesheet does not work anymore

2005-02-07 Thread Wichert Akkerman
(resend without typoed addresses)

Previously Mike Hommey wrote:
 So, i'd say there are 2 solutions :
 - removing this templace in common/common.xsl
 - replace the xsl:include by a xsl:import for the common/common.xsl file
   in docbook.xsl.

I tried the second approach and it worked fine. 

Ramon, James: DB2LaTeX does not work with current versions of the XSLT
and XML libraries due to the double definition of the
question.answer.label template. You can find more information about the
problem at http://bugs.debian.org/293989 .  The patch below fixes it.

The last maintainer release from Ardo was quite some time ago; I can
do a NMU if wanted to get this fix in.

Wichert.

diff -wur org/db2latex-xsl-0.8pre1/debian/changelog 
db2latex-xsl-0.8pre1/debian/changelog
--- org/db2latex-xsl-0.8pre1/debian/changelog   2005-02-07 21:46:01.0 
+0100
+++ db2latex-xsl-0.8pre1/debian/changelog   2005-02-07 21:40:59.0 
+0100
@@ -1,3 +1,12 @@
+db2latex-xsl (0.8pre1-2.1) unstable; urgency=medium
+
+  * Non-maintainer version
+  * Use xsl:import instead of xsl:include to load common/common.xsl .
+This works around the problematic double definition of the
+question.answer.label template. Fix from Mike Hommey.
+
+ -- Wichert Akkerman [EMAIL PROTECTED]  Mon, 07 Feb 2005 21:40:55 +0100
+
 db2latex-xsl (0.8pre1-2) unstable; urgency=low
 
   * debian/control: removed spurious newline from Build-Depends
diff -wur org/db2latex-xsl-0.8pre1/xsl/docbook.xsl 
db2latex-xsl-0.8pre1/xsl/docbook.xsl
--- org/db2latex-xsl-0.8pre1/xsl/docbook.xsl2004-03-13 17:36:19.0 
+0100
+++ db2latex-xsl-0.8pre1/xsl/docbook.xsl2005-02-07 21:39:24.0 
+0100
@@ -63,7 +63,7 @@
xsl:key name=id match=* use=@id/
 
xsl:include href=common/l10n.xsl/
-   xsl:include href=common/common.xsl/
+   xsl:import href=common/common.xsl/
xsl:include href=common/gentext.xsl/
xsl:include href=common/subtitles.xsl/
xsl:include href=common/titles.xsl/

-- 
Wichert Akkerman [EMAIL PROTECTED]It is simple to make things.
http://www.wiggy.net/   It is hard to make things simple.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]