GHC 2017-04-13

24 comments.

, https://git.io/vS9rY in Homebrew/homebrew-core
youtube-dl 2017.04.14
=====================

Created with `brew bump-formula-pr`.

, https://git.io/vS9f5 in jarun/nnn
I'm not prying ;)

, https://git.io/vS9fd in jarun/nnn
Nit: this command and the `make; sudo make install` below are not indented enough, and hence not rendered as pre. Need to further the indents by two.

, https://git.io/vS9fF in jarun/nnn
LGTM.

, https://git.io/vS9fb in jarun/nnn
Oops, I meant `brew install jarun/nnn/nnn`... Sorry!

, https://git.io/vS9fN in jarun/homebrew-nnn
This comment should be removed too.

, https://git.io/vSS51 in jarun/nnn
Cool. Just remember to update the checksum and remove the `inreplace` if you ever decide to retag.

, https://git.io/vSS5M in jarun/nnn
Sure.

, https://git.io/vSS5D in Homebrew/homebrew-core
#12390.

, https://git.io/vSS5y in Homebrew/homebrew-core
mps-youtube: update resource youtube_dl
=======================================

In response to a test failure
https://bot.brew.sh/job/Homebrew/job/homebrew-core/job/PR-12389/4/testReport/junit/brew-test-bot/sierra/test_mps_youtube/.

, https://git.io/vSS5S in Homebrew/homebrew-core
Probably. It's not related to FFmpeg 3.3 at all.

, https://git.io/vSS59 in Homebrew/homebrew-core
Looks like the vendored youtube-dl can't parse the YouTube js payload. Guess it's time to update the resource.

, https://git.io/vSS5H in jarun/nnn
By the way, currently the man page is installed into `$(PREFIX)/man`, when `$(PREFIX)/share/man` is more appropriate.

, https://git.io/vSS5Q in jarun/nnn
Set up Homebrew tap
===================

To set up a [Homebrew tap](https://github.com/Homebrew/brew/blob/master/docs/How-to-Create-and-Maintain-a-Tap.md), create a repository at jarun/homebrew-nnn, place the single file `nnn.rb` in it:

```rb
class Nnn < Formula
  desc "Free, fast, friendly file browser"
  homepage "https://github.com/jarun/nnn"
  url "https://github.com/jarun/nnn/archive/v1.0.tar.gz"
  sha256 "dd33f1001a87a427f10b6468db7dcba0bdc19643f121dbf6bb8655d5f2d3bac5"

  def install
    # Remove when Makefile is fixed
    inreplace "Makefile", "$(PREFIX)/man", "$(PREFIX)/share/man"
    system "make", "install", "PREFIX=#{prefix}"
  end

  test do
    touch "foobar"
    # Testing this curses app requires a pty
    require "pty"
    PTY.spawn(bin/"nnn") do |r, w, pid|
      w.write "q"
      assert_match "foobar", r.read
    end
  end
end

```

add an optional README if you want to, but Homebrew/Linuxbrew users who just do

```
brew install jarun/nnn
```

won't see the README.

, https://git.io/vSS57 in Homebrew/homebrew-core
The mps_youtube failure doesn't look remotely related, and it wasn't present in the first build, where only mpv failed since I forgot to bump it. Wtf.

, https://git.io/vSS55 in Homebrew/homebrew-core
The Pro model isn't coming this year.

, https://git.io/vSSXI in Homebrew/homebrew-core
You need to install a cpp to your mind. Then you won't see preprocessor macros any longer.

, https://git.io/vSSXL in Homebrew/homebrew-core
The more `ifdef __APPLE__` the merrier. /s

, https://git.io/vSSXt in Homebrew/homebrew-core
I think that's the case.

, https://git.io/vSSXq in Homebrew/homebrew-core
No, fixed in https://github.com/FFmpeg/FFmpeg/commit/0e8b68a2c4031e25082603ad88711be12210d41f.

, https://git.io/vSSXm in Homebrew/homebrew-core
ffmpeg 3.3
==========

Created with `brew bump-formula-pr`.

, https://git.io/vSy9L in jarun/nnn
I would have to do the same anyway to submit it to homebrew/core.

, https://git.io/vSy9t in jarun/nnn
As I said it's trivial :) It will be one short Ruby file similar to https://github.com/Homebrew/homebrew-core/blob/master/Formula/googler.rb, and brief instructions in this repo's README (could be as brief as `brew install jarun/nnn` I wrote above).

, https://git.io/vSy9q in jarun/nnn
You can set up a "tap" as I said in the meantime to make it easier for Mac users to install (it's a difference of `brew install jarun/nnn` vs `brew install nnn` when it's accepted into core). That might or might not help you get to 50 stars sooner.