https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=262940

--- Comment #7 from Charlie Li <[email protected]> ---
Comment on attachment 232821
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=232821
Patch for libxml2 and py-libxml2

Generally good. Don't forget to bump all consumers.

Unless there are actual verifiable cases of confused consumers, we don't need
the post-install bit on xml2-config, the .pc and xml2Conf.sh, since the correct
${PREFIX}/include/libxml2 is already present despite the additional
/usr/include. This was not previously a problem.

Enabling the ICU option may cause downstream consumers to also link with it
despite anything set to the contrary (detected via pkg). Will need further
investigation though.

As long as the git patches apply cleanly, may want to use PATCH_SITES and
PATCHFILES for them.

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to