Skip to content

chore: remove manual pthread link flag detection and LDFLAGS modifica…#360

Merged
kaorut merged 1 commit into
mainfrom
configure
May 4, 2026
Merged

chore: remove manual pthread link flag detection and LDFLAGS modifica…#360
kaorut merged 1 commit into
mainfrom
configure

Conversation

@kaorut
Copy link
Copy Markdown
Member

@kaorut kaorut commented May 4, 2026

…tion #359

Remove manual pthread link flag detection and LDFLAGS modification.

Rely on standard AX_PTHREAD substitutions for CC and LIBS, and exclude PTHREAD_CFLAGS from CPPFLAGS to avoid redundancy.

…tion #359

Remove manual pthread link flag detection and LDFLAGS modification.

Rely on standard AX_PTHREAD substitutions for CC and LIBS, and exclude PTHREAD_CFLAGS from CPPFLAGS to avoid redundancy.
@kaorut kaorut self-assigned this May 4, 2026
@kaorut kaorut added the task-bug Something isn't working label May 4, 2026
Copilot AI review requested due to automatic review settings May 4, 2026 02:26
@kaorut kaorut added this to the 1.10.0 milestone May 4, 2026
@kaorut kaorut linked an issue May 4, 2026 that may be closed by this pull request
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Autotools configuration to simplify pthread handling (per Issue #359) by removing custom link-flag detection and relying more directly on AX_PTHREAD outputs.

Changes:

  • Removed manual detection of -pthread/-pthreads in PTHREAD_CFLAGS and the corresponding LDFLAGS modification.
  • Kept AX_PTHREAD-provided substitutions for CC and LIBS.
  • Stopped appending PTHREAD_CFLAGS into CPPFLAGS to avoid redundancy.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread configure.ac
Copilot finished work on behalf of kaorut May 4, 2026 02:35
@kaorut kaorut merged commit 5f214fe into main May 4, 2026
26 checks passed
@kaorut kaorut deleted the configure branch May 4, 2026 02:42
@kaorut kaorut restored the configure branch May 4, 2026 02:42
@kaorut kaorut deleted the configure branch May 4, 2026 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

task-bug Something isn't working v1.9.2 v1.10.0

Development

Successfully merging this pull request may close these issues.

Correct configure.ac and release 1.9.2.

3 participants