I wrote

  <xsl:template match="version[@version]">
  <xsl:if test="not(position()=last())">, </xsl:if>
     <xsl:value-of select="@version"/>
  </xsl:template>


make that

<xsl:template match="version[@version]">
<xsl:if test="not(position()=1)">, </xsl:if>
   <xsl:value-of select="@version"/>
</xsl:template>


or

<xsl:template match="version[@version]">
   <xsl:value-of select="@version"/>
<xsl:if test="not(position()=last())">, </xsl:if>
</xsl:template>

________________________________________________________________________
This e-mail has been scanned for all viruses by Star. The
service is powered by MessageLabs. For more information on a proactive
anti-virus service working around the clock, around the globe, visit:
http://www.star.net.uk
________________________________________________________________________