| Summary: | Fix the broken SDK with ccache | ||
|---|---|---|---|
| Product: | [Build System, Metadata & Runtime] OE-Core | Reporter: | Laszlo Papp <lpapp> |
| Component: | devtools / tool chain | Assignee: | Laszlo Papp <lpapp> |
| Status: | CLOSED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | Low | CC: | armccurdy, jefro, joshuagloe, meta.mr.watcher, meta.watcher, richard.purdie |
| Version: | unspecified | ||
| Target Milestone: | 1.8.2 | ||
| Hardware: | x86 | ||
| OS: | Multiple | ||
| Whiteboard: | |||
| OS type for building Yocto: | --- | Type of Regression: | --- |
| Verified: | Documentation change: | Yes (doc changes required) | |
|
Description
Laszlo Papp
2014-09-09 15:26:28 UTC
Reassigning to Laszlo since the next steps with this rest with him. It is not up to us to rewrite the commit message to conform to submission guidelines, or explain the problem the patch is solving properly in the commit message which it currently does not. Please do not assign it to me. I do not intend to work on this. I have already made all the work needed available in our fork. If you do not care about the broken SDK for people using ccache, close as won't fix or so. Patch is present in the oe-core master branch [1], however was broken soon after the initial merge [2], then subsequently fixed again [3]. The YP 1.8 release contains the broken version. YP 1.8.1 is expected to contain the fix. [1] http://git.openembedded.org/openembedded-core/commit/?id=1d31ddb856a80ba9da1a64ed5d701dc0f7351ef7 [2] http://git.openembedded.org/openembedded-core/commit/?id=62930ea37cfd5ba787ee85bc8dab23c20a775253 [3] http://git.openembedded.org/openembedded-core/commit/?id=986e5f37f3450077c843777c22df6b2d0f9502c5 Hmm, it works for me with the initial merge. As far as I can see the the following change was applied against the ccache path as well as PATH, so that is probably a bit more generic change than just fixing ccache. But yeah, thank you for the update. (In reply to comment #4) > Hmm, it works for me with the initial merge. As far as I can see the the > following change was applied against the ccache path as well as PATH, so > that is probably a bit more generic change than just fixing ccache. Yes, the initial merge was fine. Unfortunately quoting got broken between the initial merge and the Yocto 1.8 release. It's fixed again now in both the master and fido branches of oe-core. > But yeah, thank you for the update. Thanks for the patch! Confirmed fixed in fido 1.8.1. Fixed in the released 1.8.1 |