Wireshark-commits: [Wireshark-commits] master-1.10 9afa5d6: Only nuke the startup item once we've i

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 7 Jun 2014 17:47:59 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=9afa5d6166790661ddc5b44e1e8cb0c2f18fa83e
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-1.10
Repository: wireshark

Commits:

9afa5d6 by Guy Harris (guy@xxxxxxxxxxxx):

    Only nuke the startup item once we've installed the launch daemon.
    
    That way, if we crash in the middle, there's still something installed
    that will try to set the permissions on the BPF devices.
    
    Change-Id: Ie0c32f9eaca08bdbb359d07e47f20c664bc66411
    Reviewed-on: https://code.wireshark.org/review/2023
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit 39ca18d71b85d0b99c0fd3c5edab48fa217d7d34)
    Reviewed-on: https://code.wireshark.org/review/2024
    (cherry picked from commit 9cf44c19d00e1072e28e353a4c8fcdd22e01e554)
    Reviewed-on: https://code.wireshark.org/review/2025
    

Actions performed:

    from  6487669   When installing the launch daemon, get rid of the old startup item.
    adds  9afa5d6   Only nuke the startup item once we've installed the launch daemon.


Summary of changes:
 packaging/macosx/Scripts/chmodbpf-postinstall.sh |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)