Don't use deprecated form of attr
The form "attr :name, true" is deprecated and causes Ruby to emit a
warning in verbose mode. Using attr_{reader,writer,accessor} is more
clear anyway, so do so.
This commit is contained in:
parent
baffb31aab
commit
f172d3a6eb
@ -73,9 +73,7 @@ class FormulaText
|
||||
end
|
||||
|
||||
class FormulaAuditor
|
||||
attr :f
|
||||
attr :text
|
||||
attr :problems, true
|
||||
attr_reader :f, :text, :problems
|
||||
|
||||
BUILD_TIME_DEPS = %W[
|
||||
autoconf
|
||||
|
||||
@ -71,12 +71,8 @@ module Homebrew extend self
|
||||
end
|
||||
|
||||
class FormulaCreator
|
||||
attr :url
|
||||
attr :sha1
|
||||
attr :name, true
|
||||
attr :version, true
|
||||
attr :path, true
|
||||
attr :mode, true
|
||||
attr_reader :url, :sha1
|
||||
attr_accessor :name, :version, :path, :mode
|
||||
|
||||
def url= url
|
||||
@url = url
|
||||
|
||||
@ -3,7 +3,7 @@ class FormulaUnspecifiedError < UsageError; end
|
||||
class KegUnspecifiedError < UsageError; end
|
||||
|
||||
class MultipleVersionsInstalledError < RuntimeError
|
||||
attr :name
|
||||
attr_reader :name
|
||||
|
||||
def initialize name
|
||||
@name = name
|
||||
@ -14,7 +14,7 @@ end
|
||||
class NotAKegError < RuntimeError; end
|
||||
|
||||
class NoSuchKegError < RuntimeError
|
||||
attr :name
|
||||
attr_reader :name
|
||||
|
||||
def initialize name
|
||||
@name = name
|
||||
@ -23,8 +23,8 @@ class NoSuchKegError < RuntimeError
|
||||
end
|
||||
|
||||
class FormulaUnavailableError < RuntimeError
|
||||
attr :name
|
||||
attr :dependent, true
|
||||
attr_reader :name
|
||||
attr_accessor :dependent
|
||||
|
||||
def dependent_s
|
||||
"(dependency of #{dependent})" if dependent and dependent != name
|
||||
@ -59,7 +59,7 @@ end
|
||||
|
||||
module Homebrew
|
||||
class InstallationError < RuntimeError
|
||||
attr :formula
|
||||
attr_reader :formula
|
||||
|
||||
def initialize formula, message=""
|
||||
super message
|
||||
@ -87,7 +87,7 @@ class UnsatisfiedDependencyError < Homebrew::InstallationError
|
||||
end
|
||||
|
||||
class UnsatisfiedRequirements < Homebrew::InstallationError
|
||||
attr :reqs
|
||||
attr_reader :reqs
|
||||
|
||||
def initialize formula, reqs
|
||||
@reqs = reqs
|
||||
@ -99,9 +99,7 @@ class UnsatisfiedRequirements < Homebrew::InstallationError
|
||||
end
|
||||
|
||||
class BuildError < Homebrew::InstallationError
|
||||
attr :exit_status
|
||||
attr :command
|
||||
attr :env
|
||||
attr_reader :exit_status, :command, :env
|
||||
|
||||
def initialize formula, cmd, args, es
|
||||
@command = cmd
|
||||
@ -159,10 +157,8 @@ end
|
||||
|
||||
# raised by Pathname#verify_checksum when verification fails
|
||||
class ChecksumMismatchError < RuntimeError
|
||||
attr :advice, true
|
||||
attr :expected
|
||||
attr :actual
|
||||
attr :hash_type
|
||||
attr_accessor :advice
|
||||
attr_reader :expected, :actual, :hash_type
|
||||
|
||||
def initialize expected, actual
|
||||
@expected = expected
|
||||
|
||||
@ -6,12 +6,9 @@ require 'bottles'
|
||||
require 'caveats'
|
||||
|
||||
class FormulaInstaller
|
||||
attr :f
|
||||
attr :tab, true
|
||||
attr :options, true
|
||||
attr :show_summary_heading, true
|
||||
attr :ignore_deps, true
|
||||
attr :show_header, true
|
||||
attr_reader :f
|
||||
attr_accessor :tab, :options, :ignore_deps
|
||||
attr_accessor :show_summary_heading, :show_header
|
||||
|
||||
def initialize ff
|
||||
@f = ff
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user