Wireshark-commits: [Wireshark-commits] master 451a241: Add c-ares to the required library list.
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 23 Nov 2019 22:46:05 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=451a241e50bd34ba097bd09a5f9f673170779aaf
Submitter: "Gerald Combs <gerald@xxxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

451a241 by Gerald Combs (gerald@xxxxxxxxxxxxx):

    Add c-ares to the required library list.
    
    Although c-ares support was techically optional, it was either on by
    default or required in all of our packaging. Go ahead and require it
    globally. C-ares is widely available and synchronous name resolution can
    easily result in a horrific user experience.
    
    Change-Id: Id67c797316ed6b8a0ab5052e55a43a1b9e2a2464
    Reviewed-on: https://code.wireshark.org/review/35188
    Petri-Dish: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Gerald Combs <gerald@xxxxxxxxxxxxx>
    

Actions performed:

    from  6a0b558   tools: remove obsolete VS2015 deployment file
     add  451a241   Add c-ares to the required library list.


Summary of changes:
 CMakeLists.txt                  |  6 ++--
 cmakeconfig.h.in                |  3 --
 docbook/release-notes.adoc      |  1 +
 epan/addr_resolv.c              | 75 +++--------------------------------------
 epan/epan.c                     |  6 ----
 packaging/rpm/wireshark.spec.in | 11 ++----
 sharkd_session.c                |  2 --
 tools/alpine-setup.sh           |  2 +-
 tools/bsd-setup.sh              |  4 +--
 tools/debian-setup.sh           |  2 +-
 10 files changed, 16 insertions(+), 96 deletions(-)