Mail archive
alpine-devel

[alpine-devel] [PATCH] abuild: check for /usr/share/{doc,man}

From: Sören Tempel <soeren+git_at_soeren-tempel.net>
Date: Fri, 14 Aug 2015 17:07:28 +0200

Output a warning if those directories exist on a non-doc package.
Furthermore, check if uncompressed man pages where installed to
/usr/share/man.

Maybe a similar check would be a good idea for the -libs and -dev
packages?
---
 abuild.in | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)
diff --git a/abuild.in b/abuild.in
index 01b4b62..17fae62 100644
--- a/abuild.in
+++ b/abuild.in
_at_@ -645,6 +645,24 @@ postcheck() {
 		error "Found /usr/lib/charset.alias"
 		return 1
 	fi
+	# look for /usr/share/doc
+	if [ -e "$dir"/usr/share/doc ] \
+			&& ! is_doc_pkg; then
+		warning "Found /usr/share/doc but package name doesn't end with -doc"
+	fi
+	# look for /usr/share/man
+	if [ -e "$dir"/usr/share/man ] \
+			&& ! is_doc_pkg; then
+		warning "Found /usr/share/man but package name doesn't end with -doc"
+
+		# check for uncompressed man pages
+		i=$(find "$dir"/usr/share/man -name '*.[0-9]' -type f | sed "s|^$dir|\t|")
+		if [ -n "$i" ]; then
+			error "Found uncompressed man pages:"
+			echo "$i"
+			return 1
+		fi
+	fi
 	# check directory permissions
 	i=$(find "$dir" -type d -perm -777 | sed "s|^$dir|\t|")
 	if [ -n "$i" ]; then
_at_@ -946,6 +964,11 @@ is_dev_pkg() {
 	test "${subpkgname%-dev}" != "$subpkgname"
 }
 
+# returns true if this is the -doc package
+is_doc_pkg() {
+	test "${subpkgname%-doc}" != "$subpkgname"
+}
+
 # check that noarch is set if needed
 archcheck() {
 	options_has "!archcheck" && return 0
-- 
2.5.0
---
Unsubscribe:  alpine-devel+unsubscribe_at_lists.alpinelinux.org
Help:         alpine-devel+help_at_lists.alpinelinux.org
---
Received on Fri Aug 14 2015 - 17:07:28 UTC