]> code.ossystems Code Review - meta-freescale.git/commitdiff
fsl-eula-unpack.bbclass: Cleanup
authorTom Hochstein <tom.hochstein@nxp.com>
Wed, 18 Mar 2020 14:55:57 +0000 (09:55 -0500)
committerOtavio Salvador <otavio@ossystems.com.br>
Fri, 27 Mar 2020 13:13:39 +0000 (10:13 -0300)
- Remove current working directory handling and use
  the equivalent runfetchcmd workdir parameter
- Simplify the SRC_URI for loop

Signed-off-by: Tom Hochstein <tom.hochstein@nxp.com>
(cherry picked from commit fee27818df40719c62bddde717737fb30cef3013)

classes/fsl-eula-unpack.bbclass

index 03f576e5722eb8ba4021eba185d35d45f5592078..9b8f2af69e225df67c851b9663f35c5031035654 100644 (file)
@@ -22,25 +22,16 @@ python fsl_bin_do_unpack() {
     fetcher = bb.fetch2.Fetch(src_uri, localdata)
 
     for url in fetcher.ud.values():
-        save_cwd = os.getcwd()
-        # Check for supported fetchers
-        if url.type in ['http', 'https', 'ftp', 'file']:
-            if url.parm.get('fsl-eula', False):
-                # If download has failed, do nothing
-                if not os.path.exists(url.localpath):
-                    bb.debug(1, "Exiting as '%s' cannot be found" % url.basename)
-                    return
-
-                # Change to the working directory
-                bb.note("Handling file '%s' as a Freescale's EULA binary." % url.basename)
-                save_cwd = os.getcwd()
-                os.chdir(rootdir)
-
-                cmd = "sh %s --auto-accept --force" % (url.localpath)
-                bb.fetch2.runfetchcmd(cmd, d, quiet=True)
-
-    # Return to the previous directory
-    os.chdir(save_cwd)
+        # Skip this fetcher if it's not under EULA or if the fetcher type is not supported
+        if not url.parm.get('fsl-eula', False) or url.type not in ['http', 'https', 'ftp', 'file']:
+            continue
+        # If download has failed, do nothing
+        if not os.path.exists(url.localpath):
+            bb.debug(1, "Exiting as '%s' cannot be found" % url.basename)
+            return
+        bb.note("Handling file '%s' as a Freescale EULA-licensed archive." % url.basename)
+        cmd = "sh %s --auto-accept --force" % (url.localpath)
+        bb.fetch2.runfetchcmd(cmd, d, quiet=True, workdir=rootdir)
 }
 
 python do_unpack() {