All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/1] base-files: media directory cleanup
@ 2013-06-28  7:48 Qi.Chen
  2013-06-28  7:48 ` [PATCH 1/1] base-files: remove the unnecessary /media/xxx directories Qi.Chen
  0 siblings, 1 reply; 3+ messages in thread
From: Qi.Chen @ 2013-06-28  7:48 UTC (permalink / raw)
  To: openembedded-core; +Cc: qingtao.cao

From: Chen Qi <Qi.Chen@windriver.com>

The following changes since commit 10e44f162c8ac71965faaeb70b8bfdeb264ba76c:

  bitbake: bitbake: python funcname can not include special character @ (2013-06-27 10:06:09 +0100)

are available in the git repository at:

  git://git.pokylinux.org/poky-contrib ChenQi/media-dir-cleanup
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/media-dir-cleanup

Chen Qi (1):
  base-files: remove the unnecessary /media/xxx directories

 meta/recipes-core/base-files/base-files/fstab     |    1 -
 meta/recipes-core/base-files/base-files_3.0.14.bb |    7 +------
 2 files changed, 1 insertion(+), 7 deletions(-)

-- 
1.7.9.5



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

* [PATCH 1/1] base-files: remove the unnecessary /media/xxx directories
  2013-06-28  7:48 [PATCH 0/1] base-files: media directory cleanup Qi.Chen
@ 2013-06-28  7:48 ` Qi.Chen
  2013-06-28  9:48   ` Richard Purdie
  0 siblings, 1 reply; 3+ messages in thread
From: Qi.Chen @ 2013-06-28  7:48 UTC (permalink / raw)
  To: openembedded-core; +Cc: qingtao.cao

From: Chen Qi <Qi.Chen@windriver.com>

A few directories under /media were created by default, /media/card,
/media/ram, /media/realroot, etc.

These directories actually have no real usage now, thus removing them.

The /media/ram entry in the fstab is also removed, as mounting a tmpfs
over /media/ram in our system brings no benefit.

Note that a duplicate '/mnt' entry in dirs755 are also removed.

[Yocto #4774]

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
 meta/recipes-core/base-files/base-files/fstab     |    1 -
 meta/recipes-core/base-files/base-files_3.0.14.bb |    7 +------
 2 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/meta/recipes-core/base-files/base-files/fstab b/meta/recipes-core/base-files/base-files/fstab
index dd613a1..e625ebc 100644
--- a/meta/recipes-core/base-files/base-files/fstab
+++ b/meta/recipes-core/base-files/base-files/fstab
@@ -6,7 +6,6 @@ devpts               /dev/pts             devpts     mode=0620,gid=5       0  0
 usbdevfs             /proc/bus/usb        usbdevfs   noauto                0  0
 tmpfs                /run                 tmpfs      mode=0755,nodev,nosuid,strictatime 0  0
 tmpfs                /var/volatile        tmpfs      defaults              0  0
-tmpfs                /media/ram           tmpfs      defaults              0  0
 
 # uncomment this if your device has a SD/MMC/Transflash slot
 #/dev/mmcblk0p1       /media/card          auto       defaults,sync,noauto  0  0
diff --git a/meta/recipes-core/base-files/base-files_3.0.14.bb b/meta/recipes-core/base-files/base-files_3.0.14.bb
index 88d1ac9..054fefa 100644
--- a/meta/recipes-core/base-files/base-files_3.0.14.bb
+++ b/meta/recipes-core/base-files/base-files_3.0.14.bb
@@ -43,9 +43,7 @@ dirs755 = "/bin /boot /dev ${sysconfdir} ${sysconfdir}/default \
            /sys ${localstatedir}/lib/misc ${localstatedir}/spool \
            ${localstatedir}/volatile \
            ${localstatedir}/volatile/log \
-           /mnt /media /media/card /media/cf /media/net /media/ram \
-           /media/union /media/realroot /media/hdd \
-           /media/mmc1"
+           /media"
 dirs3755 = "/srv  \
             ${prefix}/local ${prefix}/local/bin ${prefix}/local/games \
             ${prefix}/local/include ${prefix}/local/lib ${prefix}/local/sbin \
@@ -79,9 +77,6 @@ do_install () {
 	for d in ${volatiles}; do
 		ln -sf volatile/$d ${D}${localstatedir}/$d
 	done
-	for d in card cf net ram; do
-		ln -sf /media/$d ${D}/mnt/$d
-	done
 	ln -snf ../run ${D}${localstatedir}/run
 	ln -snf ../run/lock ${D}${localstatedir}/lock
 
-- 
1.7.9.5



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

* Re: [PATCH 1/1] base-files: remove the unnecessary /media/xxx directories
  2013-06-28  7:48 ` [PATCH 1/1] base-files: remove the unnecessary /media/xxx directories Qi.Chen
@ 2013-06-28  9:48   ` Richard Purdie
  0 siblings, 0 replies; 3+ messages in thread
From: Richard Purdie @ 2013-06-28  9:48 UTC (permalink / raw)
  To: Qi.Chen; +Cc: qingtao.cao, openembedded-core

On Fri, 2013-06-28 at 15:48 +0800, Qi.Chen@windriver.com wrote:
> From: Chen Qi <Qi.Chen@windriver.com>
> 
> A few directories under /media were created by default, /media/card,
> /media/ram, /media/realroot, etc.
> 
> These directories actually have no real usage now, thus removing them.
> 
> The /media/ram entry in the fstab is also removed, as mounting a tmpfs
> over /media/ram in our system brings no benefit.
> 
> Note that a duplicate '/mnt' entry in dirs755 are also removed.
> 
> [Yocto #4774]
> 
> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>

Just to be clear, most of these date around 10 years from OpenZaurus and
similar times. We now automount media and there is no real need for
these specific directories or the ramfs. If there is some need in legacy
systems (I did wonder about opie), I'd suggest they get created by them
rather than the core base-files.

Cheers,

Richard



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

end of thread, other threads:[~2013-06-28  9:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-28  7:48 [PATCH 0/1] base-files: media directory cleanup Qi.Chen
2013-06-28  7:48 ` [PATCH 1/1] base-files: remove the unnecessary /media/xxx directories Qi.Chen
2013-06-28  9:48   ` Richard Purdie

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.