Shouldn't you have
      <xsl:if test="position()!= last()

Instead of

> <xsl:if test="position()=last()">

 Tom


 XSL-List info and archive:  http://www.mulberrytech.com/xsl/xsl-list