Skip to content

build(openpilot): replace panda submodule with pip-distributed panda dependency#37908

Closed
adwit-agg wants to merge 4 commits into
commaai:masterfrom
adwit-agg:remove-panda-submodule
Closed

build(openpilot): replace panda submodule with pip-distributed panda dependency#37908
adwit-agg wants to merge 4 commits into
commaai:masterfrom
adwit-agg:remove-panda-submodule

Conversation

@adwit-agg

Copy link
Copy Markdown
  • Remove panda submodule entry from .gitmodules
  • Add pandacan dependency in pyproject.toml, pinned to fork branch for CI validation
  • Update SConstruct to resolve panda paths dynamically from the installed Python package location
  • Invoke panda SConscript via variant_dir='panda' with duplicate=0 so firmware binaries build in-tree without mutating read-only site-packages sources

Verification PR to commaai/panda#2392

…dependency

- Remove panda submodule entry from .gitmodules
- Add pandacan dependency in pyproject.toml, pinned to fork branch for CI validation
- Update SConstruct to resolve panda paths dynamically from the installed Python package location
- Invoke panda SConscript via variant_dir='panda' with duplicate=0 so firmware binaries build in-tree without mutating read-only site-packages sources
@adwit-agg adwit-agg force-pushed the remove-panda-submodule branch from 66dac31 to b74b848 Compare April 25, 2026 11:41
@github-actions

github-actions Bot commented Apr 25, 2026

Copy link
Copy Markdown
Contributor

Process replay diff report

Replays driving segments through this PR and compares the behavior to master.
Please review any changes carefully to ensure they are expected.

✅ 0 changed, 66 passed, 0 errors

… This should resolve the check-dirty.sh release test from failing.
@adwit-agg adwit-agg marked this pull request as ready for review April 25, 2026 12:30
@github-actions

Copy link
Copy Markdown
Contributor

This PR has had no activity for 24 days. It will be automatically closed in 7 days if there is no activity.

@github-actions github-actions Bot added the stale label Jun 10, 2026
@github-actions

Copy link
Copy Markdown
Contributor

This PR has been automatically closed due to inactivity. Feel free to re-open once activity resumes.

@github-actions github-actions Bot closed this Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant