diff mbox series

fetch/npmsw: The fetcher shouldn't have any knowledge of S

Message ID 20240522154631.2212924-1-richard.purdie@linuxfoundation.org
State Accepted, archived
Commit 84f102954e10a3390fca9c26d5c3c639e952a2c9
Headers show
Series fetch/npmsw: The fetcher shouldn't have any knowledge of S | expand

Commit Message

Richard Purdie May 22, 2024, 3:46 p.m. UTC
I don't know why there is hardcoded knowledge of S in the fetcher but there
shouldn't be and the OE unpack changes highlight this doing things it
shouldn't.

Drop the S reference and use rootdir which is the only place it should
be touching.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 lib/bb/fetch2/npmsw.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/lib/bb/fetch2/npmsw.py b/lib/bb/fetch2/npmsw.py
index ff5f8dc755..b55e885d7b 100644
--- a/lib/bb/fetch2/npmsw.py
+++ b/lib/bb/fetch2/npmsw.py
@@ -268,7 +268,7 @@  class NpmShrinkWrap(FetchMethod):
 
     def unpack(self, ud, rootdir, d):
         """Unpack the downloaded dependencies"""
-        destdir = d.getVar("S")
+        destdir = rootdir
         destsuffix = ud.parm.get("destsuffix")
         if destsuffix:
             destdir = os.path.join(rootdir, destsuffix)