On Thu, May 07, 2020 at 01:20:40PM -0700, Khem Raj wrote: > > > On 5/7/20 12:57 PM, Denys Dmytriyenko wrote: > > On Thu, May 07, 2020 at 12:44:15PM -0700, Khem Raj wrote: > >> > >> > >> On 5/3/20 3:51 PM, Steve Sakoman wrote: > >>> Should I consider these two patches for dunfell LTS? > >>> > >> > >> Yes I think they are fixing a problem and are safe. > > > > I don't think it is safe though! > > > > The code has been moved to create_sdk_files() which is a POSTPROCESS > > command > > for SDK creation. What if SDK already provides own /etc/ld.so.conf? The > > code > > just simply overwrites it. > > > > but are there such usecases ? looking closely if SDK is extending the > ld.so.conf then they should be appending to it instead of overwriting > and same is true for this patch for master.
Exactly my point - should probably append, not overwrite. As of usecases - I can think of a non-standard nativesdk tool that adjusts /etc/ld.so.conf during install... -- Denys
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#138039): https://lists.openembedded.org/g/openembedded-core/message/138039 Mute This Topic: https://lists.openembedded.org/mt/73326300/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
