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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2/19/19 11:27 AM, Fabiano Fidêncio wrote:
> Signed-off-by: Fabiano Fidêncio <fidencio@xxxxxxxxxx>
> ---
>  .../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@xxxxxxxxxx>

- Cole

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux