There's no more pull requests left to pull on this repository.

This job takes almost a minute and it's not useful for all core taps
to run this.

We're not really getting any more PRs here and this code makes this file
harder to follow and refactor.

This means that we do not need to read formulae or evaluate Ruby at
upload time.

Both the `CctoolsMachO` and `RubyMachO` module implement a common set of
methods that simplify querying `mach_data`. Move these into a shared
module, that gets included after either of these implementations is
loaded and included in `Pathname`.

The check that `t4` is not an external command would always succeed, but
not because the file wasn't executable, but because it wasn't even found
due to the missing `brew-` prefix.

Also change the valid but atypical file mode from 0744 to 0755 and apply
minor code style fixes.

Affects only legacy systems that don't have a `/usr/bin/cc` binary.

Amends 132ada2b until we properly figure
out a way of documenting built-in aliases (or not doing that) across all
commands that currently have aliases. See #270 for full discussion.

* Include --desc in brew search help text.
* Update brew search --desc text in README.
* Remove mention of alias in brew search help text to avoid any
  confusion.

Avoid breaking code highlighting of some editors (e.g. Sublime Text)
when `gem` is used as an identifier. This messes up the highlighting
below the first use and makes working with the file rather annoying.

This mainly fixes the problems for subclasses of CurlDownloadStrategy.
More specifically it fixes two things:
 * It allows the no insecure redirect check to be applied to
   CurlApacheMirrorDownloadStrategy.
 * It fixes previous broken CurlPostDownloadStrategy.

Closes #280.

Signed-off-by: Xu Cheng <xucheng@me.com>

Reference from `man curl`

>  -R, --remote-time
>    When  used,  this will make curl attempt to figure out the time-
>    stamp of the remote file, and if  that  is  available  make  the
>    local file get that same timestamp.

Closes #248.

Signed-off-by: Dominyk Tiller <dominyktiller@gmail.com>

For some reason the existing check seems to have started failing between
March and today. I haven't managed to narrow down why yet but the biggest
change between then and now was the core separation so perhaps related to that.

Perhaps at some point we started considering purely short tap names,
i.e. homebrew/devel-only rather than full tap names, i.e. homebrew/homebrew-devel-only,
in the audit mechanism.

This fixes the current issue whilst retaining the spirit of the original commit:
https://github.com/Homebrew/brew/commit/86d04e94e9caacf4aba766dd31c1707749fb5f2b

Originally introduced by me in 77975168.

This avoids weird issues with custom Rubies.

Closes #253.

At this point it's never a good compiler to use so let's just remove it.

Since `system` escapes its own spaces, cabal was interpreting
`--flags='webapp s3'` as `+'webapp +s3'` rather than `+webapp +s3`.

Closes #267.

Signed-off-by: ilovezfs <ilovezfs@icloud.com>

Used for uploading imagemagick 6.9.4-3.

There are unnecessarily verbose, have been documented to be avoided and
it’s good to nudge people towards the other style.

Indicate that feature requests should go there rather than here.

GitHub’s code load and patch-diff URLs are the result of redirects and
make it harder to modify the URL to reach the original repository.

Obvious copy/paste failure in acc9a7ca.

No need to source `cmd/update.sh` if `homebrew-update` is not going to
be called directly. Amends 750bb24a.

Closes #239.

Closes #228.

git is also used in analytics.sh

Preinstall update should be performed in a separate process,
so it can maintain and close its own update lock.