Bug 15120 - OVERLAYFS_WRITABLE_PATHS missing as vardeps for create_overlayfs_units
Summary: OVERLAYFS_WRITABLE_PATHS missing as vardeps for create_overlayfs_units
Status: RESOLVED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: core (show other bugs)
Version: 4.0
Hardware: x86 Multiple
: Medium normal
Target Milestone: 5.0
Assignee: Christoph Vogtlaender
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2023-05-11 15:50 UTC by Christoph Vogtlaender
Modified: 2024-05-23 15:08 UTC (History)
7 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Christoph Vogtlaender 2023-05-11 15:50:06 UTC
Changing OVERLAYFS_WRITABLE_PATHS in a recipe does not trigger a rebuild of the package.

e.g. given there are the definitions
> OVERLAYFS_MOUNT_POINT[app] = "/app"
> OVERLAYFS_MOUNT_POINT[data] = "/data"
in MACHINE configuration and a recipe which uses
> OVERLAYFS_WRITABLE_PATHS[app] = "/etc/myapp"
changing the overlay to "data"
> OVERLAYFS_WRITABLE_PATHS[data] = "/etc/myapp"
does not trigger a rebuild of the package.

It seems that OVERLAYFS_WRITABLE_PATHS is not considered as a recipe dependency. Adding
> do_create_overlayfs_units[vardeps] += "OVERLAYFS_WRITABLE_PATHS"
to the recipe works around the issue.

It should be considered to add
> do_create_overlayfs_units[vardeps] += "OVERLAYFS_WRITABLE_PATHS"
to meta/classes/overlayfs.bbclass
Comment 1 Randy MacLeod 2023-05-18 14:37:25 UTC
Makes sense, can you send a patch to the list?
Comment 2 Christoph Vogtlaender 2023-05-19 17:00:40 UTC
(In reply to Randy MacLeod from comment #1)
> Makes sense, can you send a patch to the list?

Yes, I will send a patch
Comment 3 Randy MacLeod 2024-01-18 16:20:32 UTC
Chrostoph, are you still planning to send a patch?
Comment 4 Christoph Vogtlaender 2024-02-20 10:49:42 UTC
Yes, I will send a patch. Sorry for the delay.
Comment 5 Christoph Vogtlaender 2024-02-20 11:52:07 UTC
Patch send to oe-core
Comment 6 Stephen K Jolley 2024-02-26 01:55:06 UTC
Patch Sent