On Wed, 2022-04-13 at 10:35 +0100, Abongwa Amahnui Bonalais wrote: > Signed-off-by: Abongwa Bonalais Amahnui <[email protected]> > --- > scripts/bitbake-prserv-tool | 22 +++++++++++----------- > 1 file changed, 11 insertions(+), 11 deletions(-)
I want to suggest a little bit of caution with the direction these changes are going. Showing the user a clear error if that have spaces in a build path is a good thing. Making oe-init-build-env handle that correctly is therefore desirable so we can get the proper error from bitbake. This is likely the entry point most users would find (toaster is another) Since we don't support spaces in build paths and we're unlikely to do so given that autotools can't, I'm not sure we want to go through too many of our scripts doing this kind of change though. Most users would use the prserv tool as an initial introduction to the project. We did already see potential problems being introduced in the toaster changes on the bitbake list and I do worry about potential regressions. Cheers, Richard
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#164317): https://lists.openembedded.org/g/openembedded-core/message/164317 Mute This Topic: https://lists.openembedded.org/mt/90437025/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
