* [PATCH] package: fix src packaging path for nativesdk
@ 2023-06-20 21:36 Maxime Roussin-Bélanger
2023-06-22 9:34 ` [OE-core] " Luca Ceresoli
0 siblings, 1 reply; 2+ messages in thread
From: Maxime Roussin-Bélanger @ 2023-06-20 21:36 UTC (permalink / raw)
To: openembedded-core; +Cc: Maxime Roussin-Belanger
From: Maxime Roussin-Belanger <maxime.roussinbelanger@gmail.com>
Directory tree doesn't contain the correct structure when packaging
for source files for the sdk. Since the structure is incorrect,
they won't be part of the installed sdk.
Having source files in nativesdk is necessary when using it as a
development sysroot for debugging. Otherwise, when debugging in an
emulation mode it's impossible to step in 3rd party library code.
Signed-off-by: Maxime Roussin-Belanger <maxime.roussinbelanger@gmail.com>
---
meta/conf/bitbake.conf | 8 ++++----
meta/lib/oe/package.py | 25 +++++++++++++++++++------
2 files changed, 23 insertions(+), 10 deletions(-)
diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 702881144e..eb438a1499 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -649,10 +649,10 @@ EXTRA_OEMAKE:prepend:task-install = "${PARALLEL_MAKEINST} "
##################################################################
# Beware: applied last to first
DEBUG_PREFIX_MAP ?= "-fcanon-prefix-map \
- -fmacro-prefix-map=${S}=/usr/src/debug/${PN}/${EXTENDPE}${PV}-${PR} \
- -fdebug-prefix-map=${S}=/usr/src/debug/${PN}/${EXTENDPE}${PV}-${PR} \
- -fmacro-prefix-map=${B}=/usr/src/debug/${PN}/${EXTENDPE}${PV}-${PR} \
- -fdebug-prefix-map=${B}=/usr/src/debug/${PN}/${EXTENDPE}${PV}-${PR} \
+ -fmacro-prefix-map=${S}=${prefix}/src/debug/${PN}/${EXTENDPE}${PV}-${PR} \
+ -fdebug-prefix-map=${S}=${prefix}/src/debug/${PN}/${EXTENDPE}${PV}-${PR} \
+ -fmacro-prefix-map=${B}=${prefix}/src/debug/${PN}/${EXTENDPE}${PV}-${PR} \
+ -fdebug-prefix-map=${B}=${prefix}/src/debug/${PN}/${EXTENDPE}${PV}-${PR} \
-fdebug-prefix-map=${STAGING_DIR_HOST}= \
-fmacro-prefix-map=${STAGING_DIR_HOST}= \
-fdebug-prefix-map=${STAGING_DIR_NATIVE}= \
diff --git a/meta/lib/oe/package.py b/meta/lib/oe/package.py
index 70040f09e7..b2799aec3e 100644
--- a/meta/lib/oe/package.py
+++ b/meta/lib/oe/package.py
@@ -687,6 +687,10 @@ def split_locales(d):
#d.setVar('RDEPENDS:%s' % pn, ' '.join(rdep))
def package_debug_vars(d):
+ prefix = d.getVar('prefix')
+ srcdir = ("%s/src/debug" % prefix)
+ libdir = ("%s/lib/debug" % prefix)
+ staticlibdir = ("%s/lib/debug-static" % prefix)
# We default to '.debug' style
if d.getVar('PACKAGE_DEBUG_SPLIT_STYLE') == 'debug-file-directory':
# Single debug-file-directory style debug info
@@ -695,9 +699,9 @@ def package_debug_vars(d):
"staticappend": "",
"dir": "",
"staticdir": "",
- "libdir": "/usr/lib/debug",
- "staticlibdir": "/usr/lib/debug-static",
- "srcdir": "/usr/src/debug",
+ "libdir": libdir,
+ "staticlibdir": staticlibdir,
+ "srcdir": srcdir,
}
elif d.getVar('PACKAGE_DEBUG_SPLIT_STYLE') == 'debug-without-src':
# Original OE-core, a.k.a. ".debug", style debug info, but without sources in /usr/src/debug
@@ -718,7 +722,7 @@ def package_debug_vars(d):
"staticdir": "/.debug-static",
"libdir": "",
"staticlibdir": "",
- "srcdir": "/usr/src/debug",
+ "srcdir": srcdir,
}
else:
# Original OE-core, a.k.a. ".debug", style debug info
@@ -729,7 +733,7 @@ def package_debug_vars(d):
"staticdir": "/.debug-static",
"libdir": "",
"staticlibdir": "",
- "srcdir": "/usr/src/debug",
+ "srcdir": srcdir,
}
return debug_vars
@@ -1257,6 +1261,7 @@ def populate_packages(d):
dvar = d.getVar('PKGD')
packages = d.getVar('PACKAGES').split()
pn = d.getVar('PN')
+ prefix = d.getVar('prefix')
bb.utils.mkdirhier(outdir)
os.chdir(dvar)
@@ -1271,7 +1276,7 @@ def populate_packages(d):
src_package_name = ('%s-src' % d.getVar('PN'))
if not src_package_name in packages:
packages.append(src_package_name)
- d.setVar('FILES:%s' % src_package_name, '/usr/src/debug')
+ d.setVar('FILES:%s' % src_package_name, ('%s/src/debug' % prefix))
# Sanity check PACKAGES for duplicates
# Sanity should be moved to sanity.bbclass once we have the infrastructure
@@ -1301,6 +1306,7 @@ def populate_packages(d):
oldumask = os.umask(0)
debug = []
+ source = []
for root, dirs, files in cpath.walk(dvar):
dir = root[len(dvar):]
if not dir:
@@ -1309,6 +1315,11 @@ def populate_packages(d):
path = "." + os.path.join(dir, f)
if "/.debug/" in path or "/.debug-static/" in path or path.endswith("/.debug"):
debug.append(path)
+ elif ("%s/src/debug" % prefix) in path:
+ if split_source_package:
+ source.append(path)
+ else:
+ debug.append(path)
for pkg in packages:
root = os.path.join(pkgdest, pkg)
@@ -1325,6 +1336,8 @@ def populate_packages(d):
if autodebug and pkg.endswith("-dbg"):
files.extend(debug)
+ if split_source_package and pkg.endswith("-src"):
+ files.extend(source)
for file in files:
if (not cpath.islink(file)) and (not cpath.exists(file)):
--
2.36.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-06-22 9:34 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-20 21:36 [PATCH] package: fix src packaging path for nativesdk Maxime Roussin-Bélanger
2023-06-22 9:34 ` [OE-core] " Luca Ceresoli
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox