Wireshark-commits: [Wireshark-commits] master 33c836a: Builds require CMake, so always install it.
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 14 May 2018 22:08:12 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=33c836af9fdad2c099966d9b9aee9c546d068f70
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

33c836a by Guy Harris (guy@xxxxxxxxxxxx):

    Builds require CMake, so always install it.
    
    Don't have an option not to install it.  (If we want to avoid installing
    it when it's already been installed by something other than this script,
    we should check for its existence before installing, and skip the
    installation step for it.)
    
    Get rid of the instructions for autotools builds; always show the
    instructions for CMake builds.
    
    We wouldn't get to the uninstall stage if it hadn't been installed by
    this script; remove the comment asking about that.
    
    Change-Id: I276ee96bf955ef4ff33dea87bc27c21111301ea0
    Reviewed-on: https://code.wireshark.org/review/27540
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  548f754   changing Author's email for Eugene Adell
    adds  33c836a   Builds require CMake, so always install it.


Summary of changes:
 tools/macos-setup.sh | 47 ++++++++++++-----------------------------------
 1 file changed, 12 insertions(+), 35 deletions(-)