Browse files

Revert "issue 4427: bpf device permission change option added"

This reverts commit 56a8f51.
  • Loading branch information...
1 parent 56a8f51 commit 2f40e2c90a5d0e03c018049f827fab599f171084 @lorensr committed Mar 2, 2011
Showing with 2 additions and 19 deletions.
  1. +2 −19 Library/Formula/wireshark.rb
View
21 Library/Formula/wireshark.rb
@@ -1,6 +1,6 @@
require 'formula'
-class Wireshark < Formula
+class Wireshark <Formula
url 'http://media-2.cacetech.com/wireshark/src/wireshark-1.4.3.tar.bz2'
md5 'ac3dcc8c128c38d9ef3d9c93d1dec83e'
homepage 'http://www.wireshark.org'
@@ -11,18 +11,7 @@ class Wireshark < Formula
depends_on 'gtk+' if ARGV.include? "--with-x"
def options
- [
- ["--with-x", "Include X11 support"],
- ['--with-daemon', 'Include launch daemon that allows admin access to capture devices']
- ]
- end
-
- def caveats
- if ARGV.include? '--with-daemon'
- 'Launch daemon installation program initiated.'
- else
- "If your list of available capture interfaces is empty (behavior on default OS X installation), uninstall and reinstall Wireshark with the '--with-daemon' option. This adds a launch daemon that changes the permissions of your BPF devices so that all users in the 'admin' group - all users with 'Allow user to administer this computer' turned on - have both read and write access to those devices."
- end
+ [["--with-x", "Include X11 support"]]
end
def install
@@ -35,12 +24,6 @@ def install
system "make"
ENV.j1 # Install failed otherwise.
system "make install"
-
- if ARGV.include? '--with-daemon'
- system 'wget https://bugs.wireshark.org/bugzilla/attachment.cgi?id=3373 -O ChmodBPF.tar.gz'
- system 'tar zxvf ChmodBPF.tar.gz'
- system 'open ChmodBPF/Install\ ChmodBPF.app'
- end
end
end

0 comments on commit 2f40e2c

Please sign in to comment.