Skip to content

formula: add htacg/tidy-html5 - #526

Open
MeteorsLiu wants to merge 1 commit into
xgo-dev:mainfrom
MeteorsLiu:issue/246-tidy-html5-luna
Open

formula: add htacg/tidy-html5#526
MeteorsLiu wants to merge 1 commit into
xgo-dev:mainfrom
MeteorsLiu:issue/246-tidy-html5-luna

Conversation

@MeteorsLiu

Copy link
Copy Markdown
Collaborator

Translate the pinned Conan Center tidy-html5 5.8.0 recipe into an idiomatic LLAR Formula.

  • Builds and installs the upstream CMake project through LLAR's CMake helper with the verified Conan options and pinned PDB patch.
  • Publishes relocatable tidy.pc metadata and verifies the Conan consumer against the complete pkg-config lookup on fresh and cache-hit builds.
  • Uses 5.8.0 as the earliest compatible fromVer; 5.6.0 was rejected because its installed static library is libtidys.a rather than libtidy.a.

Closes #246

@fennoai fennoai Bot left a comment

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.

Reviewed the complete tidy-html5 Formula unit, including version selection, option filtering, CMake configuration and patching, install cleanup, pkg-config metadata, licensing, and the consumer test. The packaging contract is coherent and I found no actionable correctness or compatibility regressions in the changed code.

Verification performed locally:

  • llar test --verbose ./htacg/tidy-html5@5.8.0 --os linux --arch amd64
  • llar test --verbose ./htacg/tidy-html5 --os linux --arch amd64
  • llar test --verbose ./htacg/tidy-html5 --os linux --arch amd64 --option shared=ON

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.

Translate Conan Center tidy-html5 recipe to LLAR

1 participant