autobuild 1.20.0 → 1.23.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3b347c81cc5ac09e4f967df3dd5f53bca82fb3ba2a1a4a2e76b1b2d20ba19869
-  data.tar.gz: 6eaa54db5306f069ce5e9809b708df1bb4c5def84dc1553c953a3e38319cbf26
+  metadata.gz: 6431400922a5accc7c11c921745648a3a7fa65e92023bc700b26e6d975bd2d6a
+  data.tar.gz: bd29297ec7e7902e067a08ecb9ea4837a051be82b7db7c6ca40c207657d587bc
 SHA512:
-  metadata.gz: ccefa69f82809031debaeee77adde985c2de4d88c738e9b439cf7e839bc511f2f4c757486625f88a91541d97c16566217a544831c35924ea76cae98920d9f59d
-  data.tar.gz: 9f4b9b5c38e9072104dc052c630ca7a49a915331526cdbc624d2ef725eef44987b97b97667ebcafead9afb145fcbb3cee5a921f699aed101ec99efd611c28352
+  metadata.gz: 726b002c5f1e6a51efe77397546762ce1aa50e546bddecaec4a379a69943646e414e6c0d7ae8a470d29be982a2a50541be800ae40d30a6206723185d1937e30a
+  data.tar.gz: 95088358f807bb6edfd52746e8c30ed85b52b05f2318c09be17fe05b45484f554205d48d280094dc378e322b26ae468a2b5ad72468c34b9b369e4d0aa085a5fa

data/.github/workflows/lint.yml

@@ -0,0 +1,25 @@
+name: Rubocop
+
+on: [pull_request]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        ruby-version: ["2.7", "2.6", "2.5"]
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby ${{ matrix.ruby-version }}
+    # To automatically get bug fixes and new Ruby versions for ruby/setup-ruby,
+    # change this to (see https://github.com/ruby/setup-ruby#versioning):
+    # uses: ruby/setup-ruby@v1
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby-version }}
+    - name: Install dependencies
+      run: bundle install
+    - name: Run rubocop
+      run: bundle exec rubocop

data/.github/workflows/test.yml

@@ -0,0 +1,30 @@
+name: Unit Tests
+
+on: [pull_request]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        ruby-version: ["2.7", "2.6", "2.5"]
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby ${{ matrix.ruby-version }}
+    # To automatically get bug fixes and new Ruby versions for ruby/setup-ruby,
+    # change this to (see https://github.com/ruby/setup-ruby#versioning):
+    # uses: ruby/setup-ruby@v1
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby-version }}
+    - name: Install dependencies
+      run: bundle install
+    - name: Run tests
+      run: bundle exec rake test
+      env:
+        GIT_AUTHOR_NAME: autobuild CI Git Identity
+        GIT_AUTHOR_EMAIL: autobuild@github.actions
+        GIT_COMMITTER_NAME: autobuild CI Git Identity
+        GIT_COMMITTER_EMAIL: autobuild@github.actions

data/.rubocop.yml

@@ -2,7 +2,7 @@ inherit_gem:
   rubocop-rock: defaults.yml
 
 AllCops:
-    TargetRubyVersion: "2.3"
+    TargetRubyVersion: "2.5"
     Exclude:
     - lib/autobuild/packages/genom.rb
     - vendor/**/*
@@ -17,9 +17,6 @@ Style/TrivialAccessors:
 Naming/PredicateName:
     Enabled: false
 
-Lint/SplatKeywordArguments:
-    Enabled: false
-
 Style/FrozenStringLiteralComment:
     Enabled: false
 
@@ -33,7 +30,7 @@ Naming/FileName:
 
 
 
-Layout/AlignParameters:
+Layout/ParameterAlignment:
     Enabled: false
 
 Layout/DotPosition:
@@ -42,7 +39,7 @@ Layout/DotPosition:
 Layout/FirstParameterIndentation:
     Enabled: false
 
-Layout/IndentAssignment:
+Layout/AssignmentIndentation:
     Enabled: false
 
 Layout/IndentationWidth:
@@ -96,7 +93,7 @@ Style/PerlBackrefs:
 Style/StringLiterals:
     Enabled: false
 
-Layout/IndentHeredoc:
+Layout/HeredocIndentation:
     Enabled: false
 
 Metrics/LineLength:
@@ -104,4 +101,14 @@ Metrics/LineLength:
     Exclude:
     - test/**/*
 
+Style/StringConcatenation:
+    Enabled: false
+
+Style/StderrPuts:
+    Enabled: false
+
+Style/GlobalStdStream:
+    Enabled: false
 
+Style/OptionalBooleanParameter:
+    Enabled: false

data/autobuild.gemspec

@@ -5,7 +5,7 @@ require 'autobuild/version'
 Gem::Specification.new do |s|
     s.name = "autobuild"
     s.version = Autobuild::VERSION
-    s.required_ruby_version = '>= 2.3.0'
+    s.required_ruby_version = '>= 2.5.0'
     s.authors = ["Sylvain Joyeux"]
     s.email = "sylvain.joyeux@m4x.org"
     s.summary = "Library to handle build systems and import mechanisms"
@@ -23,11 +23,12 @@ Gem::Specification.new do |s|
     s.files = `git ls-files -z`.split("\x0")
         .reject { |f| f.match(%r{^(test|spec|features)/}) }
 
-    s.add_runtime_dependency 'pastel', '~> 0.7.0', '>= 0.7.0'
-    s.add_runtime_dependency "rake", "~> 12.0", ">= 12.3.0"
-    s.add_runtime_dependency 'tty-cursor', '~> 0.5.0'
-    s.add_runtime_dependency 'tty-prompt', '~> 0.15.0'
-    s.add_runtime_dependency 'tty-screen', '~> 0.6.4'
+    s.add_runtime_dependency "concurrent-ruby", "~> 1.1"
+    s.add_runtime_dependency "pastel", "~> 0.7.0"
+    s.add_runtime_dependency "rake", "~> 13.0"
+    s.add_runtime_dependency 'tty-cursor', '~> 0.7.0'
+    s.add_runtime_dependency 'tty-prompt', '~> 0.21.0'
+    s.add_runtime_dependency 'tty-screen', '~> 0.8.0'
     s.add_runtime_dependency "utilrb", "~> 3.0", ">= 3.0"
     s.add_development_dependency "fakefs"
     s.add_development_dependency "flexmock", '~> 2.0', ">= 2.0.0"

data/bin/autobuild

@@ -2,7 +2,7 @@
 
 begin
     require 'daemons'
-rescue LoadError # rubocop:disable Lint/HandleExceptions
+rescue LoadError # rubocop:disable Lint/SuppressedException
 end
 
 require 'autobuild'

data/lib/autobuild/build_logfile.rb

@@ -6,8 +6,7 @@ module Autobuild
     class BuildLogfile
         Entry = Struct.new :package, :phase, :start_time, :duration
 
-        attr_reader :by_package
-        attr_reader :by_phase
+        attr_reader :by_package, :by_phase
 
         def initialize(entries = Array.new)
             @entries = entries.dup

data/lib/autobuild/config.rb

@@ -198,7 +198,7 @@ module Autobuild
                 opts.separator ""
                 opts.separator "General behaviour"
                 opts.on('--nice NICE', Integer,
-                    'nice the subprocesses to the given value') do |v|
+                        'nice the subprocesses to the given value') do |v|
                     Autobuild.nice = v
                 end
                 opts.on("-h", "--help", "Show this message") do
@@ -253,7 +253,7 @@ module Autobuild
                 opts.separator ""
                 opts.separator "Mail reports"
                 opts.on("--mail-from EMAIL", String,
-                    "From: field of the sent mails") do |from_email|
+                        "From: field of the sent mails") do |from_email|
                     mail[:from] = from_email
                 end
                 opts.on("--mail-to EMAILS", String, "comma-separated list of emails "\
@@ -262,7 +262,7 @@ module Autobuild
                     mail[:to] += emails.split(',')
                 end
                 opts.on("--mail-subject SUBJECT", String,
-                    "Subject: field of the sent mails") do |subject_email|
+                        "Subject: field of the sent mails") do |subject_email|
                     mail[:subject] = subject_email
                 end
                 opts.on("--mail-smtp HOSTNAME", String, "address of the mail server "\
@@ -301,7 +301,7 @@ module Autobuild
 
     def self.apply(packages, buildname = "autobuild", phases = [], options = Hash.new)
         options = Kernel.validate_options options,
-            parallel: Autobuild.parallel_build_level
+                                          parallel: Autobuild.parallel_build_level
 
         if Autobuild.mail[:to]
             if !Autobuild::HAS_RMAIL
@@ -361,7 +361,7 @@ module Autobuild
                             yield(task.package, phase) if package_tasks[task.name]
                         end
                     else
-                        proc { }
+                        proc {}
                     end
 
                 invoker.invoke_parallel([Rake::Task["#{buildname}-#{phase}"]],

data/lib/autobuild/environment.rb

@@ -29,37 +29,44 @@ module Autobuild
 
     SHELL_VAR_EXPANSION =
         if windows? then "%%%s%%".freeze
-        else "$%s".freeze
+        else
+            "$%s".freeze
         end
     SHELL_SET_COMMAND =
         if windows? then "set %s=%s".freeze
-        else "%s=\"%s\"".freeze
+        else
+            "%s=\"%s\"".freeze
         end
     SHELL_CONDITIONAL_SET_COMMAND =
         if windows? then "set %s=%s".freeze
-        else "if test -z \"$%1$s\"; then\n  %1$s=\"%3$s\"\n"\
+        else
+            "if test -z \"$%1$s\"; then\n  %1$s=\"%3$s\"\n"\
             "else\n  %1$s=\"%2$s\"\nfi".freeze
         end
     SHELL_UNSET_COMMAND = "unset %s".freeze
     SHELL_EXPORT_COMMAND =
         if windows? then "set %s".freeze
-        else "export %s".freeze
+        else
+            "export %s".freeze
         end
     SHELL_SOURCE_SCRIPT =
         if windows? then "%s".freeze
-        else '. "%s"'.freeze
+        else
+            '. "%s"'.freeze
         end
 
     LIBRARY_PATH =
         if macos? then 'DYLD_LIBRARY_PATH'.freeze
         elsif windows? then 'PATH'.freeze
-        else 'LD_LIBRARY_PATH'.freeze
+        else
+            'LD_LIBRARY_PATH'.freeze
         end
 
     LIBRARY_SUFFIX =
         if macos? then 'dylib'
         elsif windows? then 'dll'
-        else 'so'
+        else
+            'so'
         end
 
     ORIGINAL_ENV = Hash.new
@@ -89,10 +96,7 @@ module Autobuild
         # path separator (File::PATH_SEPARATOR)
         attr_reader :environment
 
-        attr_reader :inherited_variables
-
-        attr_reader :system_env
-        attr_reader :original_env
+        attr_reader :inherited_variables, :system_env, :original_env, :target_arch
 
         # The set of environment variables that are known to hold paths on the
         # filesystem
@@ -162,7 +166,8 @@ module Autobuild
                 inherited_environment.delete(name)
                 init_from_env(name)
             else
-                environment.keys.each do |env_key|
+                keys = environment.keys # get keys first to avoid delete-while-iterating
+                keys.each do |env_key|
                     reset(env_key)
                 end
             end
@@ -180,7 +185,8 @@ module Autobuild
                 environment[name] = nil
                 inherited_environment[name] = nil
             else
-                environment.keys.each do |env_key|
+                keys = environment.keys # get keys first to avoid delete-while-iterating
+                keys.each do |env_key|
                     clear(env_key)
                 end
             end
@@ -212,7 +218,8 @@ module Autobuild
             if @inherit
                 if name
                     @inherited_variables.include?(name)
-                else true
+                else
+                    true
                 end
             end
         end
@@ -228,7 +235,9 @@ module Autobuild
         # @see inherit? inherit
         def inherit=(value)
             @inherit = value
-            inherited_environment.keys.each do |env_name|
+            # get keys first to avoid modify-while-iterating
+            keys = inherited_environment.keys
+            keys.each do |env_name|
                 init_from_env(env_name)
             end
         end
@@ -244,7 +253,8 @@ module Autobuild
             flag =
                 if !names.last.respond_to?(:to_str)
                     names.pop
-                else true
+                else
+                    true
                 end
 
             if flag
@@ -328,7 +338,7 @@ module Autobuild
                     end
             end
             options = Kernel.validate_options options,
-                inheritance_mode: :expand
+                                              inheritance_mode: :expand
             inheritance_mode = options[:inheritance_mode]
 
             if !include?(name)
@@ -341,7 +351,8 @@ module Autobuild
                         inherited_environment[name] || []
                     elsif inheritance_mode == :keep && inherit?(name)
                         ["$#{name}"]
-                    else []
+                    else
+                        []
                     end
 
                 value = []
@@ -438,7 +449,8 @@ module Autobuild
             if file
                 @source_before << { file: file, shell: shell }
                 source_before(shell: shell) # for backwards compatibility
-            else @source_before.select { |pair| pair[:shell] == shell }
+            else
+                @source_before.select { |pair| pair[:shell] == shell }
                                .map { |item| item[:file] }
             end
         end
@@ -456,7 +468,8 @@ module Autobuild
             if file
                 @source_after << { file: file, shell: shell }
                 source_after(shell: shell) # for backwards compatibility
-            else @source_after.select { |pair| pair[:shell] == shell }
+            else
+                @source_after.select { |pair| pair[:shell] == shell }
                               .map { |item| item[:file] }
             end
         end
@@ -511,8 +524,8 @@ module Autobuild
             end
             export.update.each do |name, (with_inheritance, without_inheritance)|
                 io.puts format(SHELL_CONDITIONAL_SET_COMMAND, name,
-                    with_inheritance.join(File::PATH_SEPARATOR),
-                    without_inheritance.join(File::PATH_SEPARATOR))
+                               with_inheritance.join(File::PATH_SEPARATOR),
+                               without_inheritance.join(File::PATH_SEPARATOR))
                 io.puts format(SHELL_EXPORT_COMMAND, name)
             end
             source_after(shell: shell).each do |path|
@@ -541,7 +554,8 @@ module Autobuild
                     with_inheritance = with_inheritance.map do |value|
                         if value == variable_expansion
                             base_env[name]
-                        else value
+                        else
+                            value
                         end
                     end
                     result[name] = with_inheritance.join(File::PATH_SEPARATOR)
@@ -555,9 +569,8 @@ module Autobuild
         # DEPRECATED: use add_path instead
         def self.pathvar(path, varname)
             if File.directory?(path)
-                if block_given?
-                    return unless yield(path)
-                end
+                return if block_given? && !yield(path)
+
                 add_path(varname, path)
             end
         end
@@ -604,7 +617,8 @@ module Autobuild
             @arch_size ||=
                 if RbConfig::CONFIG['host_cpu'] =~ /64/
                     64
-                else 32
+                else
+                    32
                 end
             @arch_size
         end
@@ -614,8 +628,6 @@ module Autobuild
             @arch_size, @arch_names = nil
         end
 
-        attr_reader :target_arch
-
         def arch_names
             return @arch_names if @arch_names
 
@@ -635,38 +647,85 @@ module Autobuild
             add_prefix(newprefix, includes)
         end
 
-        # rubocop:disable Metrics/LineLength
-        PKGCONFIG_INFO = [
-            %r{Scanning directory (?:#\d+ )?'(.*/)((?:lib|lib64|share)/.*)'$},
-            %r{Cannot open directory (?:#\d+ )?'.*/((?:lib|lib64|share)/.*)' in package search path:.*}
-        ].freeze
-        # rubocop:enable Metrics/LineLength
+        def default_cmake_search_globs(prefix, *file_globs)
+            lib_globs = %w[lib]
+
+            case arch_size
+            when 32
+                lib_globs << "lib32"
+                lib_globs << "libx32"
+            when 64
+                lib_globs << "lib64"
+            end
+
+            unless arch_names.empty?
+                arch_names.each do |arch_name|
+                    lib_globs << File.join("lib", arch_name)
+                end
+            end
+
+            lib_share_glob = "{#{lib_globs.join(',')},share}"
+            file_glob = "{#{file_globs.join(',')}}"
+
+            # Reference: https://cmake.org/cmake/help/latest/command/find_package.html
+            #
+            # <prefix>/                                                       (W)
+            # <prefix>/(cmake|CMake)/                                         (W)
+            # <prefix>/<name>*/                                               (W)
+            # <prefix>/<name>*/(cmake|CMake)/                                 (W)
+            # <prefix>/(lib/<arch>|lib*|share)/cmake/<name>*/                 (U)
+            # <prefix>/(lib/<arch>|lib*|share)/<name>*/                       (U)
+            # <prefix>/(lib/<arch>|lib*|share)/<name>*/(cmake|CMake)/         (U)
+            # <prefix>/<name>*/(lib/<arch>|lib*|share)/cmake/<name>*/         (W/U)
+            # <prefix>/<name>*/(lib/<arch>|lib*|share)/<name>*/               (W/U)
+            # <prefix>/<name>*/(lib/<arch>|lib*|share)/<name>*/(cmake|CMake)/ (W/U)
+            [
+                File.join(prefix, file_glob),
+                File.join(prefix, "{cmake,CMake}", file_glob),
+                File.join(prefix, "*", file_glob),
+                File.join(prefix, "*", "{cmake/CMake}", file_glob),
+                File.join(prefix, lib_share_glob, "cmake", "*", file_glob),
+                File.join(prefix, lib_share_glob, "*", file_glob),
+                File.join(prefix, lib_share_glob, "*", "{cmake,CMake}", file_glob),
+                File.join(prefix, "*", lib_share_glob, "cmake", "*", file_glob),
+                File.join(prefix, "*", lib_share_glob, "*", file_glob),
+                File.join(prefix, "*", lib_share_glob, "*", "{cmake,CMake}", file_glob)
+            ]
+        end
+
+        def has_cmake_files?(prefix, *file_globs)
+            default_cmake_search_globs(prefix, *file_globs).each do |glob_path|
+                return true unless Dir[glob_path].empty?
+            end
+            false
+        end
+
+        PKGCONFIG_PATH_RX = %r{.*/((?:lib|lib64|share)/.*)}.freeze
+
+        def pkgconfig_tool_path
+            @pkgconfig_tool_path ||= Autobuild.tool_in_path("pkg-config", env: self)
+        rescue ArgumentError
+            nil
+        end
 
         # Returns the system-wide search path that is embedded in pkg-config
         def default_pkgconfig_search_suffixes
-            found_path_rx = PKGCONFIG_INFO[0]
-            nonexistent_path_rx = PKGCONFIG_INFO[1]
+            return [] unless pkgconfig_tool_path
 
-            unless @default_pkgconfig_search_suffixes
-                pkg_config = Autobuild.tool("pkg-config")
-                output = `LANG=C PKG_CONFIG_PATH= #{pkg_config} --debug 2>&1`.split("\n")
-                found_paths = output.grep(found_path_rx).
-                    map { |l| l.gsub(found_path_rx, '\2') }.
-                    to_set
-                not_found = output.grep(nonexistent_path_rx).
-                    map { |l| l.gsub(nonexistent_path_rx, '\1') }.
-                    to_set
-                @default_pkgconfig_search_suffixes = found_paths | not_found
-            end
-            @default_pkgconfig_search_suffixes
+            @default_pkgconfig_search_suffixes ||=
+                `LANG=C #{pkgconfig_tool_path} --variable pc_path pkg-config`
+                    .strip
+                    .split(":")
+                    .grep(PKGCONFIG_PATH_RX)
+                    .map { |l| l.gsub(PKGCONFIG_PATH_RX, '\1') }
+                    .to_set
         end
 
         # Updates the environment when a new prefix has been added
         def add_prefix(newprefix, includes = nil)
-            if !includes || includes.include?('PATH')
-                if File.directory?("#{newprefix}/bin")
-                    add_path('PATH', "#{newprefix}/bin")
-                end
+            if (!includes || includes.include?('PATH')) &&
+               File.directory?("#{newprefix}/bin")
+                add_path('PATH', "#{newprefix}/bin")
             end
 
             if !includes || includes.include?('PKG_CONFIG_PATH')
@@ -686,6 +745,16 @@ module Autobuild
                 end
             end
 
+            cmake_pairs = []
+            cmake_pairs << ["CMAKE_PREFIX_PATH", ["*-config.cmake", "*Config.cmake"]]
+            cmake_pairs << ["CMAKE_MODULE_PATH", ["Find*.cmake"]]
+            cmake_pairs.each do |cmake_var, cmake_file_globs|
+                if !includes || includes.include?(cmake_var)
+                    has_cmake = has_cmake_files?(newprefix, *cmake_file_globs)
+                    add_path(cmake_var, newprefix) if has_cmake
+                end
+            end
+
             # Validate the new rubylib path
             if !includes || includes.include?('RUBYLIB')
                 new_rubylib = "#{newprefix}/lib"
@@ -717,7 +786,7 @@ module Autobuild
                 begin
                     stat = File.stat(full)
                     return full if stat.file? && stat.executable?
-                rescue ::Exception # rubocop:disable Lint/HandleExceptions
+                rescue ::Exception # rubocop:disable Lint/SuppressedException
                 end
             end
             nil
@@ -743,7 +812,7 @@ module Autobuild
         def prepare
             # Set up some important autobuild parameters
             inherit 'PATH', 'PKG_CONFIG_PATH', 'RUBYLIB', \
-                LIBRARY_PATH, 'CMAKE_PREFIX_PATH', 'PYTHONPATH'
+                    LIBRARY_PATH, 'CMAKE_PREFIX_PATH', 'PYTHONPATH'
         end
 
         # Method called to filter the environment variables before they are set,
@@ -868,9 +937,8 @@ module Autobuild
     # @deprecated use {Env#add_path} on {.env} instead
     def self.pathvar(path, varname)
         if File.directory?(path)
-            if block_given?
-                return unless yield(path)
-            end
+            return if block_given? && !yield(path)
+
             env.add_path(varname, path)
         end
     end

data/lib/autobuild/exceptions.rb

@@ -24,6 +24,7 @@ module Autobuild
             @target = target
             @phase  = phase
             @retry  = options[:retry]
+            super()
         end
 
         alias exception_message to_s
@@ -32,7 +33,8 @@ module Autobuild
             target_name =
                 if target.respond_to?(:name)
                     target.name
-                else target.to_str
+                else
+                    target.to_str
                 end
 
             if target && phase
@@ -52,10 +54,11 @@ module Autobuild
     class ConfigException < PhaseException
         def initialize(target = nil, phase = nil, options = Hash.new)
             options, other_options = Kernel.filter_options options,
-                retry: false
+                                                           retry: false
             super(target, phase, options.merge(other_options))
         end
     end
+
     ## An error occured in a package
     class PackageException < PhaseException
         def mail?
@@ -64,7 +67,7 @@ module Autobuild
 
         def initialize(target = nil, phase = nil, options = Hash.new)
             options, other_options = Kernel.filter_options options,
-                retry: false
+                                                           retry: false
             super(target, phase, options.merge(other_options))
         end
     end
@@ -76,6 +79,7 @@ module Autobuild
 
     # The subcommand is not found
     class CommandNotFound  < PhaseException; end
+
     # An error occured while running a subcommand
     class SubcommandFailed < PhaseException
         def mail?
@@ -84,8 +88,10 @@ module Autobuild
 
         attr_writer :retry
         attr_reader :command, :logfile, :status, :output
+
         def initialize(*args)
-            if args.size == 1
+            case args.size
+            when 1
                 sc = args[0]
                 target = sc.target
                 command = sc.command
@@ -93,7 +99,7 @@ module Autobuild
                 status = sc.status
                 output = sc.output
                 @orig_message = sc.exception_message
-            elsif args.size == 4 || args.size == 5
+            when 4, 5
                 target, command, logfile, status, output = *args
             else
                 raise ArgumentError, "wrong number of arguments, should be 1 or 4..5"
@@ -140,13 +146,14 @@ module Autobuild
 
     # The exception type that is used to report multiple errors that occured
     # when ignore_errors is set
-    class CompositeException < Autobuild::Exception
+    class CompositeException < PhaseException
         # The array of exception objects representing all the errors that
         # occured during the build
         attr_reader :original_errors
 
         def initialize(original_errors)
             @original_errors = original_errors
+            super()
         end
 
         def mail?