{"action":"created","issue":{"url":"https://api.github.com/repos/Homebrew/homebrew/issues/41239","labels_url":"https://api.github.com/repos/Homebrew/homebrew/issues/41239/labels{/name}","comments_url":"https://api.github.com/repos/Homebrew/homebrew/issues/41239/comments","events_url":"https://api.github.com/repos/Homebrew/homebrew/issues/41239/events","html_url":"https://github.com/Homebrew/homebrew/issues/41239","id":92366458,"number":41239,"title":"[PATCH] audit should skip head-only error when in */head-only directory and same for revel-only","user":{"login":"rkrug","id":487172,"avatar_url":"https://avatars.githubusercontent.com/u/487172?v=3","gravatar_id":"","url":"https://api.github.com/users/rkrug","html_url":"https://github.com/rkrug","followers_url":"https://api.github.com/users/rkrug/followers","following_url":"https://api.github.com/users/rkrug/following{/other_user}","gists_url":"https://api.github.com/users/rkrug/gists{/gist_id}","starred_url":"https://api.github.com/users/rkrug/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rkrug/subscriptions","organizations_url":"https://api.github.com/users/rkrug/orgs","repos_url":"https://api.github.com/users/rkrug/repos","events_url":"https://api.github.com/users/rkrug/events{/privacy}","received_events_url":"https://api.github.com/users/rkrug/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-07-01T12:11:53Z","updated_at":"2015-07-02T15:23:31Z","closed_at":null,"body":"Hi\r\n\r\nI have a head-only recipe which obviously throws an error when running ```brew audit ...```. In the code from ```audit.rb```, recipes in the taps ```home-brew/homebrew-head-only``` and ```home-brew/homebrew-devel-only``` are excluded from throwing errors when they are head-only or devel-only. \r\nI would suggest to add the same behaviour for private (non-homebrew) taps so that audit can be used reliably without throwing expected errors.\r\n\r\nThis is especially important for running automated tests with travis-ci as I have to exclude these  recipes from audit as otherwise the test will be classified as failed. \r\n\r\nTherefore I suggest the following patch:\r\n\r\n```\r\nFrom 6ffd15a128630f463d8a1ee835e7569dbe79609f Mon Sep 17 00:00:00 2001\r\nFrom: \"Rainer M. Krug\" <R.M.Krug@gmail.com>\r\nDate: Tue, 30 Jun 2015 09:38:30 +0200\r\nSubject: [PATCH] Make audit recognise hed-only and devel-only in all taps\r\n\r\n- add wildcard into expressions to identify head-only and dev-only taps\r\n---\r\n Library/Homebrew/cmd/audit.rb | 4 ++--\r\n 1 file changed, 2 insertions(+), 2 deletions(-)\r\n\r\ndiff --git a/Library/Homebrew/cmd/audit.rb b/Library/Homebrew/cmd/audit.rb\r\nindex 3a41dbd..4a6a305 100644\r\n--- a/Library/Homebrew/cmd/audit.rb\r\n+++ b/Library/Homebrew/cmd/audit.rb\r\n@@ -421,11 +421,11 @@ class FormulaAuditor\r\n   end\r\n\r\n   def audit_specs\r\n-    if head_only?(formula) && formula.tap.to_s.downcase != \"homebrew/homebrew-head-only\"\r\n+    if head_only?(formula) && formula.tap.to_s.downcase != \"*/homebrew-head-only\"\r\n       problem \"Head-only (no stable download)\"\r\n     end\r\n\r\n-    if devel_only?(formula) && formula.tap.to_s.downcase != \"homebrew/homebrew-devel-only\"\r\n+    if devel_only?(formula) && formula.tap.to_s.downcase != \"*/homebrew-devel-only\"\r\n       problem \"Devel-only (no stable download)\"\r\n     end\r\n\r\n--\r\n2.4.5\r\n```\r\n\r\nThanks,\r\n\r\nRainer"},"comment":{"url":"https://api.github.com/repos/Homebrew/homebrew/issues/comments/118066892","html_url":"https://github.com/Homebrew/homebrew/issues/41239#issuecomment-118066892","issue_url":"https://api.github.com/repos/Homebrew/homebrew/issues/41239","id":118066892,"user":{"login":"rkrug","id":487172,"avatar_url":"https://avatars.githubusercontent.com/u/487172?v=3","gravatar_id":"","url":"https://api.github.com/users/rkrug","html_url":"https://github.com/rkrug","followers_url":"https://api.github.com/users/rkrug/followers","following_url":"https://api.github.com/users/rkrug/following{/other_user}","gists_url":"https://api.github.com/users/rkrug/gists{/gist_id}","starred_url":"https://api.github.com/users/rkrug/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rkrug/subscriptions","organizations_url":"https://api.github.com/users/rkrug/orgs","repos_url":"https://api.github.com/users/rkrug/repos","events_url":"https://api.github.com/users/rkrug/events{/privacy}","received_events_url":"https://api.github.com/users/rkrug/received_events","type":"User","site_admin":false},"created_at":"2015-07-02T15:23:31Z","updated_at":"2015-07-02T15:23:31Z","body":"\n\nEnvoyé de mon iPhone\n\n> Le 2 juil. 2015 à 16:55, Xu Cheng <notifications@github.com> a écrit :\n> \n> Personally, I find it less meaningful to ignore such error based on tap name. On the one hand, in most common situations, 3rd-party taps would be named after the software's name rather than head-only. And they are likely to use the same tap name regardless the development state(beta or stable). On the other hand, such behavior is less intuitive and can cause confusing especially without document.\n> \n\nI consider it as more consistent if private repos are dealt with in the same way as the official ones in regards to audit. At the moment, this error is excluded for the homebrew-head-only tap. I think that the consistency would require the same exclusion rules for all taps. \n> IMHO, I would suggest @rkrug to use a custom command. It doesn't complex anything. neovim/homebrew-neovim has been doing this for a long time. If we really want to support ignoring officially, I think it's best to accomplish this through an option --ignore-headonly, like what I proposed in #34121.\n> \n\nThis would work in my case, but it would not be consistent. Especially S head-only formulas can be really useful in development. \n> —\n> Reply to this email directly or view it on GitHub.\n> \n"}}