This is causing submodules to be checked out into a path with a literal
"$path" component:

$ find . | grep "\$path$"
./$path
./Carthage/Checkouts/Commandant/$path
./Carthage/Checkouts/Commandant/Carthage/Checkouts/Quick/$path
./Carthage/Checkouts/PrettyColors/$path
./Carthage/Checkouts/Quick/$path
./Carthage/Checkouts/ReactiveCocoa/$path
./Carthage/Checkouts/ReactiveCocoa/Carthage/Checkouts/Quick/$path
./Carthage/Checkouts/ReactiveTask/$path
./Carthage/Checkouts/ReactiveTask/Carthage/Checkouts/Quick/$path
./Carthage/Checkouts/ReactiveTask/Carthage/Checkouts/ReactiveCocoa/$path
./Carthage/Checkouts/ReactiveTask/Carthage/Checkouts/ReactiveCocoa/Carthage.checkout/Quick/$path

This reverts commit 3a5f099297e9ab3b799af5f7a65edd8d64bc7bae.

Before:

    $ brew info llvm
    ==> Dependencies
    Build: xz , xz , xz , xz , xz , xz 

    $ brew info --json=v1 llvm
    ... "dependencies":["xz","xz","xz","xz","xz","xz"], ...

After

    $ brew info llvm
    ==> Dependencies
    Build: xz 



    $ brew info --json=v1 llvm
    ... "dependencies":["xz"], ...

Closes Homebrew/homebrew#36653.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

for issue Homebrew/homebrew#36646

Closes Homebrew/homebrew#36649.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

See
https://github.com/Homebrew/homebrew/pull/36629#discussion_r24293912


amongst other recent examples. The example formula isn’t as strongly
policed as the ` brew create ` default, this tries to do something
about that.

Closes Homebrew/homebrew#36639.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

Losslessly compress with jpegoptim and optipng.

Closes Homebrew/homebrew#36606.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

https://github.com/Homebrew/homebrew-brewdler/pull/43#discussion_r24248760



Closes Homebrew/homebrew#36611.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

Fixes Homebrew/homebrew#35382.

This probably doesn't matter in practice, but I don't want to take any
chances.

Fixes Homebrew/homebrew#36335.

It shadows system headers sometimes (probably just Server.app?) but we don't
really care.

Don't match `help` at the end of every string because `brew installhelp`
shouldn't really be running help.

Closes Homebrew/homebrew#36559.
Closes Homebrew/homebrew#36576.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

Update advice about easy-install.pth files.

Closes Homebrew/homebrew#36547.

e.g. devel-only or head-only formulae.

Since pyenv v20141211, it bundles built-in pip-rehash feature.

Closes Homebrew/homebrew#36524.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

Fixes Homebrew/homebrew#36484.

This commits adds support for updating a formula with changed git
submodules with GitDownloadStrategy when the cloned repository is still
in the cache. This is done by running git submodule sync --recursive
before updating the submodules, so that git can reload the submodule
config from the updated .gitmodules file.

Fixes Homebrew/homebrew#36395.
Closes Homebrew/homebrew#36466.

Signed-off-by: Jack Nagel <jacknagel@gmail.com>

We want the apr requirement to act as a build-time requirement in case
it is satisfied (that is, the CLT is installed), as the resulting
binaries will link to the system libapr which is always present.

When it is *not* satisfied by the CLT, and we need to install the
formula, we have to treat it as a runtime dependency since the resulting
binaries will link to it.

Fixes Homebrew/homebrew#36301.
Fixes Homebrew/homebrew#36438.
Closes Homebrew/homebrew#36443.

If apr is installed, we still want to grab it during dependency
resolution in case we need to pass options to the generated dependency.

In other words, it doesn't make sense for the satisfiable condition to
include the default formula, as the default formula is the fallback for
when the requirement is *not* satisfied.

Requirements with default formulae cannot fail the build.

This matches the recommendation in the message.

Closes Homebrew/homebrew#36436.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

Fix for linking into the system OpenSSL, strict audit fixes, and the
plist.

Closes Homebrew/homebrew#36386.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

The project was deprecated in favor of percona-toolkit:
http://blog.codenode.com/2015/01/hack-mysql-has-been-retired.html

Closes Homebrew/homebrew#36342.

Closes Homebrew/homebrew#36009.

Sitecopy currently compiles against the insecure system OpenSSL, and
won’t compile against our OpenSSL because it wants SSLv2 at runtime
(Ouch) and it won’t compile against GnuTLS because GnuTLS have
considerably changed the structure of their build since the last update
of sitecopy (2008).

Closes Homebrew/homebrew#36389.

Signed-off-by: Jack Nagel <jacknagel@gmail.com>

Closes Homebrew/homebrew#36368.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

Closes Homebrew/homebrew#35626.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

Closes Homebrew/homebrew#36308.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

See discussion in https://github.com/Homebrew/homebrew-devel-only/pull/8



In essence, the test bot currently does this:

```
==> brew install --verbose --build-bottle j2objc			 FAILED
Error: j2objc is a devel-only formula
Install with `brew install --devel j2objc`
```

I’m proposing that we pass that arg to the test-bot automatically to
allow the bot to pass head-only and devel-only formulae without that
failure message. I’ve also trimmed the arguments further down to
prevent it duplicating the —devel install in formulae which define no
stable block.

Closes Homebrew/homebrew#36030.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

Closes Homebrew/homebrew#36179.
Closes Homebrew/homebrew#36184.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>