Bug 228712 - www/aria2: add asynchronous DNS support via libcares
Summary: www/aria2: add asynchronous DNS support via libcares
Status: Closed FIXED
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: 2018-06-02 23:18 UTC by yorik.sar+rerFlEJcdSDuqrOxzVrAbdzsrs
Modified: 2018-06-04 16:36 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (sunpoet)


Attachments
patch to add async DNS support (1.19 KB, patch)
2018-06-02 23:18 UTC, yorik.sar+rerFlEJcdSDuqrOxzVrAbdzsrs
no flags Details | Diff
Proposed patch (since 471508 revision) (594 bytes, patch)
2018-06-04 14:45 UTC, lightside
lightside: maintainer-approval? (sunpoet)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description yorik.sar+rerFlEJcdSDuqrOxzVrAbdzsrs 2018-06-02 23:18:17 UTC
Created attachment 193952 [details]
patch to add async DNS support

I've ran into necessity to use async DNS feature, but it was disabled at compile time, from the looks of it from the dawn of ages.

Attached patch seems to be working fine for me.
Comment 1 commit-hook freebsd_committer freebsd_triage 2018-06-03 18:17:43 UTC
A commit references this bug:

Author: sunpoet
Date: Sun Jun  3 18:17:29 UTC 2018
New revision: 471508
URL: https://svnweb.freebsd.org/changeset/ports/471508

Log:
  Add CARES option

  PR:		228712
  Submtited by:	<yorik.sar@gmail.com>

Changes:
  head/www/aria2/Makefile
Comment 2 Po-Chuan Hsieh freebsd_committer freebsd_triage 2018-06-03 18:18:10 UTC
Committed. Thanks!
Comment 3 yorik.sar+rerFlEJcdSDuqrOxzVrAbdzsrs 2018-06-03 18:27:58 UTC
Thanks for a swift reply! I didn't know there's already an option defined for libcares. Cool :)
Comment 4 lightside 2018-06-04 14:45:23 UTC
Created attachment 193995 [details]
Proposed patch (since 471508 revision)

(In reply to comment #1)
There is --without-libcares configure option:
-8<--
% ./configure --help | grep ares
  --without-libcares      Do not use libcares. [default=check]
-->8-

The attachment #193952 [details] contains correct option's name.
Attached patch after ports r471508 changes.

- Rename libares to libcares for CARES_CONFIGURE_WITH variable
Comment 5 lightside 2018-06-04 14:55:22 UTC
(In reply to comment #4)
How to check:
-8<--
% sh -c "make WITHOUT=CARES clean configure 2>&1" | grep ^LibCares
LibCares:       no (CFLAGS='' LIBS='')
% sh -c "make WITH=CARES clean configure 2>&1" | grep ^LibCares
LibCares:       yes (CFLAGS='-I/usr/local/include ' LIBS='-L/usr/local/lib -lcares ')
% make clean
<..>
-->8-
Comment 6 Po-Chuan Hsieh freebsd_committer freebsd_triage 2018-06-04 16:36:20 UTC
(In reply to lightside from comment #4)

Fixed in r471562. Thanks!