10799 2016-12-13 08:32:54 +0000 compatibility with rm_work.bbclass 2017-02-24 22:17:08 +0000 1 1 7 Build System, Metadata & Runtime Other YP Layers meta-swupd unspecified x86 Multiple RESOLVED FIXED Medium normal 2.3 10584 1 patrick.ohly aaron.zinghini aaron.zinghini sgw --- --- No (bug/feature does not impact docs) oldest_to_newest 69074 0 patrick.ohly 2016-12-13 08:32:54 +0000 Build breaks with meta-swupd fail when rm_work.bbclass is used because meta-swupd accesses content from other recipes which might have been removed already by do_rm_work. I have patches ready, but they depend on enhancing do_rm_work. I intend to submit together with bug #10584. 70634 1 3631 aaron.zinghini 2017-02-10 19:39:47 +0000 Created attachment 3631 swupd-image.bbclass stacktrace 70635 2 aaron.zinghini 2017-02-10 19:41:22 +0000 This change appears to break things for me. I believe the problem is a missing expand arg in the "workdir = d.getVar('WORKDIR')" statement. 70636 3 aaron.zinghini 2017-02-10 19:47:08 +0000 Actually this appears to be because I am not using the master branch of poky and that the default for getVars has changed. This leads me to the question of whether there will be a stable Morty branch of meta-swupd anytime soon? 70652 4 patrick.ohly 2017-02-13 08:48:15 +0000 (In reply to comment #3) > Actually this appears to be because I am not using the master branch of poky > and that the default for getVars has changed. This leads me to the question > of whether there will be a stable Morty branch of meta-swupd anytime soon? I've taken over maintenance of the layer for now, but it is unclear whether that will continue, and targeting older releases wasn't part of that (tentative) plan either. I see two possible outcomes: 1. We have some interest in using swupd and resources to support it. Then proper testing needs to be set up, including older releases, and Morty will be supported. 2. There's not enough interest to justify the work. Then some of those still interested will need to step up and take over maintenance. Regarding this particular problem, is it enough to add "True" to the d.getVar() call? If that works for you, can you submit a patch? That can go into the master branch and then we can avoid forking for a while longer. 70666 5 aaron.zinghini 2017-02-13 17:13:26 +0000 Sad to hear there isn't much interest in this layer. The commercial project I work on is looking at using this so it would seem I will be helping to keep this alive. I think adding the explicit "True" would suffice for now. I'm pretty new to contributing to open-source projects and git. Do you have any info/guidelines I can use to get setup so I can submit patches? 70668 6 patrick.ohly 2017-02-13 17:33:12 +0000 (In reply to comment #5) > Sad to hear there isn't much interest in this layer. There is interest, it's just unclear whether there's an intersection between those willing to use it and those willing to maintain it ;-/ > The commercial project > I work on is looking at using this so it would seem I will be helping to > keep this alive. Out of curiosity, which features of swupd made you choose it over the competing solutions? See also https://wiki.yoctoproject.org/wiki/System_Update > I think adding the explicit "True" would suffice for now. I'm pretty new to > contributing to open-source projects and git. Do you have any > info/guidelines I can use to get setup so I can submit patches? "git clone git://git.yoctoproject.org/meta-swupd", then make the change, test it, commit it with a commit message according to http://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines, subscribe to openembedded-devel@lists.openembedded.org (see http://www.openembedded.org/wiki/Mailing_lists) then send with "git send-email --to=openembedded-devel@lists.openembedded.org '--subject-prefix=meta-swupd][PATCH' HEAD~..HEAD". Then it ends up getting reviewed and merged. 70669 7 aaron.zinghini 2017-02-13 17:47:01 +0000 The main reason for preferring swupd is for the fast minimal updates and moving away from our current approach which requires full system imaging to get OS updates. A lot of the competing projects also require U-boot, which is not an option for our HW. I will follow the guidelines you mentioned and submit the patch today. 70670 8 patrick.ohly 2017-02-13 18:56:03 +0000 (In reply to comment #7) > The main reason for preferring swupd is for the fast minimal updates and > moving away from our current approach which requires full system imaging to > get OS updates. Agreed, file-based approaches have an advantage there. Do you do live updates (including swapping out files which may be currently in use), followed by an obligatory reboot, or do you selectively restart daemons that use old files? > A lot of the competing projects also require U-boot, which > is not an option for our HW. Is your hardware booting with UEFI? > I will follow the guidelines you mentioned and submit the patch today. Thanks! 70672 9 aaron.zinghini 2017-02-13 21:35:34 +0000 The way we would use swupd would be to check for updates on boot, apply, then reboot. Yes HW is booting with UEFI. 70939 10 aaron.zinghini 2017-02-24 21:45:52 +0000 Patch has been upstreamed. 3631 2017-02-10 19:39:47 +0000 2017-02-10 19:39:47 +0000 swupd-image.bbclass stacktrace swupdlog.txt text/plain 898 aaron.zinghini RmlsZTogJy92YXIvbGliL2plbmtpbnMvd29ya3NwYWNlL3lvY3RvL21ldGEtc3d1cGQvY2xhc3Nl cy9zd3VwZC1pbWFnZS5iYmNsYXNzJywgbGluZW5vOiAxNTgsIGZ1bmN0aW9uOiBfX2Fub25fMjQ3 X192YXJfbGliX2plbmtpbnNfd29ya3NwYWNlX09yaW9uX3lvY3RvX21ldGFfc3d1cGRfY2xhc3Nl c19zd3VwZF9pbWFnZV9iYmNsYXNzCiAgICAgMDE1NDogICAgICAgIG1lZ2Ffcm9vdGZzID0gbWVn YV9yb290ZnMucmVwbGFjZSgnLycgKyBwbiArJy8nLCAnL2J1bmRsZS0lcy1tZWdhLycgJSAocG5f YmFzZSBvciBwbikpCiAgICAgMDE1NTogICAgICAgIGQuc2V0VmFyKCdNRUdBX0lNQUdFX1JPT1RG UycsIG1lZ2Ffcm9vdGZzKQogICAgIDAxNTY6ICAgICAgICBtZWdhX2FyY2hpdmUgPSBtZWdhX3Jv b3RmcyArICcudGFyJwogICAgIDAxNTc6ICAgICAgICB3b3JrZGlyID0gZC5nZXRWYXIoJ1dPUktE SVInKQogKioqIDAxNTg6ICAgICAgICBkLnNldFZhcignTUVHQV9JTUFHRV9BUkNISVZFJywgbWVn YV9hcmNoaXZlKQogICAgIDAxNTk6ICAgICAgICBtZWdhX2FyY2hpdmVfcmVsID0gb3MucGF0aC5y ZWxwYXRoKG1lZ2FfYXJjaGl2ZSwgd29ya2RpcikKICAgICAwMTYwOiAgICAgICAgaWYgb3MucGF0 aC5zZXAgbm90IGluIG1lZ2FfYXJjaGl2ZV9yZWw6CiAgICAgMDE2MTogICAgICAgICAgICMgVGhl IG1lZ2EgYXJjaGl2ZSBpcyBpbiBvdXIgd29yayBkaXJlY3RvcnkgYW5kIG11c3QgYmUKICAgICAw MTYyOiAgICAgICAgICAgIyBwcmVzZXJ2ZWQgZm9yIG90aGVyIHZpcnR1YWwgaW1hZ2VzIGV2ZW4g d2hlbiBybV93b3JrLmJiY2xhc3MKRXhjZXB0aW9uOiBUeXBlRXJyb3I6IGdldFZhcigpIG1pc3Np bmcgMSByZXF1aXJlZCBwb3NpdGlvbmFsIGFyZ3VtZW50OiAnZXhwYW5kJw==