Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 914621

Summary: app-emacs/nxml-gentoo-schemas: make metadata.xml indentation consistent with app-vim/gentoo-syntax
Product: Gentoo Linux Reporter: Rahil Bhimjiani <me>
Component: Current packagesAssignee: GNU Emacs project <gnu-emacs>
Status: RESOLVED INVALID    
Severity: normal CC: mgorny, vim
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description Rahil Bhimjiani 2023-09-25 01:07:31 UTC
For metadata.xml emacs uses 4 spaces and vim uses tabs which causes messy diffs and CI to complain about indentation even though it looks fine in Emacs. 

https://github.com/gentoo/gentoo-syntax/blob/master/ftplugin/gentoo-metadata.vim#L16

So please solve this discrepancy. Thank you
Comment 1 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2023-09-25 01:08:34 UTC
CI only complains about inconsistent indentation, to be clear, rather than one choice or the other.
Comment 2 Ulrich Müller gentoo-dev 2023-09-25 09:07:31 UTC
We don't have a policy on spaces vs tabs in XML. Therefore it is customisable by the user in app-emacs/ebuild-mode (with the default following that of Emacs, namely to use two spaces):

M-x customize-variable RET ebuild-mode-xml-indent-tabs RET

Ebuild Mode Xml Indent Tabs: Boolean: Toggle  off (nil)
   State : STANDARD.
   If non-nil, use tab characters for indenting of XML.
   If nil, use two spaces.
Groups: Ebuild


I am closing this bug as invalid, because nxml-gentoo-schemas installs only the Relax-NG schemas, which are about XML syntax but don't say anything about whitespace.