{"action":"created","issue":{"url":"https://api.github.com/repos/bundler/bundler/issues/3633","labels_url":"https://api.github.com/repos/bundler/bundler/issues/3633/labels{/name}","comments_url":"https://api.github.com/repos/bundler/bundler/issues/3633/comments","events_url":"https://api.github.com/repos/bundler/bundler/issues/3633/events","html_url":"https://github.com/bundler/bundler/pull/3633","id":74386414,"number":3633,"title":"fast clean manpage build","user":{"login":"pjump","id":3763707,"avatar_url":"https://avatars.githubusercontent.com/u/3763707?v=3","gravatar_id":"","url":"https://api.github.com/users/pjump","html_url":"https://github.com/pjump","followers_url":"https://api.github.com/users/pjump/followers","following_url":"https://api.github.com/users/pjump/following{/other_user}","gists_url":"https://api.github.com/users/pjump/gists{/gist_id}","starred_url":"https://api.github.com/users/pjump/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pjump/subscriptions","organizations_url":"https://api.github.com/users/pjump/orgs","repos_url":"https://api.github.com/users/pjump/repos","events_url":"https://api.github.com/users/pjump/events{/privacy}","received_events_url":"https://api.github.com/users/pjump/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-05-08T14:45:19Z","updated_at":"2015-05-10T12:48:40Z","closed_at":"2015-05-09T21:14:27Z","pull_request":{"url":"https://api.github.com/repos/bundler/bundler/pulls/3633","html_url":"https://github.com/bundler/bundler/pull/3633","diff_url":"https://github.com/bundler/bundler/pull/3633.diff","patch_url":"https://github.com/bundler/bundler/pull/3633.patch"},"body":"@indirect I don't care for it. As a matter of fact, I think it's an awesome idea that each `rake build`, regardless of whether it has actual work to do or not, should give people an obligatory 3 seconds to ponder about how awesome bundler is. :)\r\n"},"comment":{"url":"https://api.github.com/repos/bundler/bundler/issues/comments/100635267","html_url":"https://github.com/bundler/bundler/pull/3633#issuecomment-100635267","issue_url":"https://api.github.com/repos/bundler/bundler/issues/3633","id":100635267,"user":{"login":"pjump","id":3763707,"avatar_url":"https://avatars.githubusercontent.com/u/3763707?v=3","gravatar_id":"","url":"https://api.github.com/users/pjump","html_url":"https://github.com/pjump","followers_url":"https://api.github.com/users/pjump/followers","following_url":"https://api.github.com/users/pjump/following{/other_user}","gists_url":"https://api.github.com/users/pjump/gists{/gist_id}","starred_url":"https://api.github.com/users/pjump/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pjump/subscriptions","organizations_url":"https://api.github.com/users/pjump/orgs","repos_url":"https://api.github.com/users/pjump/repos","events_url":"https://api.github.com/users/pjump/events{/privacy}","received_events_url":"https://api.github.com/users/pjump/received_events","type":"User","site_admin":false},"created_at":"2015-05-10T12:48:40Z","updated_at":"2015-05-10T12:48:40Z","body":"You could've just hit merge and add the clobber task if that's what you wanted. Your solution didn't eliminate the `generated_files` block-local variable shared between inner blocks (which, I assume, is what you referred to by global state, which I think there's little point to eliminating in Rakefiles as those are inherently about global state (but it doesn't matter, because rakefiles aren't libraries to be reshared)), only replaced it with the `source` block-local variable that's shared among inner blocks (same thing, different name),  made it more \"indirect\", about twice as long, less efficient (the each + include? should be much less efficient that `Array#-`, but that shouldn't matter here with the fileset size as small as it is) and introduced two bugs (leftover *.txt files don't get cleaned now and the /gemfile\\.5(\\.txt)?/ is overly broad and should be /^gemfile\\.5(\\.txt)?$/  or better yet, if you do the mappings right, you shouldn't need to give this file special treatment). "}}