diff mbox series

[bitbake-devel,v3] bitbake-setup: Add strip to value from settings.conf

Message ID 20260209021712.9724-1-liuyd.fnst@fujitsu.com
State New
Headers show
Series [bitbake-devel,v3] bitbake-setup: Add strip to value from settings.conf | expand

Commit Message

Liu Yiding Feb. 9, 2026, 2:17 a.m. UTC
If user add "" to dl-dir as DL_DIR, it will show FetchError as following.

 raise FetchError("Unable to fetch URL from any source.", u)
bb.fetch2.FetchError: Fetcher failure for URL: 'git://git.openembedded.org/bitbake;protocol=https;rev=master;branch=master;destsuffix=bitbake'. Unable to fetch URL from any source.

But from this log, user can't locate the real reason which is that the path "/aa/bb" is invalid.

So I make this fix, even if user add "" to dl-dir, the dl-dir will be the valid path.
Other settings value also need this fix.

Signed-off-by: Liu Yiding <liuyd.fnst@fujitsu.com>
---
 bin/bitbake-setup | 6 ++++++
 1 file changed, 6 insertions(+)
diff mbox series

Patch

diff --git a/bin/bitbake-setup b/bin/bitbake-setup
index d0d9929d3..e18b01019 100755
--- a/bin/bitbake-setup
+++ b/bin/bitbake-setup
@@ -920,6 +920,12 @@  def load_settings(settings_path):
     if os.path.exists(settings_path):
         logger.info('Loading settings from {}'.format(settings_path))
         settings.read_file(open(settings_path))
+
+        for section in settings.sections():
+            for key in settings[section]:
+                value = settings[section][key]
+                settings[section][key] = value.strip('\"\'')
+
     return settings
 
 def change_setting(top_dir, args):