914621 – app-emacs/nxml-gentoo-schemas: make metadata.xml indentation consistent with app-vim/gentoo-syntax

Bug 914621 - app-emacs/nxml-gentoo-schemas: make metadata.xml indentation consistent with app-vim/gentoo-syntax

Summary: app-emacs/nxml-gentoo-schemas: make metadata.xml indentation consistent with ...

Status:	RESOLVED INVALID

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	GNU Emacs project

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2023-09-25 01:07 UTC by Rahil Bhimjiani
Modified:	2023-09-25 09:07 UTC (History)
CC List:	2 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

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

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.