From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: ross.burton@arm.com, openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH 1/7] sstate/relocatable: remove manual relocation of .pc files
Date: Fri, 20 Mar 2026 12:48:45 +0000 [thread overview]
Message-ID: <b74d3f738ec8e83165d2f81c772b7936216f0fc3.camel@linuxfoundation.org> (raw)
In-Reply-To: <20260320122732.1630715-1-ross.burton@arm.com>
On Fri, 2026-03-20 at 12:27 +0000, Ross Burton via lists.openembedded.org wrote:
> There's no need to manually relocate .pc files in relocatable.bbclass
> as the path remapping in sstate.bbclass can do it, if we add .pc to
> SSTATE_SCAN_FILES.
>
> This removes some hardcoded behaviour, and puts absolute sysroot paths
> into the .pc files instead of paths like ${pcfiledir}/../../include,
> which makes it possible for pkgconfig to normalise and strip redundant
> paths.
>
> This also has the side effect of not forcibly turning .pc symlinks into
> real files, which the sed was doing.
Please don't effectively revert https://git.openembedded.org/openembedded-core/commit/meta/classes/relocatable.bbclass?id=a3f4e9ff55c7d9eba175e83eb6c99cf349a74432
The sstate relocation code creates loads of copies of files rather than
allowing us to hardlink them and is bad for performance. By making the
files relocatable, we don't have to do all the fixup handling in that
code and it is generally lower overhead on the system. I think I did
this to stop a lot of sstate objects needing relocations at all.
If there is a symlink issue, we should just make it skip those.
Cheers,
Richard
prev parent reply other threads:[~2026-03-20 12:48 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-20 12:27 [PATCH 1/7] sstate/relocatable: remove manual relocation of .pc files Ross Burton
2026-03-20 12:27 ` [PATCH 2/7] pkgconf: inherit pkgconfig Ross Burton
2026-03-20 12:27 ` [PATCH 3/7] pkgconf: ensure pkgconf-native doesn't try and look on the host Ross Burton
2026-03-20 12:27 ` [PATCH 4/7] pkgconf: minimise wrapper Ross Burton
2026-03-20 12:27 ` [PATCH 5/7] class/pkgconfig: use pkgconf instead of pkgconfig Ross Burton
2026-03-21 1:40 ` [OE-core] " Khem Raj
2026-03-21 11:59 ` Ross Burton
2026-03-25 0:05 ` Dmitry Baryshkov
2026-03-25 0:32 ` [OE-core] " Khem Raj
2026-03-20 12:27 ` [PATCH 6/7] pkgconfig: remove Ross Burton
2026-03-20 12:27 ` [PATCH 7/7] layer.conf: update pkgconfig-native exclusions to pkgconf-native Ross Burton
2026-03-20 12:46 ` Patchtest results for " patchtest
2026-03-20 12:48 ` Richard Purdie [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b74d3f738ec8e83165d2f81c772b7936216f0fc3.camel@linuxfoundation.org \
--to=richard.purdie@linuxfoundation.org \
--cc=openembedded-core@lists.openembedded.org \
--cc=ross.burton@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox