[Bug 256078] textproc/libxml2: Update to 2.9.12 - WIP

From: <bugzilla-noreply_at_freebsd.org>
Date: Sat, 22 May 2021 11:38:02 UTC
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=256078

            Bug ID: 256078
           Summary: textproc/libxml2: Update to 2.9.12 - WIP
           Product: Ports & Packages
           Version: Latest
          Hardware: Any
               URL: http://www.xmlsoft.org/news.html
                OS: Any
            Status: New
          Severity: Affects Only Me
          Priority: ---
         Component: Individual Port(s)
          Assignee: desktop@FreeBSD.org
          Reporter: daniel.engberg.lists@pyret.net
             Flags: maintainer-feedback?(desktop@FreeBSD.org)
          Assignee: desktop@FreeBSD.org

Created attachment 225170
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=225170&action=edit
Patch for libxml2 (WIP)

Hi,

I've been hacking on libxml2 a bit including converting it to CMake and would
like some input, there are some differences between current version and the new
version that I'm not sure about on how to handle.

./work/stage/usr/local/lib/xml2Conf.sh
 -- CMAKE --
prefix is expanded
XML2_LIBS="-lxml2 -lz -llzma -pthread   -lm"

 -- Autotools --
XML2_LIBS="-lxml2 -lz  -L/usr/lib -llzma  -L/usr/lib  -lm"


./work/stage/usr/local/libdata/pkgconfig/libxml-2.0.pc
 -- CMAKE --
Libs.private:  -pthread -lz -llzma  -lm

 -- Autotools --
Libs.private:  -lz  -L/usr/lib -llzma -L/usr/lib -lm


Questions:
Is -L/usr/lib needed? From what I can tell CMake doesn't have a variable for
this, suggestions are welcome. In worst case we can hardcode it.

Current version in ports forces -DPIC cflag, is this needed?
https://cgit.freebsd.org/ports/tree/textproc/libxml2/files/patch-Makefile.in

make test passes on 13-STABLE (amd64)

Motivation for moving to CMake is mainly because it's faster and much easier to
debug, I do plan to upstream as many changes as possible but perhaps it's
futile effort?

I'm not sure how to handle py-libxml2 in a sane way. :/

Note for users:
https://gitlab.gnome.org/GNOME/libxslt/-/issues/51
https://git.alpinelinux.org/aports/diff/main/libxslt/APKBUILD?id=3ec3aafa689b134df3f999baf8841099c6e8c422

Changes for 2.9.12
https://gitlab.gnome.org/GNOME/libxml2/-/commit/b48e77cf4f6fa0792c5f4b639707a2b0675e461b

Thanks in advance,
Daniel

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