{"action":"created","issue":{"url":"https://api.github.com/repos/advancedtelematic/meta-updater/issues/446","repository_url":"https://api.github.com/repos/advancedtelematic/meta-updater","labels_url":"https://api.github.com/repos/advancedtelematic/meta-updater/issues/446/labels{/name}","comments_url":"https://api.github.com/repos/advancedtelematic/meta-updater/issues/446/comments","events_url":"https://api.github.com/repos/advancedtelematic/meta-updater/issues/446/events","html_url":"https://github.com/advancedtelematic/meta-updater/pull/446","id":390269118,"node_id":"MDExOlB1bGxSZXF1ZXN0MjM4MDcxNDc0","number":446,"title":"Make INITRAMFS_FSTYPES an override","user":{"login":"OYTIS","id":1929183,"node_id":"MDQ6VXNlcjE5MjkxODM=","avatar_url":"https://avatars1.githubusercontent.com/u/1929183?v=4","gravatar_id":"","url":"https://api.github.com/users/OYTIS","html_url":"https://github.com/OYTIS","followers_url":"https://api.github.com/users/OYTIS/followers","following_url":"https://api.github.com/users/OYTIS/following{/other_user}","gists_url":"https://api.github.com/users/OYTIS/gists{/gist_id}","starred_url":"https://api.github.com/users/OYTIS/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OYTIS/subscriptions","organizations_url":"https://api.github.com/users/OYTIS/orgs","repos_url":"https://api.github.com/users/OYTIS/repos","events_url":"https://api.github.com/users/OYTIS/events{/privacy}","received_events_url":"https://api.github.com/users/OYTIS/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2018-12-12T15:10:09Z","updated_at":"2018-12-18T10:57:10Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/advancedtelematic/meta-updater/pulls/446","html_url":"https://github.com/advancedtelematic/meta-updater/pull/446","diff_url":"https://github.com/advancedtelematic/meta-updater/pull/446.diff","patch_url":"https://github.com/advancedtelematic/meta-updater/pull/446.patch"},"body":"It is how the other SOTA-specific variables are treated.\r\n\r\nSigned-off-by: Anton Gerasimov <anton.gerasimov@here.com>"},"comment":{"url":"https://api.github.com/repos/advancedtelematic/meta-updater/issues/comments/448181164","html_url":"https://github.com/advancedtelematic/meta-updater/pull/446#issuecomment-448181164","issue_url":"https://api.github.com/repos/advancedtelematic/meta-updater/issues/446","id":448181164,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0ODE4MTE2NA==","user":{"login":"Jmennius","id":11856312,"node_id":"MDQ6VXNlcjExODU2MzEy","avatar_url":"https://avatars3.githubusercontent.com/u/11856312?v=4","gravatar_id":"","url":"https://api.github.com/users/Jmennius","html_url":"https://github.com/Jmennius","followers_url":"https://api.github.com/users/Jmennius/followers","following_url":"https://api.github.com/users/Jmennius/following{/other_user}","gists_url":"https://api.github.com/users/Jmennius/gists{/gist_id}","starred_url":"https://api.github.com/users/Jmennius/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Jmennius/subscriptions","organizations_url":"https://api.github.com/users/Jmennius/orgs","repos_url":"https://api.github.com/users/Jmennius/repos","events_url":"https://api.github.com/users/Jmennius/events{/privacy}","received_events_url":"https://api.github.com/users/Jmennius/received_events","type":"User","site_admin":false},"created_at":"2018-12-18T10:57:10Z","updated_at":"2018-12-18T10:57:10Z","author_association":"CONTRIBUTOR","body":"@OYTIS, this is exactly what I am concerned about.\r\nThe end result of this is that you can only override this kind of 'weak conditional assignment' with an additional override. This is extremely inconvenient.\r\n\r\nFor example, let's take a `SOTA_MACHINE` variable.\r\nWhen you implement your machine (in Yocto/OE terms) based on RPi, you create a machine config file `mymachine.conf`, in which:\r\n- you usually include a base board config (e.g. RPi) to inherit its configuration\r\n- extend `MACHINEOVERRIDES` to include `rpi`\r\n- and in the case of meta-updater integration, you set `SOTA_MACHINE` to `MACHINE` in your distro configuration (if you are not doing any tricky name mappings)\r\n- alternatively, you can set `SOTA_MACHINE` to a value of `MACHINE` in your machine configuration, but this is no good\r\n\r\nHaving this style of override makes you do `SOTA_MACHINE_mymachine = \"${MACHINE}\"` in your distro config, which is quite bad, don't you agree?\r\n\r\nRegarding the PR itself,\r\nI think this should be fixed in OE to have weak default value assignment, and the meta-updater to use ordinary default assignment.\r\nAlternatively, we can still use plain assignment in meta-updater and other layers should be able to override this assignment by ordering their assignment after meta-updater one.\r\nIMO, using overrides that way is, to say the least, very inconvenient.\r\n\r\nP.S. I also don't think this is properly documented (please point me if you know)."}}
{"actor":{"display_login":"Jmennius"}}