Consume xDevSM as the xdevsm PyPI package instead of a submodule - #9
Open
Thecave3 wants to merge 1 commit into
Open
Consume xDevSM as the xdevsm PyPI package instead of a submodule#9Thecave3 wants to merge 1 commit into
xdevsm PyPI package instead of a submodule#9Thecave3 wants to merge 1 commit into
Conversation
Migrate all seven example xApps (kpm_basic, kpm_prb, prb_control, radio_bearer_control, ho, digital_twin_prb, traffic_balancer) off the xDevSM git submodule onto the installable `xdevsm` package: - Remove the xDevSM submodule and every per-xApp setup_imports.py shim; add `xdevsm` to each xApp's requirements.txt. - Rewrite `xDevSM.` imports to the `xdevsm.*` namespace. - Docker: drop `COPY xDevSM` and the xDevSM LD_LIBRARY_PATH entry (encoders now load package-relative); bump images to python:3.11-slim-bullseye (xdevsm requires >= 3.11; bullseye/glibc 2.31 loads the exec-stack encoders without an interpreter patch); record the installed xdevsm version instead of XDEVSM_COMMIT. - Update README and CLAUDE.md (submodule → pip package). Note: graphify-out/ is a generated knowledge graph and still references the old xDevSM/ paths; regenerate with `/graphify --update`. Assisted-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Collaborator
Author
|
This PR intentionally does not hand-edit the generated knowledge graph under After this PR merges, rebuild it so the graph/wiki reflects the /graphify --updateThen commit the regenerated |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Switch the example xApps off the xDevSM git submodule and consume it as the installable
xdevsmPyPI package (paired withwineslab/xDevSM-dApp).xDevSMsubmodule and the per-xAppsetup_imports.pysys.path shims; addxdevsm>=0.1.0to each xApp'srequirements.txt.xDevSM.imports to thexdevsm.*namespace.COPY xDevSMand the…/xDevSM/sm_framework/libentry fromLD_LIBRARY_PATH(encoders now load package-relative); bump the example images topython:3.11-slim-bullseye(xdevsmrequires ≥ 3.11; bullseye/glibc 2.31 loads the exec-stack encoders without an interpreter patch). Update README.Depends on
wineslab/xDevSM-dApp(thexdevsmpackage) being published to PyPI.🤖 Generated with Claude Code