Skip to content

Conversation

@shubhanshumt26
Copy link
Contributor

@shubhanshumt26 shubhanshumt26 commented Jan 6, 2026

Justification

AB#3039672

Changes

Upstream openembedded-core dropped support for TCLIBCAPPEND, making separate TMPDIRs such as tmp-glibc unnecessary.

This PR updates NILRT to follow upstream behavior by replacing all tmp-glibc references with tmp. This aligns NILRT with current
OE-Core defaults and ensures consistent TMPDIR usage across glibc and musl builds.

No functional changes are introduced; this is a cleanup to maintain compatibility with upstream OE-Core.

Testing

  • Verified that all packages are generated in tmp dir only.

Process

  • This PR should be cherry-picked to the next/ ref.

Suggested Reviewers:

  • @ni/rtos

Upstream openembedded-core dropped TCLIBCAPPEND, making
separate TMPDIRs such as tmp-glibc unnecessary.

Update NILRT to follow upstream behavior by replacing all
tmp-glibc references with tmp. This aligns NILRT with current
OE-Core defaults and ensures consistent TMPDIR usage across
glibc and musl builds.

Upstream ref: conf/defaultsetup.conf: Drop TCLIBCAPPEND -  openembedded/openembedded-core@ebcd355

Signed-off-by: Shubhanshu Mani Tripathi <shubhanshu.mani.tripathi@emerson.com>
@shubhanshumt26 shubhanshumt26 requested a review from a team as a code owner January 6, 2026 08:52
Copy link
Contributor

@chaitu236 chaitu236 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This would require simultaneous updates to ni-central -next components and updates to rest of ni-central components later when we will make this the default distribution.

Are we forced to make this update? If not, this is a lot of work with very little ROI. I'd suggest making this a tech debt item and putting it in backlog.

@shubhanshumt26
Copy link
Contributor Author

This would require simultaneous updates to ni-central -next components and updates to rest of ni-central components later when we will make this the default distribution.

Are we forced to make this update? If not, this is a lot of work with very little ROI. I'd suggest making this a tech debt item and putting it in backlog.

Yes, this update is required.
To avoid the large scope and low ROI of updating all ni-central components now, I’ve created a PR :
local.conf.sample: temporarily force TMPDIR to tmp-glibc with a temporary override matching the previous behavior.

This has been tested and works as expected. I’ve also created a Tech Debt item to properly update the components and remove this workaround later when tmp-glibc becomes the default.

Hence closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants