Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

harfbuzz/10.2.0 package update #39343

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

octo-sts[bot]
Copy link
Contributor

@octo-sts octo-sts bot commented Jan 11, 2025

@octo-sts octo-sts bot added request-version-update request for a newer version of a package automated pr labels Jan 11, 2025
Copy link
Contributor Author

octo-sts bot commented Jan 11, 2025

⚠️ EXPERIMENTAL

Please use 👍 or 👎 on this comment to indicate if you agree or disagree with the recommendation.

To provide more detailed feedback please comment on the recommendation prefixed with /ai-verify:

e.g. /ai-verify partially helpful but I also added bash to the build environment

Gen AI suggestions to solve the build error:

Based on the error output, I'll provide a structured analysis and solution:

• Detected Error: src/meson.build:3:15: ERROR: Command `/home/build/src/gen-hb-version.py 10.2.0 /home/build/output/src/hb-version.h /home/build/src/hb-version.h.in` failed with status 127

• Error Category: Build Configuration

• Failure Point: Python script execution during Meson configuration phase

• Root Cause Analysis: Status code 127 typically indicates "command not found" which suggests Python execution environment issues. The gen-hb-version.py script can't execute properly despite being found by Meson.

• Suggested Fix:
Add Python 3 as a build dependency in the environment section:

environment:
  contents:
    packages:
      - python3
      # ... existing packages ...

• Explanation: The harfbuzz build system requires Python 3 to generate version-specific headers. Status 127 indicates the Python interpreter isn't available in the build environment despite the script being present.

• Additional Notes:

  1. Python is required for the build-time version header generation
  2. The script was found but couldn't execute, indicating missing runtime
  3. No need to modify the script path as Meson found it correctly
  4. Adding Python3 as a build dependency resolves the execution environment

• References:

@octo-sts octo-sts bot added the ai/skip-comment Stop AI from commenting on PR label Jan 11, 2025
the AI suggested comment was right this time, the build was invoking a
python script during build and python was not available, adding python3
builds the package.

Signed-off-by: kranurag7 <[email protected]>
@octo-sts octo-sts bot added bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. manual/review-needed labels Jan 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ai/skip-comment Stop AI from commenting on PR automated pr bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. manual/review-needed request-version-update request for a newer version of a package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants