aboutsummaryrefslogtreecommitdiffstats
path: root/Library/Homebrew
diff options
context:
space:
mode:
authorMike McQuaid2013-01-26 13:03:22 +0000
committerMike McQuaid2013-01-30 22:36:12 -0800
commit1da2d5dd92dbe9baf7d6e047bcb7f39f06e9c52e (patch)
tree578606a7d9895a249b15829a5595d7a180c11bd1 /Library/Homebrew
parentdedc6aec754e248127fed3d098e4614063a073e3 (diff)
downloadhomebrew-1da2d5dd92dbe9baf7d6e047bcb7f39f06e9c52e.tar.bz2
Remove MD5 support.
Closes #17317.
Diffstat (limited to 'Library/Homebrew')
-rw-r--r--Library/Homebrew/checksums.rb2
-rw-r--r--Library/Homebrew/cmd/audit.rb3
-rw-r--r--Library/Homebrew/extend/pathname.rb5
-rw-r--r--Library/Homebrew/test/test_checksums.rb18
-rw-r--r--Library/Homebrew/test/test_formula.rb3
5 files changed, 1 insertions, 30 deletions
diff --git a/Library/Homebrew/checksums.rb b/Library/Homebrew/checksums.rb
index defaa8bb1..28ca023fe 100644
--- a/Library/Homebrew/checksums.rb
+++ b/Library/Homebrew/checksums.rb
@@ -1,7 +1,7 @@
class Checksum
attr_reader :hash_type, :hexdigest
- TYPES = [:md5, :sha1, :sha256]
+ TYPES = [:sha1, :sha256]
def initialize type=:sha1, val=nil
@hash_type = type
diff --git a/Library/Homebrew/cmd/audit.rb b/Library/Homebrew/cmd/audit.rb
index 3951d35f8..d0300d5d6 100644
--- a/Library/Homebrew/cmd/audit.rb
+++ b/Library/Homebrew/cmd/audit.rb
@@ -244,13 +244,10 @@ class FormulaAuditor
next if cksum.nil?
len = case cksum.hash_type
- when :md5 then 32
when :sha1 then 40
when :sha256 then 64
end
- problem "md5 is broken, deprecated: use sha1 instead" if cksum.hash_type == :md5
-
if cksum.empty?
problem "#{cksum.hash_type} is empty"
else
diff --git a/Library/Homebrew/extend/pathname.rb b/Library/Homebrew/extend/pathname.rb
index 77719c984..7453f647b 100644
--- a/Library/Homebrew/extend/pathname.rb
+++ b/Library/Homebrew/extend/pathname.rb
@@ -216,11 +216,6 @@ class Pathname
incr_hash.hexdigest
end
- def md5
- require 'digest/md5'
- incremental_hash(Digest::MD5)
- end
-
def sha1
require 'digest/sha1'
incremental_hash(Digest::SHA1)
diff --git a/Library/Homebrew/test/test_checksums.rb b/Library/Homebrew/test/test_checksums.rb
index 880796281..62e9d5e61 100644
--- a/Library/Homebrew/test/test_checksums.rb
+++ b/Library/Homebrew/test/test_checksums.rb
@@ -12,24 +12,6 @@ class ChecksumTests < Test::Unit::TestCase
end
end
- def test_md5
- valid_md5 = TestBall.new
- valid_md5.stable.instance_eval do
- md5 '060844753f2a3b36ecfc3192d307dab2'
- end
-
- good_checksum valid_md5
- end
-
- def test_badmd5
- invalid_md5 = TestBall.new
- invalid_md5.stable.instance_eval do
- md5 '61aa838a9e4050d1876a295a9e62cbe6'
- end
-
- bad_checksum invalid_md5
- end
-
def test_sha1
valid_sha1 = TestBall.new
valid_sha1.stable.instance_eval do
diff --git a/Library/Homebrew/test/test_formula.rb b/Library/Homebrew/test/test_formula.rb
index 69583c15c..0836dddd9 100644
--- a/Library/Homebrew/test/test_formula.rb
+++ b/Library/Homebrew/test/test_formula.rb
@@ -110,11 +110,8 @@ class FormulaTests < Test::Unit::TestCase
assert_match /[0-9a-fA-F]{40}/, f.stable.checksum.hexdigest
assert_match /[0-9a-fA-F]{64}/, f.devel.checksum.hexdigest
- assert_nil f.stable.md5
assert_nil f.stable.sha256
- assert_nil f.bottle.md5
assert_nil f.bottle.sha256
- assert_nil f.devel.md5
assert_nil f.devel.sha1
assert_equal 1, f.stable.mirrors.length