Skip to content

Don't set python compile flags when python is not included#53781

Draft
pgimalac wants to merge 3 commits into
mainfrom
pgimalac/python-compile-flags-no-build-tag
Draft

Don't set python compile flags when python is not included#53781
pgimalac wants to merge 3 commits into
mainfrom
pgimalac/python-compile-flags-no-build-tag

Conversation

@pgimalac

Copy link
Copy Markdown
Member

What does this PR do?

Do not set python related compiler / linker flags when python is not included in Go build tags.

Motivation

  • simplify flags / environment variables when python is not used
  • do not pass -lpython (ie. linking with libpython) for trace-agent on AIX
  • make cross compilation easier

Describe how you validated your changes

CI

Additional Notes

@pgimalac pgimalac added changelog/no-changelog No changelog entry needed qa/done QA done before merge and regressions are covered by tests labels Jul 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

@codex review

@dd-octo-sts dd-octo-sts Bot added internal Identify a non-fork PR team/agent-devx labels Jul 17, 2026
@github-actions github-actions Bot added the medium review PR review might take time label Jul 17, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f4701aa0b7

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread tasks/libs/common/utils.py
@datadog-datadog-us1-prod

datadog-datadog-us1-prod Bot commented Jul 17, 2026

Copy link
Copy Markdown

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 52.01% (+0.12%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 0c7e61d | Docs | Datadog PR Page | Give us feedback!

@dd-octo-sts

dd-octo-sts Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Files inventory check summary

File checks results against ancestor 343715bd:

Results for datadog-agent_7.83.0~devel.git.190.0c7e61d.pipeline.125234477-1_amd64.deb:

No change detected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/no-changelog No changelog entry needed internal Identify a non-fork PR medium review PR review might take time qa/done QA done before merge and regressions are covered by tests team/agent-devx team/agent-runtimes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant