On 3/19/2024 10:24 AM, Ryan Eatmon via lists.yoctoproject.org wrote:
On 3/19/2024 10:18 AM, Andrew Davis wrote:
On 3/5/24 2:49 PM, Denys Dmytriyenko wrote:
On Tue, Mar 05, 2024 at 01:11:45PM -0600, Andrew Davis via
lists.yoctoproject.org wrote:
There is already a version of this file provided by oe-core that
is kept up to date. Do not override it.
Signed-off-by: Andrew Davis <[email protected]>
---
.../recipes-core/base-files/base-files/fstab | 13
-------------
.../recipes-core/base-files/base-files_%.bbappend | 2 +-
2 files changed, 1 insertion(+), 14 deletions(-)
delete mode 100644
meta-arago-distro/recipes-core/base-files/base-files/fstab
diff --git
a/meta-arago-distro/recipes-core/base-files/base-files/fstab
b/meta-arago-distro/recipes-core/base-files/base-files/fstab
deleted file mode 100644
index 8ddadd98..00000000
--- a/meta-arago-distro/recipes-core/base-files/base-files/fstab
+++ /dev/null
@@ -1,13 +0,0 @@
-# stock fstab - you probably want to override this with a machine
specific one
-
-/dev/root / auto
defaults 1 1
-proc /proc proc
defaults 0 0
-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,size=50M 0 0
-tmpfs /media/ram tmpfs
defaults,size=16M 0 0
The major difference is these 2 tmpfs entries - upstream doesn't have
/media/ram, as that was something TI testing framework was using. Plus
upstream doesn't set the size parameter - by default up to 50% of RAM
can
get used by tmpfs and there were issues with some low-RAM legacy
platforms
in the past.
If neither of these differences matter any longer, this can be removed.
If the tests do need this, then they should mount the needed tmpfs as
part
of the test setup, not just assume it to be setup by the distro.
Simple as:
# mount -t tmpfs -o size=50M tmpfs /media/ram
The only way to push the test framework to do the right thing is to
stop doing it for it.
In general I agree, but that is easier said than done. We need to
coordinate this change with the test team, not just dump it on them. I'm
not rejecting this patch, just waiting until test has confirmed we no
longer need it.
Good news. We ran a special set of RC tests with this change in place
and everything looks like fine. I'll accept this patch now for
scarthgap/master.
Andrew
-# 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-arago-distro/recipes-core/base-files/base-files_%.bbappend
b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend
index e5c53e80..7d1f6751 100644
--- a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend
+++ b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend
@@ -1,4 +1,4 @@
# look for files in this layer first
FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:"
-PR:append = ".arago4"
+PR:append = ".arago5"
--
2.39.2
--
Ryan Eatmon [email protected]
-----------------------------------------
Texas Instruments, Inc. - LCPD - MGTS
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#15259):
https://lists.yoctoproject.org/g/meta-arago/message/15259
Mute This Topic: https://lists.yoctoproject.org/mt/104751776/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
