Skip to content

Add env-config-container to packagefeed-ni-core for x64#1007

Open
AshwinUjjwal wants to merge 1 commit into
ni:nilrt/master/nextfrom
AshwinUjjwal:dev/env-config-fixes
Open

Add env-config-container to packagefeed-ni-core for x64#1007
AshwinUjjwal wants to merge 1 commit into
ni:nilrt/master/nextfrom
AshwinUjjwal:dev/env-config-fixes

Conversation

@AshwinUjjwal
Copy link
Copy Markdown
Contributor

@AshwinUjjwal AshwinUjjwal commented May 22, 2026

Summary of Changes

  • Include env-config-container package in the NI core package feed
  • Ensures the container-specific configuration package is available for x64 deployments
  • Required for nilrt-slim-container and nilrt-runmode-container builds

Justification

AB#3201993

Testing

  • bitbake packagefeed-ni-core
  • bitbake nilrt-slim-container
  • bitbake nilrt-runmode-container

Procedure

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds env-config-container to the x64 packagegroup runtime dependencies so it gets included in the NI core image/feed set.

Changes:

  • Extend RDEPENDS:${PN}:append:x64 with env-config-container

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread recipes-core/packagegroups/packagefeed-ni-core.bb
Copy link
Copy Markdown
Contributor

@rajendra-desai-ni rajendra-desai-ni left a comment

Choose a reason for hiding this comment

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

  • You need to mainly test bitbake packagefeed-ni-core since that is the recipe you are making changes to.
  • Follow 50/72 rule for commit messages.
  • You could reword the commit title to pkgfeed-ni-core: add env-config-container for x64 or something.

@rajendra-desai-ni rajendra-desai-ni requested a review from a team May 22, 2026 13:37
@AshwinUjjwal AshwinUjjwal force-pushed the dev/env-config-fixes branch from a22c962 to 55543a7 Compare May 22, 2026 13:43
- Include env-config-container package in the NI core package feed
- Required for nilrt-slim-container and nilrt-runmode-container builds

Signed-off-by: AshwinUjjwal <ashwinujjwal.bharti@emerson.com>
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.

3 participants