All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH][meta-filesystems] fuse: Remove unneeded RDEPENDS on util-linux-mount
@ 2017-02-26 11:33 Mike Looijmans
  2017-02-28 14:37 ` Philip Balister
  0 siblings, 1 reply; 2+ messages in thread
From: Mike Looijmans @ 2017-02-26 11:33 UTC (permalink / raw)
  To: openembedded-devel

Fuse claimed to need util-linux-mount at runtime, which isn't true. This
drags util-linux-mount into any image that uses fuse.

Encountered no problems with busybox's mount command and fuse (and never
had). Fuse doesn't call the "mount" program anywhere, so the dependency
doesn't make sense anyway.

Signed-off-by: Mike Looijmans <mike.looijmans@topic.nl>
---
 meta-filesystems/recipes-support/fuse/fuse_2.9.4.bb | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/meta-filesystems/recipes-support/fuse/fuse_2.9.4.bb b/meta-filesystems/recipes-support/fuse/fuse_2.9.4.bb
index 9a2dc11..153fcb4 100644
--- a/meta-filesystems/recipes-support/fuse/fuse_2.9.4.bb
+++ b/meta-filesystems/recipes-support/fuse/fuse_2.9.4.bb
@@ -29,9 +29,6 @@ DEPENDS = "gettext-native"
 
 PACKAGES =+ "fuse-utils-dbg fuse-utils libulockmgr libulockmgr-dev libulockmgr-dbg"
 
-# Fusermount requires features from the util-linux version of mount.
-RDEPENDS_${PN}_class-target += "util-linux-mount"
-
 RRECOMMENDS_${PN}_class-target = "kernel-module-fuse libulockmgr fuse-utils"
 
 FILES_${PN} += "${libdir}/libfuse.so.*"
-- 
2.1.4



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH][meta-filesystems] fuse: Remove unneeded RDEPENDS on util-linux-mount
  2017-02-26 11:33 [PATCH][meta-filesystems] fuse: Remove unneeded RDEPENDS on util-linux-mount Mike Looijmans
@ 2017-02-28 14:37 ` Philip Balister
  0 siblings, 0 replies; 2+ messages in thread
From: Philip Balister @ 2017-02-28 14:37 UTC (permalink / raw)
  To: Mike Looijmans, openembedded-devel, jackie.huang

On 02/26/2017 06:33 AM, Mike Looijmans wrote:
> Fuse claimed to need util-linux-mount at runtime, which isn't true. This
> drags util-linux-mount into any image that uses fuse.
> 
> Encountered no problems with busybox's mount command and fuse (and never
> had). Fuse doesn't call the "mount" program anywhere, so the dependency
> doesn't make sense anyway.

The RDEPEND was added by Jackie from Wind River in September 2015. Any
idea why it was needed then?

Philip

> 
> Signed-off-by: Mike Looijmans <mike.looijmans@topic.nl>
> ---
>  meta-filesystems/recipes-support/fuse/fuse_2.9.4.bb | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/meta-filesystems/recipes-support/fuse/fuse_2.9.4.bb b/meta-filesystems/recipes-support/fuse/fuse_2.9.4.bb
> index 9a2dc11..153fcb4 100644
> --- a/meta-filesystems/recipes-support/fuse/fuse_2.9.4.bb
> +++ b/meta-filesystems/recipes-support/fuse/fuse_2.9.4.bb
> @@ -29,9 +29,6 @@ DEPENDS = "gettext-native"
>  
>  PACKAGES =+ "fuse-utils-dbg fuse-utils libulockmgr libulockmgr-dev libulockmgr-dbg"
>  
> -# Fusermount requires features from the util-linux version of mount.
> -RDEPENDS_${PN}_class-target += "util-linux-mount"
> -
>  RRECOMMENDS_${PN}_class-target = "kernel-module-fuse libulockmgr fuse-utils"
>  
>  FILES_${PN} += "${libdir}/libfuse.so.*"
> 


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-02-28 14:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-26 11:33 [PATCH][meta-filesystems] fuse: Remove unneeded RDEPENDS on util-linux-mount Mike Looijmans
2017-02-28 14:37 ` Philip Balister

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.