Skip to content

build: strip duplicate spaces after Libs.private: in libcurl.pc - #22544

Closed
vszakats wants to merge 4 commits into
curl:masterfrom
vszakats:am-strip-dupe-space-aft-colon
Closed

vszakats wants to merge 4 commits into
curl:masterfrom
vszakats:am-strip-dupe-space-aft-colon

Conversation

@vszakats

@vszakats vszakats commented Aug 11, 2026

Copy link
Copy Markdown
Member

When LIBCURL_PC_LDFLAGS_PRIVATE is empty and LIBCURL_PC_LIBS_PRIVATE
is not.

To keep it tidy.

To match CMake-generated ones and to keep it tidy.

Copilot AI 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.

Pull request overview

Normalizes spacing in Autotools-generated libcurl.pc files to align with CMake output.

Changes:

  • Strips duplicate spaces following colons.
  • Updates the cleanup comment accordingly.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread configure.ac Outdated
@vszakats vszakats changed the title configure: strip duplicate spaces after colons in libcurl.pc build: strip duplicate spaces after colons in libcurl.pc Aug 11, 2026
@vszakats
vszakats force-pushed the am-strip-dupe-space-aft-colon branch from d84ca6a to 2ec9fa2 Compare August 11, 2026 09:32

Copilot AI 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.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@vszakats vszakats changed the title build: strip duplicate spaces after colons in libcurl.pc build: strip duplicate spaces after Libs.private: in libcurl.pc Aug 11, 2026
@vszakats vszakats closed this in 2c22d30 Aug 11, 2026
@vszakats
vszakats deleted the am-strip-dupe-space-aft-colon branch August 11, 2026 09:57
vszakats added a commit that referenced this pull request Aug 12, 2026
To keep it tidy and to support properties in the template that only
appear in the generated `libcurl.pc` when filled.

E.g. `Required.private` may remain empty after #22548.

Also:
- GHA/windows: dump `libcurl.pc` to log in every job
- tidy up a pre-existing regex in the updated command.
  2c22d30 #22544

Ref: b9254da #22548
Cherry-picked from #22543

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

Labels

Development

Successfully merging this pull request may close these issues.

2 participants