[Buildroot] [PATCH] auto remove empty /usr/share dir

[Buildroot] [PATCH] auto remove empty /usr/share dir

[Mike Frysinger]

Often times, the only reason /usr/share exists is because of documentation
installed into it.  So once we're done cleaning the docs, attempt to rmdir
/usr/share.  If it's empty, things will work out nicely.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
 Makefile |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/Makefile b/Makefile
index 69e4031..4cc62e9 100644
--- a/Makefile
+++ b/Makefile
@@ -403,6 +403,7 @@ ifneq ($(BR2_HAVE_DOCUMENTATION),y)
 	rm -rf $(TARGET_DIR)/usr/info $(TARGET_DIR)/usr/share/info
 	rm -rf $(TARGET_DIR)/usr/doc $(TARGET_DIR)/usr/share/doc
 	rm -rf $(TARGET_DIR)/usr/share/gtk-doc
+	rmdir $(TARGET_DIR)/usr/share 2>/dev/null || :
 endif
 	find $(TARGET_DIR) -type f -perm +111 | xargs $(STRIPCMD) 2>/dev/null || true
 	find $(TARGET_DIR)/lib/modules -type f -name '*.ko' | \
-- 
1.7.3.2

[Buildroot] [PATCH] auto remove empty /usr/share dir

[Thomas Petazzoni]

On Thu, 18 Nov 2010 16:25:58 -0500
Mike Frysinger <vapier@gentoo.org> wrote:

> Often times, the only reason /usr/share exists is because of documentation
> installed into it.  So once we're done cleaning the docs, attempt to rmdir
> /usr/share.  If it's empty, things will work out nicely.
> 
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>

Acked-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [PATCH] auto remove empty /usr/share dir

[Peter Korsgaard]

>>>>> "Mike" == Mike Frysinger <vapier@gentoo.org> writes:

 Mike> Often times, the only reason /usr/share exists is because of
 Mike> documentation installed into it.  So once we're done cleaning the
 Mike> docs, attempt to rmdir /usr/share.  If it's empty, things will
 Mike> work out nicely.

Committed, thanks. I prefer to tell make to simply ignore the return
value rather than the '|| :' shell code though.

-- 
Bye, Peter Korsgaard