[Libosinfo] [osinfo-db PATCH v2 09/29] ubuntu, jeos: Fix hostname's template indentation

Cole Robinson crobinso at redhat.com
Wed Feb 27 18:55:04 UTC 2019


On 2/19/19 11:27 AM, Fabiano Fidêncio wrote:
> Signed-off-by: Fabiano Fidêncio <fidencio at redhat.com>
> ---
>  .../ubuntu.com/ubuntu-preseed-jeos.xml.in      | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/data/install-script/ubuntu.com/ubuntu-preseed-jeos.xml.in b/data/install-script/ubuntu.com/ubuntu-preseed-jeos.xml.in
> index 37f9d7f..4f4bc69 100644
> --- a/data/install-script/ubuntu.com/ubuntu-preseed-jeos.xml.in
> +++ b/data/install-script/ubuntu.com/ubuntu-preseed-jeos.xml.in
> @@ -44,16 +44,16 @@
>            </xsl:choose>
>          </xsl:template>
>  
> -	<xsl:template name="hostname">
> -	  <xsl:choose>
> -	    <xsl:when test="config/hostname != ''">
> +        <xsl:template name="hostname">
> +          <xsl:choose>
> +	        <xsl:when test="config/hostname != ''">
>                <xsl:value-of select="config/hostname"/>
> -	    </xsl:when>
> -	    <xsl:otherwise>
> -	      <xsl:text>ubuntu</xsl:text>
> -	    </xsl:otherwise>
> -	  </xsl:choose>
> -	</xsl:template>
> +	        </xsl:when>
> +	        <xsl:otherwise>
> +	          <xsl:text>ubuntu</xsl:text>
> +	        </xsl:otherwise>
> +	      </xsl:choose>
> +	    </xsl:template>
>  
>  <xsl:template match="/install-script-config">d-i debian-installer/locale string <xsl:value-of select="config/l10n-language"/>
>  d-i console-setup/ask_detect boolean false
> 

There's still hard tabs here afterwards though, so please fix it if it
doesn't cause too much conflict later in the series.

We should remove all hardtabs from the code base IMO but it doesn't need
to block this series.

Reviewed-by: Cole Robinson <crobinso at redhat.com>

- Cole




More information about the Libosinfo mailing list