11307 2017-04-07 09:18:46 +0000 sstate.bbclass tries to modify mirror files 2017-04-13 14:28:55 +0000 1 1 7 Build System, Metadata & Runtime OE-Core configuration unspecified x86 Multiple RESOLVED FIXED Medium+ normal 2.3 M4 1 ptsneves eduard.bartosh --- --- No (bug/feature does not impact docs) oldest_to_newest 72172 0 ptsneves 2017-04-07 09:18:46 +0000 Commit 51edde653707e7a3cd2186082458f01f32cd1996 makes a wrong assumption that SSTATE_MIRRORS have write permissions. A mirror is by definition outside of it's user control. In my use case it happens I does not have permissions to update the access time of the dereferenced symbolic-link file. If we had the possibility to modify the sstate-mirror we would have set SSTATE_DIR instead. There are 3 possible fixes: * Revert this patch. It seems tailored to a very special use case which breaks the definition of mirror. * Only touch the file if the file access time is writeable. * Set (yet) another option to touch SSTATE_MIRROR files. I also can imagine this raising sever performance issues if the mirror is used by several users at the same time. The server will need to update the file meta data every time I can provide the patch for one of the 3 options above. Related to (https://bugzilla.yoctoproject.org/show_bug.cgi?id=10857) Best regards Paulo Neves 72269 1 3705 ptsneves 2017-04-11 08:13:04 +0000 Created attachment 3705 Added test for write ability This patch fixes the bug with minimal impact on the original functionality. 72412 2 eduard.bartosh 2017-04-12 21:14:11 +0000 Paulo, thank you for the patch. Sent it for review: http://lists.openembedded.org/pipermail/openembedded-core/2017-April/135621.html 72461 3 eduard.bartosh 2017-04-13 14:28:55 +0000 The fix has been merged into master 3705 2017-04-11 08:13:04 +0000 2017-04-11 08:13:04 +0000 Added test for write ability 0001-Added-test-for-write-ability-of-sstate-mirror-.sigin.patch text/plain 1102 ptsneves RnJvbSBhYThjYWI5NjYwNzQ0YzhkNTI3N2Y4ZTgyZjEwYTg0NDg4NGFmZjc4IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBQYXVsbyBOZXZlcyA8cGF1bG8uZGVfc291c2FfbmV2ZXNAbm9r aWEuY29tPgpEYXRlOiBUdWUsIDExIEFwciAyMDE3IDEwOjU3OjQ3ICswMzAwClN1YmplY3Q6IFtQ QVRDSF0gQWRkZWQgdGVzdCBmb3Igd3JpdGUgYWJpbGl0eSBvZiBzc3RhdGUtbWlycm9yIC5zaWdp bmZvCgotLS0KIG1ldGEvY2xhc3Nlcy9zc3RhdGUuYmJjbGFzcyB8IDIgKy0KIDEgZmlsZSBjaGFu Z2VkLCAxIGluc2VydGlvbigrKSwgMSBkZWxldGlvbigtKQoKZGlmZiAtLWdpdCBhL21ldGEvY2xh c3Nlcy9zc3RhdGUuYmJjbGFzcyBiL21ldGEvY2xhc3Nlcy9zc3RhdGUuYmJjbGFzcwppbmRleCBh ZjU4ODU0OGMyZWY4NWQ5NjY3ODdkZjdhZTE0MzJlMTc0NGJkYmE2Li4zNmNiYmQxNTFhYTFjOTNm NGNhMzc3YjNhMjNjMDExYTUyNzM4OWQ4IDEwMDY0NAotLS0gYS9tZXRhL2NsYXNzZXMvc3N0YXRl LmJiY2xhc3MKKysrIGIvbWV0YS9jbGFzc2VzL3NzdGF0ZS5iYmNsYXNzCkBAIC03NTMsNyArNzUz LDcgQEAgcHl0aG9uIHNzdGF0ZV9zaWduX3BhY2thZ2UgKCkgewogc3N0YXRlX3VucGFja19wYWNr YWdlICgpIHsKIAl0YXIgLXh2emYgJHtTU1RBVEVfUEtHfQogCSMgdXBkYXRlIC5zaWdpbmZvIGF0 aW1lIG9uIGxvY2FsL05GUyBtaXJyb3IKLQlbIC1oICR7U1NUQVRFX1BLR30uc2lnaW5mbyBdICYm IHRvdWNoIC1hICR7U1NUQVRFX1BLR30uc2lnaW5mbworCVsgLXcgJHtTU1RBVEVfUEtHfS5zaWdp bmZvIF0gJiYgWyAtaCAke1NTVEFURV9QS0d9LnNpZ2luZm8gXSAmJiB0b3VjaCAtYSAke1NTVEFU RV9QS0d9LnNpZ2luZm8KIAkjIFVzZSAiISAtdyB8fCIgdG8gcmV0dXJuIHRydWUgZm9yIHJlYWQg b25seSBmaWxlcwogCVsgISAtdyAke1NTVEFURV9QS0d9IF0gfHwgdG91Y2ggLS1uby1kZXJlZmVy ZW5jZSAke1NTVEFURV9QS0d9CiAJWyAhIC13ICR7U1NUQVRFX1BLR30uc2lnIF0gfHwgWyAhIC1l ICR7U1NUQVRFX1BLR30uc2lnIF0gfHwgdG91Y2ggLS1uby1kZXJlZmVyZW5jZSAke1NTVEFURV9Q S0d9LnNpZwotLSAKMi42LjIKCg==