* [PATCH v2] udev-extraconf: update mount.sh to use /run/media instead of /media
@ 2014-04-01 3:54 Denys Dmytriyenko
2014-04-06 22:46 ` Denys Dmytriyenko
0 siblings, 1 reply; 6+ messages in thread
From: Denys Dmytriyenko @ 2014-04-01 3:54 UTC (permalink / raw)
To: openembedded-core; +Cc: Denys Dmytriyenko
From: Denys Dmytriyenko <denys@ti.com>
This is done to work around the issue of auto-mounting block devices
(i.e. SD cards) when root filesystem is still in read-only mode and
creating /media/<device> mount-points by udev is not possible. That
is due to udev (/etc/rcS.d/S03udev) getting started earlier than
checkroot (/etc/rcS.d/S10checkroot.sh) gets a chance to re-mount the
rootfs as read-write.
Although, canonical FHS specifies /media/<device> as a mount point
for removable media devices, the latest 2.3 version was released in
2004 and since then FreeDesktop/udisks and other tools adopted the
new /run/media/<user>/<device> location. That was done to overcome
read-only rootfs limitation, since /run is usually a tmpfs mounted
partition, plus avoid name-clash between users.
For our embedded systems environment we assume single-user operation
and hence simplify mount point to just /run/media/<device>. But for
proper per-user mounting to /run/media/<user>/<device>, some sort of
session management is required along with the tool like udisks, that
is out of scope of this simple udev-based auto-mounting.
Signed-off-by: Denys Dmytriyenko <denys@ti.com>
---
v2 - drop PR, bump PV, elaborate on session/user mounting option
| 14 +++++++-------
| 2 --
2 files changed, 7 insertions(+), 9 deletions(-)
rename meta/recipes-core/udev/{udev-extraconf_1.0.bb => udev-extraconf_1.1.bb} (99%)
--git a/meta/recipes-core/udev/udev-extraconf/mount.sh b/meta/recipes-core/udev/udev-extraconf/mount.sh
index cb57e47..3e4f21f 100644
--- a/meta/recipes-core/udev/udev-extraconf/mount.sh
+++ b/meta/recipes-core/udev/udev-extraconf/mount.sh
@@ -20,7 +20,7 @@ done
automount() {
name="`basename "$DEVNAME"`"
- ! test -d "/media/$name" && mkdir -p "/media/$name"
+ ! test -d "/run/media/$name" && mkdir -p "/run/media/$name"
# Silent util-linux's version of mounting auto
if [ "x`readlink $MOUNT`" = "x/bin/mount.util-linux" ] ;
then
@@ -38,12 +38,12 @@ automount() {
;;
esac
- if ! $MOUNT -t auto $DEVNAME "/media/$name"
+ if ! $MOUNT -t auto $DEVNAME "/run/media/$name"
then
- #logger "mount.sh/automount" "$MOUNT -t auto $DEVNAME \"/media/$name\" failed!"
- rm_dir "/media/$name"
+ #logger "mount.sh/automount" "$MOUNT -t auto $DEVNAME \"/run/media/$name\" failed!"
+ rm_dir "/run/media/$name"
else
- logger "mount.sh/automount" "Auto-mount of [/media/$name] successful"
+ logger "mount.sh/automount" "Auto-mount of [/run/media/$name] successful"
touch "/tmp/.automount-$name"
fi
}
@@ -60,7 +60,7 @@ rm_dir() {
# No ID_FS_TYPE for cdrom device, yet it should be mounted
name="`basename "$DEVNAME"`"
-[ -e /sys/block/$name/device/media ] && media_type=`cat /sys/block/$name/device/media`
+[ -e /sys/block/$name/device/run/media ] && media_type=`cat /sys/block/$name/device/run/media`
if [ "$ACTION" = "add" ] && [ -n "$DEVNAME" ] && [ -n "$ID_FS_TYPE" -o "$media_type" = "cdrom" ]; then
if [ -x "$PMOUNT" ]; then
@@ -87,5 +87,5 @@ if [ "$ACTION" = "remove" ] && [ -x "$UMOUNT" ] && [ -n "$DEVNAME" ]; then
# Remove empty directories from auto-mounter
name="`basename "$DEVNAME"`"
- test -e "/tmp/.automount-$name" && rm_dir "/media/$name"
+ test -e "/tmp/.automount-$name" && rm_dir "/run/media/$name"
fi
--git a/meta/recipes-core/udev/udev-extraconf_1.0.bb b/meta/recipes-core/udev/udev-extraconf_1.1.bb
similarity index 99%
rename from meta/recipes-core/udev/udev-extraconf_1.0.bb
rename to meta/recipes-core/udev/udev-extraconf_1.1.bb
index 3810b28..d69056d 100644
--- a/meta/recipes-core/udev/udev-extraconf_1.0.bb
+++ b/meta/recipes-core/udev/udev-extraconf_1.1.bb
@@ -4,8 +4,6 @@ LICENSE = "MIT"
LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=4d92cd373abda3937c2bc47fbc49d690 \
file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
-PR = "r16"
-
SRC_URI = " \
file://automount.rules \
file://mount.sh \
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH v2] udev-extraconf: update mount.sh to use /run/media instead of /media
2014-04-01 3:54 [PATCH v2] udev-extraconf: update mount.sh to use /run/media instead of /media Denys Dmytriyenko
@ 2014-04-06 22:46 ` Denys Dmytriyenko
2014-04-07 15:49 ` Saul Wold
0 siblings, 1 reply; 6+ messages in thread
From: Denys Dmytriyenko @ 2014-04-06 22:46 UTC (permalink / raw)
To: openembedded-core
ping
On Mon, Mar 31, 2014 at 11:54:11PM -0400, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@ti.com>
>
> This is done to work around the issue of auto-mounting block devices
> (i.e. SD cards) when root filesystem is still in read-only mode and
> creating /media/<device> mount-points by udev is not possible. That
> is due to udev (/etc/rcS.d/S03udev) getting started earlier than
> checkroot (/etc/rcS.d/S10checkroot.sh) gets a chance to re-mount the
> rootfs as read-write.
>
> Although, canonical FHS specifies /media/<device> as a mount point
> for removable media devices, the latest 2.3 version was released in
> 2004 and since then FreeDesktop/udisks and other tools adopted the
> new /run/media/<user>/<device> location. That was done to overcome
> read-only rootfs limitation, since /run is usually a tmpfs mounted
> partition, plus avoid name-clash between users.
>
> For our embedded systems environment we assume single-user operation
> and hence simplify mount point to just /run/media/<device>. But for
> proper per-user mounting to /run/media/<user>/<device>, some sort of
> session management is required along with the tool like udisks, that
> is out of scope of this simple udev-based auto-mounting.
>
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
> v2 - drop PR, bump PV, elaborate on session/user mounting option
>
> meta/recipes-core/udev/udev-extraconf/mount.sh | 14 +++++++-------
> .../udev/{udev-extraconf_1.0.bb => udev-extraconf_1.1.bb} | 2 --
> 2 files changed, 7 insertions(+), 9 deletions(-)
> rename meta/recipes-core/udev/{udev-extraconf_1.0.bb => udev-extraconf_1.1.bb} (99%)
>
> diff --git a/meta/recipes-core/udev/udev-extraconf/mount.sh b/meta/recipes-core/udev/udev-extraconf/mount.sh
> index cb57e47..3e4f21f 100644
> --- a/meta/recipes-core/udev/udev-extraconf/mount.sh
> +++ b/meta/recipes-core/udev/udev-extraconf/mount.sh
> @@ -20,7 +20,7 @@ done
> automount() {
> name="`basename "$DEVNAME"`"
>
> - ! test -d "/media/$name" && mkdir -p "/media/$name"
> + ! test -d "/run/media/$name" && mkdir -p "/run/media/$name"
> # Silent util-linux's version of mounting auto
> if [ "x`readlink $MOUNT`" = "x/bin/mount.util-linux" ] ;
> then
> @@ -38,12 +38,12 @@ automount() {
> ;;
> esac
>
> - if ! $MOUNT -t auto $DEVNAME "/media/$name"
> + if ! $MOUNT -t auto $DEVNAME "/run/media/$name"
> then
> - #logger "mount.sh/automount" "$MOUNT -t auto $DEVNAME \"/media/$name\" failed!"
> - rm_dir "/media/$name"
> + #logger "mount.sh/automount" "$MOUNT -t auto $DEVNAME \"/run/media/$name\" failed!"
> + rm_dir "/run/media/$name"
> else
> - logger "mount.sh/automount" "Auto-mount of [/media/$name] successful"
> + logger "mount.sh/automount" "Auto-mount of [/run/media/$name] successful"
> touch "/tmp/.automount-$name"
> fi
> }
> @@ -60,7 +60,7 @@ rm_dir() {
>
> # No ID_FS_TYPE for cdrom device, yet it should be mounted
> name="`basename "$DEVNAME"`"
> -[ -e /sys/block/$name/device/media ] && media_type=`cat /sys/block/$name/device/media`
> +[ -e /sys/block/$name/device/run/media ] && media_type=`cat /sys/block/$name/device/run/media`
>
> if [ "$ACTION" = "add" ] && [ -n "$DEVNAME" ] && [ -n "$ID_FS_TYPE" -o "$media_type" = "cdrom" ]; then
> if [ -x "$PMOUNT" ]; then
> @@ -87,5 +87,5 @@ if [ "$ACTION" = "remove" ] && [ -x "$UMOUNT" ] && [ -n "$DEVNAME" ]; then
>
> # Remove empty directories from auto-mounter
> name="`basename "$DEVNAME"`"
> - test -e "/tmp/.automount-$name" && rm_dir "/media/$name"
> + test -e "/tmp/.automount-$name" && rm_dir "/run/media/$name"
> fi
> diff --git a/meta/recipes-core/udev/udev-extraconf_1.0.bb b/meta/recipes-core/udev/udev-extraconf_1.1.bb
> similarity index 99%
> rename from meta/recipes-core/udev/udev-extraconf_1.0.bb
> rename to meta/recipes-core/udev/udev-extraconf_1.1.bb
> index 3810b28..d69056d 100644
> --- a/meta/recipes-core/udev/udev-extraconf_1.0.bb
> +++ b/meta/recipes-core/udev/udev-extraconf_1.1.bb
> @@ -4,8 +4,6 @@ LICENSE = "MIT"
> LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=4d92cd373abda3937c2bc47fbc49d690 \
> file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
>
> -PR = "r16"
> -
> SRC_URI = " \
> file://automount.rules \
> file://mount.sh \
> --
> 1.9.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2] udev-extraconf: update mount.sh to use /run/media instead of /media
2014-04-06 22:46 ` Denys Dmytriyenko
@ 2014-04-07 15:49 ` Saul Wold
2014-04-07 21:05 ` Denys Dmytriyenko
0 siblings, 1 reply; 6+ messages in thread
From: Saul Wold @ 2014-04-07 15:49 UTC (permalink / raw)
To: Denys Dmytriyenko, openembedded-core
On 04/06/2014 03:46 PM, Denys Dmytriyenko wrote:
> ping
>
This change is a little late for 1.6 as we are prepping for the final RC
this week. It's kind of an API change since it moves the mount location
/media, should we be linking /media to /run/media?
There is also an issue with the Freescale PPC BSP, which has a bbappend
for 1.0, so some coordination with them will be needed.
I can revisit this for 1.7 in a week or so.
Sau!
> On Mon, Mar 31, 2014 at 11:54:11PM -0400, Denys Dmytriyenko wrote:
>> From: Denys Dmytriyenko <denys@ti.com>
>>
>> This is done to work around the issue of auto-mounting block devices
>> (i.e. SD cards) when root filesystem is still in read-only mode and
>> creating /media/<device> mount-points by udev is not possible. That
>> is due to udev (/etc/rcS.d/S03udev) getting started earlier than
>> checkroot (/etc/rcS.d/S10checkroot.sh) gets a chance to re-mount the
>> rootfs as read-write.
>>
>> Although, canonical FHS specifies /media/<device> as a mount point
>> for removable media devices, the latest 2.3 version was released in
>> 2004 and since then FreeDesktop/udisks and other tools adopted the
>> new /run/media/<user>/<device> location. That was done to overcome
>> read-only rootfs limitation, since /run is usually a tmpfs mounted
>> partition, plus avoid name-clash between users.
>>
>> For our embedded systems environment we assume single-user operation
>> and hence simplify mount point to just /run/media/<device>. But for
>> proper per-user mounting to /run/media/<user>/<device>, some sort of
>> session management is required along with the tool like udisks, that
>> is out of scope of this simple udev-based auto-mounting.
>>
>> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>> ---
>> v2 - drop PR, bump PV, elaborate on session/user mounting option
>>
>> meta/recipes-core/udev/udev-extraconf/mount.sh | 14 +++++++-------
>> .../udev/{udev-extraconf_1.0.bb => udev-extraconf_1.1.bb} | 2 --
>> 2 files changed, 7 insertions(+), 9 deletions(-)
>> rename meta/recipes-core/udev/{udev-extraconf_1.0.bb => udev-extraconf_1.1.bb} (99%)
>>
>> diff --git a/meta/recipes-core/udev/udev-extraconf/mount.sh b/meta/recipes-core/udev/udev-extraconf/mount.sh
>> index cb57e47..3e4f21f 100644
>> --- a/meta/recipes-core/udev/udev-extraconf/mount.sh
>> +++ b/meta/recipes-core/udev/udev-extraconf/mount.sh
>> @@ -20,7 +20,7 @@ done
>> automount() {
>> name="`basename "$DEVNAME"`"
>>
>> - ! test -d "/media/$name" && mkdir -p "/media/$name"
>> + ! test -d "/run/media/$name" && mkdir -p "/run/media/$name"
>> # Silent util-linux's version of mounting auto
>> if [ "x`readlink $MOUNT`" = "x/bin/mount.util-linux" ] ;
>> then
>> @@ -38,12 +38,12 @@ automount() {
>> ;;
>> esac
>>
>> - if ! $MOUNT -t auto $DEVNAME "/media/$name"
>> + if ! $MOUNT -t auto $DEVNAME "/run/media/$name"
>> then
>> - #logger "mount.sh/automount" "$MOUNT -t auto $DEVNAME \"/media/$name\" failed!"
>> - rm_dir "/media/$name"
>> + #logger "mount.sh/automount" "$MOUNT -t auto $DEVNAME \"/run/media/$name\" failed!"
>> + rm_dir "/run/media/$name"
>> else
>> - logger "mount.sh/automount" "Auto-mount of [/media/$name] successful"
>> + logger "mount.sh/automount" "Auto-mount of [/run/media/$name] successful"
>> touch "/tmp/.automount-$name"
>> fi
>> }
>> @@ -60,7 +60,7 @@ rm_dir() {
>>
>> # No ID_FS_TYPE for cdrom device, yet it should be mounted
>> name="`basename "$DEVNAME"`"
>> -[ -e /sys/block/$name/device/media ] && media_type=`cat /sys/block/$name/device/media`
>> +[ -e /sys/block/$name/device/run/media ] && media_type=`cat /sys/block/$name/device/run/media`
>>
>> if [ "$ACTION" = "add" ] && [ -n "$DEVNAME" ] && [ -n "$ID_FS_TYPE" -o "$media_type" = "cdrom" ]; then
>> if [ -x "$PMOUNT" ]; then
>> @@ -87,5 +87,5 @@ if [ "$ACTION" = "remove" ] && [ -x "$UMOUNT" ] && [ -n "$DEVNAME" ]; then
>>
>> # Remove empty directories from auto-mounter
>> name="`basename "$DEVNAME"`"
>> - test -e "/tmp/.automount-$name" && rm_dir "/media/$name"
>> + test -e "/tmp/.automount-$name" && rm_dir "/run/media/$name"
>> fi
>> diff --git a/meta/recipes-core/udev/udev-extraconf_1.0.bb b/meta/recipes-core/udev/udev-extraconf_1.1.bb
>> similarity index 99%
>> rename from meta/recipes-core/udev/udev-extraconf_1.0.bb
>> rename to meta/recipes-core/udev/udev-extraconf_1.1.bb
>> index 3810b28..d69056d 100644
>> --- a/meta/recipes-core/udev/udev-extraconf_1.0.bb
>> +++ b/meta/recipes-core/udev/udev-extraconf_1.1.bb
>> @@ -4,8 +4,6 @@ LICENSE = "MIT"
>> LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=4d92cd373abda3937c2bc47fbc49d690 \
>> file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
>>
>> -PR = "r16"
>> -
>> SRC_URI = " \
>> file://automount.rules \
>> file://mount.sh \
>> --
>> 1.9.1
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2] udev-extraconf: update mount.sh to use /run/media instead of /media
2014-04-07 15:49 ` Saul Wold
@ 2014-04-07 21:05 ` Denys Dmytriyenko
2014-04-07 22:22 ` Richard Purdie
0 siblings, 1 reply; 6+ messages in thread
From: Denys Dmytriyenko @ 2014-04-07 21:05 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
On Mon, Apr 07, 2014 at 08:49:44AM -0700, Saul Wold wrote:
> On 04/06/2014 03:46 PM, Denys Dmytriyenko wrote:
> >ping
> >
>
> This change is a little late for 1.6 as we are prepping for the
> final RC this week. It's kind of an API change since it moves the
> mount location /media, should we be linking /media to /run/media?
>
> There is also an issue with the Freescale PPC BSP, which has a
> bbappend for 1.0, so some coordination with them will be needed.
>
> I can revisit this for 1.7 in a week or so.
>
> Sau!
I understand the timing concern and am fine waiting another week for when 1.7
window opens.
But I'm slightly confused as to since when OE-Core is concerned about breaking
bbappends in other layers? I guess some layers are not created equal, eh? :)
--
Denys
> >On Mon, Mar 31, 2014 at 11:54:11PM -0400, Denys Dmytriyenko wrote:
> >>From: Denys Dmytriyenko <denys@ti.com>
> >>
> >>This is done to work around the issue of auto-mounting block devices
> >>(i.e. SD cards) when root filesystem is still in read-only mode and
> >>creating /media/<device> mount-points by udev is not possible. That
> >>is due to udev (/etc/rcS.d/S03udev) getting started earlier than
> >>checkroot (/etc/rcS.d/S10checkroot.sh) gets a chance to re-mount the
> >>rootfs as read-write.
> >>
> >>Although, canonical FHS specifies /media/<device> as a mount point
> >>for removable media devices, the latest 2.3 version was released in
> >>2004 and since then FreeDesktop/udisks and other tools adopted the
> >>new /run/media/<user>/<device> location. That was done to overcome
> >>read-only rootfs limitation, since /run is usually a tmpfs mounted
> >>partition, plus avoid name-clash between users.
> >>
> >>For our embedded systems environment we assume single-user operation
> >>and hence simplify mount point to just /run/media/<device>. But for
> >>proper per-user mounting to /run/media/<user>/<device>, some sort of
> >>session management is required along with the tool like udisks, that
> >>is out of scope of this simple udev-based auto-mounting.
> >>
> >>Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> >>---
> >>v2 - drop PR, bump PV, elaborate on session/user mounting option
> >>
> >> meta/recipes-core/udev/udev-extraconf/mount.sh | 14 +++++++-------
> >> .../udev/{udev-extraconf_1.0.bb => udev-extraconf_1.1.bb} | 2 --
> >> 2 files changed, 7 insertions(+), 9 deletions(-)
> >> rename meta/recipes-core/udev/{udev-extraconf_1.0.bb => udev-extraconf_1.1.bb} (99%)
> >>
> >>diff --git a/meta/recipes-core/udev/udev-extraconf/mount.sh b/meta/recipes-core/udev/udev-extraconf/mount.sh
> >>index cb57e47..3e4f21f 100644
> >>--- a/meta/recipes-core/udev/udev-extraconf/mount.sh
> >>+++ b/meta/recipes-core/udev/udev-extraconf/mount.sh
> >>@@ -20,7 +20,7 @@ done
> >> automount() {
> >> name="`basename "$DEVNAME"`"
> >>
> >>- ! test -d "/media/$name" && mkdir -p "/media/$name"
> >>+ ! test -d "/run/media/$name" && mkdir -p "/run/media/$name"
> >> # Silent util-linux's version of mounting auto
> >> if [ "x`readlink $MOUNT`" = "x/bin/mount.util-linux" ] ;
> >> then
> >>@@ -38,12 +38,12 @@ automount() {
> >> ;;
> >> esac
> >>
> >>- if ! $MOUNT -t auto $DEVNAME "/media/$name"
> >>+ if ! $MOUNT -t auto $DEVNAME "/run/media/$name"
> >> then
> >>- #logger "mount.sh/automount" "$MOUNT -t auto $DEVNAME \"/media/$name\" failed!"
> >>- rm_dir "/media/$name"
> >>+ #logger "mount.sh/automount" "$MOUNT -t auto $DEVNAME \"/run/media/$name\" failed!"
> >>+ rm_dir "/run/media/$name"
> >> else
> >>- logger "mount.sh/automount" "Auto-mount of [/media/$name] successful"
> >>+ logger "mount.sh/automount" "Auto-mount of [/run/media/$name] successful"
> >> touch "/tmp/.automount-$name"
> >> fi
> >> }
> >>@@ -60,7 +60,7 @@ rm_dir() {
> >>
> >> # No ID_FS_TYPE for cdrom device, yet it should be mounted
> >> name="`basename "$DEVNAME"`"
> >>-[ -e /sys/block/$name/device/media ] && media_type=`cat /sys/block/$name/device/media`
> >>+[ -e /sys/block/$name/device/run/media ] && media_type=`cat /sys/block/$name/device/run/media`
> >>
> >> if [ "$ACTION" = "add" ] && [ -n "$DEVNAME" ] && [ -n "$ID_FS_TYPE" -o "$media_type" = "cdrom" ]; then
> >> if [ -x "$PMOUNT" ]; then
> >>@@ -87,5 +87,5 @@ if [ "$ACTION" = "remove" ] && [ -x "$UMOUNT" ] && [ -n "$DEVNAME" ]; then
> >>
> >> # Remove empty directories from auto-mounter
> >> name="`basename "$DEVNAME"`"
> >>- test -e "/tmp/.automount-$name" && rm_dir "/media/$name"
> >>+ test -e "/tmp/.automount-$name" && rm_dir "/run/media/$name"
> >> fi
> >>diff --git a/meta/recipes-core/udev/udev-extraconf_1.0.bb b/meta/recipes-core/udev/udev-extraconf_1.1.bb
> >>similarity index 99%
> >>rename from meta/recipes-core/udev/udev-extraconf_1.0.bb
> >>rename to meta/recipes-core/udev/udev-extraconf_1.1.bb
> >>index 3810b28..d69056d 100644
> >>--- a/meta/recipes-core/udev/udev-extraconf_1.0.bb
> >>+++ b/meta/recipes-core/udev/udev-extraconf_1.1.bb
> >>@@ -4,8 +4,6 @@ LICENSE = "MIT"
> >> LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=4d92cd373abda3937c2bc47fbc49d690 \
> >> file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
> >>
> >>-PR = "r16"
> >>-
> >> SRC_URI = " \
> >> file://automount.rules \
> >> file://mount.sh \
> >>--
> >>1.9.1
> >>
> >>--
> >>_______________________________________________
> >>Openembedded-core mailing list
> >>Openembedded-core@lists.openembedded.org
> >>http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2] udev-extraconf: update mount.sh to use /run/media instead of /media
2014-04-07 21:05 ` Denys Dmytriyenko
@ 2014-04-07 22:22 ` Richard Purdie
2014-04-21 17:26 ` Denys Dmytriyenko
0 siblings, 1 reply; 6+ messages in thread
From: Richard Purdie @ 2014-04-07 22:22 UTC (permalink / raw)
To: Denys Dmytriyenko; +Cc: openembedded-core
On Mon, 2014-04-07 at 17:05 -0400, Denys Dmytriyenko wrote:
> I understand the timing concern and am fine waiting another week for when 1.7
> window opens.
>
> But I'm slightly confused as to since when OE-Core is concerned about breaking
> bbappends in other layers? I guess some layers are not created equal, eh? :)
The question is more one of timing. I don't think its reasonable that we
break things like that between -rc2 and -rc3 of 1.6 without a better
reason...
I'm even nervous changing the sstate checksums but I think the CONFFILES
change does warrant that for example.
Cheers,
Richard
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] udev-extraconf: update mount.sh to use /run/media instead of /media
2014-04-07 22:22 ` Richard Purdie
@ 2014-04-21 17:26 ` Denys Dmytriyenko
0 siblings, 0 replies; 6+ messages in thread
From: Denys Dmytriyenko @ 2014-04-21 17:26 UTC (permalink / raw)
To: Richard Purdie; +Cc: openembedded-core
On Mon, Apr 07, 2014 at 11:22:11PM +0100, Richard Purdie wrote:
> On Mon, 2014-04-07 at 17:05 -0400, Denys Dmytriyenko wrote:
> > I understand the timing concern and am fine waiting another week for when 1.7
> > window opens.
> >
> > But I'm slightly confused as to since when OE-Core is concerned about breaking
> > bbappends in other layers? I guess some layers are not created equal, eh? :)
>
> The question is more one of timing. I don't think its reasonable that we
> break things like that between -rc2 and -rc3 of 1.6 without a better
> reason...
>
> I'm even nervous changing the sstate checksums but I think the CONFFILES
> change does warrant that for example.
Is 1.7 window now open? Should I resubmit this patch again? Thanks.
--
Denys
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-04-21 17:27 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-01 3:54 [PATCH v2] udev-extraconf: update mount.sh to use /run/media instead of /media Denys Dmytriyenko
2014-04-06 22:46 ` Denys Dmytriyenko
2014-04-07 15:49 ` Saul Wold
2014-04-07 21:05 ` Denys Dmytriyenko
2014-04-07 22:22 ` Richard Purdie
2014-04-21 17:26 ` Denys Dmytriyenko
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox