Skip to content

Added build script for utf8proc for python3.14 with ubi 10.2 - #8726

Open
shivansh-ibm wants to merge 4 commits into
ppc64le:masterfrom
shivansh-ibm:utf8proc_rt26.1
Open

Added build script for utf8proc for python3.14 with ubi 10.2#8726
shivansh-ibm wants to merge 4 commits into
ppc64le:masterfrom
shivansh-ibm:utf8proc_rt26.1

Conversation

@shivansh-ibm

Copy link
Copy Markdown
Contributor

Checklist

  • Have you checked and followed all the points mention in the CONTRIBUTING.MD
  • Have you validated script on UBI 9 container
  • Did you run the script(s) on fresh container with set -e option enabled and observe success ?
  • Did you have Legal approvals for patch files ?

Comment thread u/utf8proc/utf8proc_ubi_10.2.sh Outdated
cmake --build . --target install
# cd $WORK_DIR
cd ..
pip3.14 install setuptools

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use python3.14 -m pip install setuptools instead of pip3.14 install setuptools to fix the gha build failure.
https://github.com/ppc64le/build-scripts/actions/runs/33624677436/job/100230254747?pr=8726#:~:text=temp_build_script.sh%3A%20line,utf8proc_ubi_10.2.sh%20v2.6.1

@shivansh-ibm

Copy link
Copy Markdown
Contributor Author

The package build is passing successfully on UBI 10.2. However, during the test phase, the script fails to find the libutf8proc.so file. These files are generated under the utf8proc/lib64 directory during the utf8proc build.

The current pyproject.toml that we download using wget does not include the lib64 directory in the wheel package. As a result, the built wheel don't have libutfproc.so with current pyproject.toml

I have updated the pyproject.toml to include the lib64 directory as part of the wheel package. Once the PR is merged, the test script should work as expected.

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