Skip to content

add C++ bindings for aarch64 - #13

Open
pranabdas wants to merge 9 commits into
devfrom
feat/SOF-7923
Open

add C++ bindings for aarch64#13
pranabdas wants to merge 9 commits into
devfrom
feat/SOF-7923

Conversation

@pranabdas

Copy link
Copy Markdown
Member

No description provided.

Comment thread scripts/build-pbs-c-bindings.sh Outdated
TORQUE_INCLUDE="${TORQUE_INCLUDE:-/usr/local/include/torque}"
TORQUE_LIBDIR="${TORQUE_LIBDIR:-/usr/local/lib64}"
OUT_DIR="${OUT_DIR:-dist}"
SURFSARA_REPO="${SURFSARA_REPO:-https://github.com/mat3ra/surfsara-pbs-python.git}"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There seem to be some convoluted logic for building this - can it be decoupled?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Comment thread scripts/build-pbs-c-bindings.sh Outdated
@@ -0,0 +1,92 @@
#!/usr/bin/env bash

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

How are the so files generated? Should this be done in a GH WF?

Comment thread scripts/build-pbs-c-bindings.sh Outdated
@@ -0,0 +1,92 @@
#!/usr/bin/env bash
# Build _pbs.cpython-310-<arch>-linux-gnu.so against installed Torque (5.x wrap).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why Torque 5?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The directory in https://github.com/mat3ra/surfsara-pbs-python is named torque 5.x, but building c-bindings work with the current torque v6.1.3 headers.

@pranabdas pranabdas changed the title build pbs c bindings via gihub ci add C++ bindings for aarch64 Jul 30, 2026
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