GitHub PullRequestReviewCommentEvent at 2014-03-14 13:09:42
Repository: homebrew
Actor: jstepien
Payload: {"comment":{"url":"https://api.github.com/repos/Homebrew/homebrew/pulls/comments/10606222","id":10606222,"diff_hunk":"@@ -0,0 +1,25 @@\n+require \"formula\"\n+\n+class Lz4 < Formula\n+  homepage \"http://code.google.com/p/lz4/\"\n+  url \"https://dl.dropboxusercontent.com/u/59565338/LZ4/lz4-r114.tar.gz\"\n+  sha1 \"7b6c4c3b01edbb60e4c07657c3c41e8b5e95770e\"\n+  version \"r114\"\n+\n+  def install\n+    inreplace \"Makefile\", /-Wl,-soname=[^ ]+/, \"\"\n+    inreplace \"Makefile\", /\\.so/, \".dylib\"\n+    system \"make\", \"install\", \"PREFIX=#{prefix}\"\n+    rm prefix/\"lib/liblz4.dylib.1\"\n+    rm prefix/\"lib/liblz4.dylib.1.0.0\"","path":"Library/Formula/lz4.rb","position":14,"original_position":14,"commit_id":"97e0602c7b266b0f25e6220a520455ecf585d213","original_commit_id":"97e0602c7b266b0f25e6220a520455ecf585d213","user":{"login":"jstepien","id":39084,"avatar_url":"https://gravatar.com/avatar/8666f7a05688031004c8c0bc7e0dcef7?d=https%3A%2F%2Fidenticons.github.com%2F92882bb23ae78cc5bde62988da054d55.png&r=x","gravatar_id":"8666f7a05688031004c8c0bc7e0dcef7","url":"https://api.github.com/users/jstepien","html_url":"https://github.com/jstepien","followers_url":"https://api.github.com/users/jstepien/followers","following_url":"https://api.github.com/users/jstepien/following{/other_user}","gists_url":"https://api.github.com/users/jstepien/gists{/gist_id}","starred_url":"https://api.github.com/users/jstepien/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jstepien/subscriptions","organizations_url":"https://api.github.com/users/jstepien/orgs","repos_url":"https://api.github.com/users/jstepien/repos","events_url":"https://api.github.com/users/jstepien/events{/privacy}","received_events_url":"https://api.github.com/users/jstepien/received_events","type":"User","site_admin":false},"body":"`brew audit` reported that the lib directory contains entries which aren't libraries. That's why I remove them here. Should I not?","created_at":"2014-03-14T13:09:42Z","updated_at":"2014-03-14T13:09:42Z","html_url":"https://github.com/Homebrew/homebrew/pull/27540#discussion_r10606222","pull_request_url":"https://api.github.com/repos/Homebrew/homebrew/pulls/27540","_links":{"self":{"href":"https://api.github.com/repos/Homebrew/homebrew/pulls/comments/10606222"},"html":{"href":"https://github.com/Homebrew/homebrew/pull/27540#discussion_r10606222"},"pull_request":{"href":"https://api.github.com/repos/Homebrew/homebrew/pulls/27540"}}}}