Skip to content

jetpack-nixos: rebased#1591

Merged
brianmcgillion merged 1 commit intotiiuae:mainfrom
brianmcgillion:jetpack-nixos-rebased
Nov 29, 2025
Merged

jetpack-nixos: rebased#1591
brianmcgillion merged 1 commit intotiiuae:mainfrom
brianmcgillion:jetpack-nixos-rebased

Conversation

@brianmcgillion
Copy link
Copy Markdown
Collaborator

@brianmcgillion brianmcgillion commented Nov 27, 2025

Description of Changes

Type of Change

  • New Feature
  • Bug Fix
  • Improvement / Refactor

Related Issues / Tickets

Checklist

  • Clear summary in PR description
  • Detailed and meaningful commit message(s)
  • Commits are logically organized and squashed if appropriate
  • Contribution guidelines followed
  • Ghaf documentation updated with the commit - https://tiiuae.github.io/ghaf/
  • Author has run make-checks and it passes
  • All automatic GitHub Action checks pass - see actions
  • Author has added reviewers and removed PR draft status

Testing Instructions

Applicable Targets

  • Orin AGX aarch64
  • Orin NX aarch64
  • Lenovo X1 x86_64
  • Dell Latitude x86_64
  • System 76 x86_64

Installation Method

  • Requires full re-installation
  • Can be updated with nixos-rebuild ... switch
  • Other:

Test Steps To Verify:

  1. ...

@brianmcgillion brianmcgillion marked this pull request as draft November 27, 2025 02:34
@brianmcgillion brianmcgillion added the Needs Testing CI Team to pre-verify label Nov 27, 2025
@brianmcgillion brianmcgillion marked this pull request as ready for review November 27, 2025 03:30
@brianmcgillion
Copy link
Copy Markdown
Collaborator Author

this is purely a rebase of our branch onto the latest jetpack-nixos

Copy link
Copy Markdown
Contributor

@emrahbillur emrahbillur left a comment

Choose a reason for hiding this comment

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

Already built several images with Orin NX and flashed Orin AGX. Both normal and cross compile succeeds.
Also tested all cosmic features on Orins.
Quite responsive with no obvious errors observed

@emrahbillur
Copy link
Copy Markdown
Contributor

On downstream side the nixpkgs with cuda causing issues. So I'll test it with rebased nixpkgs.

@emrahbillur
Copy link
Copy Markdown
Contributor

When used with cuda there is mismatch between nixpkgs and jetpack-nixos :

    262|       fargs = functionArgs f;

   error: path '/nix/store/g02rq8ap30x3fp8zrz07ip5v1s0pzidn-source/pkgs/development/cuda-modules/packages/backendStdenv.nix' does not exist

@emrahbillur emrahbillur self-requested a review November 27, 2025 12:42
@brianmcgillion
Copy link
Copy Markdown
Collaborator Author

brianmcgillion commented Nov 27, 2025

When used with cuda there is mismatch between nixpkgs and jetpack-nixos :

    262|       fargs = functionArgs f;

   error: path '/nix/store/g02rq8ap30x3fp8zrz07ip5v1s0pzidn-source/pkgs/development/cuda-modules/packages/backendStdenv.nix' does not exist

looking briefly at this. it is a path difference of 25.05 vs 25.11. 25.11 final release date is 30th Nov. and 25.05 will be deprecated then. the issue is https://github.com/anduril/jetpack-nixos/blob/master/mk-overlay.nix#L184 which hardcodes the path and will have to be adjusted to meet the supported mainline.

https://github.com/NixOS/nixpkgs/tree/master/pkgs/development/cuda-modules/backendStdenv was added by the same person who made the change in jetpack-nixos

Signed-off-by: Brian McGillion <bmg.avoin@gmail.com>
@brianmcgillion brianmcgillion removed the Needs Testing CI Team to pre-verify label Nov 29, 2025
@brianmcgillion brianmcgillion mentioned this pull request Nov 29, 2025
@brianmcgillion brianmcgillion merged commit 1ed34d1 into tiiuae:main Nov 29, 2025
30 checks passed
@brianmcgillion brianmcgillion deleted the jetpack-nixos-rebased branch November 29, 2025 20:08
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