RP asked on #yocto to add TARGET_DBGSRC_DIR variable for
/usr/src/debug/${PN}/${PV} so that all recipes fixed in follow up changes
(and 10+ more for meta-oe layers) can use this instead of hardcoding the
/usr/src/debug/${PN}/${PV} value.Will send v4 later today. On Thu, Nov 16, 2023 at 8:36 AM Martin Jansa via lists.openembedded.org <[email protected]> wrote: > * ${EXTENDPE} and ${PR} was already dropped from ${WORKDIR} in: > https://lists.openembedded.org/g/openembedded-core/message/185818 > adjust -fmacro-prefix-map and -fdebug-prefix-map similarly. > As we don't depend on PR in WORKDIR maybe we don't need it in > /usr/src/debug paths as well anymore. > > * The interesting side-effect of this is that for DISTROs which still > use PR (e.g. webOS OSE), the difference in -fdebug-prefix-map causes > all binaries to be slightly different when rebuilt with different PR > (due to this path, even when otherwise they are binary reproducible) > and when the number of digits in PR changes (e.g. from r9 to r10) it > also results in slightly bigger binaries (and more unnecessary changes > in buildhistory diffs). > > For example this foo binary, the difference between "new" and "old" is > extra patch applied in SRC_URI (which doesn't affect the foo binary, > but was the reason for PR bump). > > And the binary is bigger with r10, identical with r6 and slightly > different due to debug-prefix-map between r6 and r7: > > $ ls -la 1.0.0-175-*/image/usr/sbin/foo > -rwxr-xr-x 1 bitbake bitbake 30182460 Nov 8 08:29 > 1.0.0-175-new-r10/image/usr/sbin/foo > -rwxr-xr-x 1 bitbake bitbake 30182224 Nov 8 08:02 > 1.0.0-175-new-r6/image/usr/sbin/foo > -rwxr-xr-x 1 bitbake bitbake 30182224 Nov 8 07:49 > 1.0.0-175-new-r7/image/usr/sbin/foo > -rwxr-xr-x 1 bitbake bitbake 30182224 Nov 8 07:39 > 1.0.0-175-old-r6/image/usr/sbin/foo > > $ md5sum 1.0.0-175-*/image/usr/sbin/foo > 8e3ab67d596f8cc2f9c6320dab10af01 1.0.0-175-new-r10/image/usr/sbin/foo > f6ff1e64fe6affb9ba0d8f278cf7ed79 1.0.0-175-new-r6/image/usr/sbin/foo > 6469a0b10aac233911e63c5ea97b03c0 1.0.0-175-new-r7/image/usr/sbin/foo > f6ff1e64fe6affb9ba0d8f278cf7ed79 1.0.0-175-old-r6/image/usr/sbin/foo > > Signed-off-by: Martin Jansa <[email protected]> > --- > v3: added additional fix for perf (and changes in other recipes were moved > to separate commits) > > meta/conf/bitbake.conf | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf > index 67ff769046..3836a389b8 100644 > --- a/meta/conf/bitbake.conf > +++ b/meta/conf/bitbake.conf > @@ -648,10 +648,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}=/usr/src/debug/${PN}/${PV} \ > + -fdebug-prefix-map=${S}=/usr/src/debug/${PN}/${PV} \ > + -fmacro-prefix-map=${B}=/usr/src/debug/${PN}/${PV} \ > + -fdebug-prefix-map=${B}=/usr/src/debug/${PN}/${PV} \ > -fdebug-prefix-map=${STAGING_DIR_HOST}= \ > -fmacro-prefix-map=${STAGING_DIR_HOST}= \ > -fdebug-prefix-map=${STAGING_DIR_NATIVE}= \ > -- > 2.42.1 > > > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#190769): https://lists.openembedded.org/g/openembedded-core/message/190769 Mute This Topic: https://lists.openembedded.org/mt/102623139/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
