Class: RuboCop::Cop::FormulaAudit::Text Private

Inherits:
RuboCop::Cop::FormulaCop show all
Extended by:
AutoCorrector
Defined in:
rubocops/text.rb

Overview

This class is part of a private API. This class may only be used in the Homebrew/brew repository. Third parties should avoid using this class if possible, as it may be removed or changed without warning.

This cop checks for various problems in a formula’s source code.

Instance Attribute Summary

Attributes inherited from RuboCop::Cop::FormulaCop

#file_path

Instance Method Summary collapse

Methods inherited from RuboCop::Cop::FormulaCop

#audit_comments, #audit_urls, #caveats_strings, #depends_on?, #depends_on_name_type?, #formula_tap, #get_checksum_node, #on_class, #tap_style_exception?, #versioned_formula?

Methods included from HelperFunctions

#block_size, #check_precedence, #class_name, #component_precedes?, #end_column, #expression_negated?, #find_all_blocks, #find_block, #find_blocks, #find_const, #find_every_func_call_by_name, #find_every_method_call_by_name, #find_instance_call, #find_instance_method_call, #find_method_calls_by_name, #find_method_def, #find_method_with_args, #find_node_method_by_name, #find_strings, #format_component, #line_number, #line_start_column, #method_called?, #method_called_ever?, #method_called_in_block?, #method_name, #node_equals?, #offending_node, #parameters, #parameters_passed?, #problem, #regex_match_group, #size, #source_buffer, #start_column, #string_content

Instance Method Details

#audit_formula(node, _class_node, _parent_class_node, body_node) ⇒ Object

This method is part of a private API. This method may only be used in the Homebrew/brew repository. Third parties should avoid using this method if possible, as it may be removed or changed without warning.



15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
# File 'rubocops/text.rb', line 15

def audit_formula(node, _class_node, _parent_class_node, body_node)
  full_source_content = source_buffer(node).source

  if (match = full_source_content.match(/^require ['"]formula['"]$/))
    range = source_range(source_buffer(node), match.pre_match.count("\n") + 1, 0, match[0].length)
    add_offense(range, message: "`#{match}` is now unnecessary") do |corrector|
      corrector.remove(range_with_surrounding_space(range: range))
    end
  end

  if !find_node_method_by_name(body_node, :plist_options) &&
     find_method_def(body_node, :plist)
    problem "Please set plist_options when using a formula-defined plist."
  end

  if (depends_on?("openssl") || depends_on?("openssl@1.1")) && depends_on?("libressl")
    problem "Formulae should not depend on both OpenSSL and LibreSSL (even optionally)."
  end

  if formula_tap == "homebrew-core" && (depends_on?("veclibfort") || depends_on?("lapack"))
    problem "Formulae in homebrew/core should use OpenBLAS as the default serial linear algebra library."
  end

  if method_called_ever?(body_node, :virtualenv_create) ||
     method_called_ever?(body_node, :virtualenv_install_with_resources)
    find_method_with_args(body_node, :resource, "setuptools") do
      problem "Formulae using virtualenvs do not need a `setuptools` resource."
    end
  end

  unless method_called_ever?(body_node, :go_resource)
    # processed_source.ast is passed instead of body_node because `require` would be outside body_node
    find_method_with_args(processed_source.ast, :require, "language/go") do
      problem "require \"language/go\" is unnecessary unless using `go_resource`s"
    end
  end

  find_instance_method_call(body_node, "Formula", :factory) do
    problem "\"Formula.factory(name)\" is deprecated in favor of \"Formula[name]\""
  end

  find_method_with_args(body_node, :system, "xcodebuild") do
    problem %q(use "xcodebuild *args" instead of "system 'xcodebuild', *args")
  end

  find_method_with_args(body_node, :system, "go", "get") do
    problem "Do not use `go get`. Please ask upstream to implement Go vendoring"
  end

  find_method_with_args(body_node, :system, "dep", "ensure") do |d|
    next if parameters_passed?(d, /vendor-only/)
    next if @formula_name == "goose" # needed in 2.3.0

    problem "use \"dep\", \"ensure\", \"-vendor-only\""
  end

  find_method_with_args(body_node, :system, "cargo", "build") do
    problem "use \"cargo\", \"install\", *std_cargo_args"
  end

  find_every_method_call_by_name(body_node, :system).each do |m|
    next unless parameters_passed?(m, /make && make/)

    offending_node(m)
    problem "Use separate `make` calls"
  end

  body_node.each_descendant(:dstr) do |dstr_node|
    dstr_node.each_descendant(:begin) do |interpolation_node|
      next unless interpolation_node.source.match?(/#\{\w+\s*\+\s*['"][^}]+\}/)

      offending_node(interpolation_node)
      problem "Do not concatenate paths in string interpolation"
    end
  end

  prefix_path(body_node) do |prefix_node, path|
    next unless (match = path.match(%r{^(bin|include|libexec|lib|sbin|share|Frameworks)(?:/| |$)}))

    offending_node(prefix_node)
    problem "Use `#{match[1].downcase}` instead of `prefix + \"#{match[1]}\"`"
  end
end