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

Bug 541576

Summary: dev-util/universalindentgui-1.2.0-r1: support for Uncrustify indenter
Product: Gentoo Linux Reporter: . <dev.rindeal+gentoo>
Component: New packagesAssignee: Qt Bug Alias <qt>
Status: RESOLVED WONTFIX    
Severity: enhancement CC: qt
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description . 2015-02-28 02:03:50 UTC
Although UniversalIndentGUI supports 20+ backends, only a small portion of them is supported in the ebuild. I prefer Uncrustify, which is missing and thus I've patched the ebuild to support it (hope I've done it right). Patches are attached in the "Additional information" section.

Reproducible: Always




Patch for ebuild:

@@ -15,7 +15,7 @@
 LICENSE="GPL-2"
 SLOT="0"
 KEYWORDS="~amd64 ~x86"
-IUSE="debug examples html perl php python ruby xml"
+IUSE="debug examples html perl php python ruby xml uncrustify"
 
 DEPEND="dev-qt/qtcore:4
 	dev-qt/qtgui:4
@@ -34,6 +34,7 @@
 	php? ( dev-php/PEAR-PHP_Beautifier )
 	ruby? ( dev-lang/ruby )
 	xml? ( dev-util/xmlindent )
+	uncrustify? ( dev-util/uncrustify )
 "
 
 DOCS="CHANGELOG.txt readme.html"
@@ -99,6 +100,10 @@
 		UIGUIFILES="${UIGUIFILES} xmlindent"
 	fi
 
+	if use uncrustify; then
+		UIGUIFILES="${UIGUIFILES} uncrustify"
+	fi
+
 	local IFILES= I=
 	for I in ${UINDENTERS}; do
 		IFILES="${IFILES} indenters/${I}"



Patch for metadata.xml:


@@ -8,5 +8,6 @@
 	</maintainer>
 	<use>
 		<flag name="html">Add support for HTML files</flag>
+		<flag name="uncrustify">Add support for Uncrustify indenter and beautifier</flag>
 	</use>
 </pkgmetadata>
\ No newline at end of file
Comment 1 Michael Palimaka (kensington) gentoo-dev 2015-03-04 18:35:08 UTC
Given the large number of backends, I wonder if it would make sense to install all the support files unconditionally, then just add an elog advising of backend names that are packaged.
Comment 2 . 2015-03-08 12:15:17 UTC
(In reply to Michael Palimaka (kensington) from comment #1)
> Given the large number of backends, I wonder if it would make sense to
> install all the support files unconditionally, then just add an elog
> advising of backend names that are packaged.

I've just reviewed universalindentgui package in debian (https://sources.debian.net/src/universalindentgui/1.2.0-1/debian/control/) and arch (https://aur.archlinux.org/packages/universalindentgui/), and they both install full universalindentgui without backends and then recommends these backends as optional deps. So yes, it'd probably make sense.
Comment 3 Michael Palimaka (kensington) gentoo-dev 2017-10-12 12:24:37 UTC
Package removed.