Bug 254559 - www/aria2: Makefile cleanup, switch to expat, make libuv and static lib optional
Summary: www/aria2: Makefile cleanup, switch to expat, make libuv and static lib optional
Status: Open
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Po-Chuan Hsieh
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-03-25 21:38 UTC by daniel.engberg.lists
Modified: 2021-04-25 22:51 UTC (History)
1 user (show)

See Also:


Attachments
Patch for aria2 (3.36 KB, patch)
2021-03-25 21:38 UTC, daniel.engberg.lists
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description daniel.engberg.lists 2021-03-25 21:38:33 UTC
Created attachment 223583 [details]
Patch for aria2

Rearrange Makefile for readability and pet portlint
Switch xml library to expat, smaller and slightly faster
Add option to use libuv and make static lib optional and non default options

No functional change

Compile tested on FreeBSD 13.0-RC1 (amd64) (make, make check-plist)
Poudriere testport OK 12.2-RELEASE (amd64)
Poudriere testport OK 11.4-RELEASE (amd64)
Comment 1 daniel.engberg.lists 2021-04-14 08:17:02 UTC
Friendly ping
Comment 2 Alexey Dokuchaev freebsd_committer 2021-04-17 08:40:02 UTC
Over to maintainer.  FWIW, it does not look like "cleanup" at all, more like gratuitous shuffling Makefile lines around.  Please don't fix what ain't broken.
Comment 3 Po-Chuan Hsieh freebsd_committer 2021-04-25 21:18:55 UTC
upstream says libxml2 has precedence over expat.
Comment 4 daniel.engberg.lists 2021-04-25 22:51:57 UTC
That seems to more of a notice rather than preference. The instructions for Windows and Android do however specifically say expat and not libxml2 fwiw.
It says similar for SSL lib, "GnuTLS has precedence over OpenSSL" so I don't think there's anything to read into.