Hi guys, Could I poke you again to take a look at these patches for me? :)
Thanks! Faraaz On 2/19/16, 5:10 PM, "Faraaz Sareshwala" <[email protected]> wrote: >Thanks for taking a look Roman! > >All commits are published together on my bigtop fork’s master branch >(https://github.com/fsareshwala/bigtop). There are individual branches for >each patch as well so that I could make the pull request for each clean. Let >me know if you need anything else :). > >Faraaz > > > >On 2/19/16, 4:11 PM, "[email protected] on behalf of Roman Shaposhnik" ><[email protected] on behalf of [email protected]> wrote: > >>Hi Faraaz! >> >>On Fri, Feb 19, 2016 at 1:51 PM, Faraaz Sareshwala >><[email protected]> wrote: >>> I’ve made all the requested changes for the submitted patches. I think the >>> puppet recipes >>> and smoke tests are pretty much ready to go. Let me know if you guys have >>> any comments >>> on that. Can someone take a look at the packaging so we can get that >>> reviewed and merged? >> >>Thanks for the contributions! Just to make it easier on reviewers >>could you please publish all the required patches in a GH repo? >>So I can just pull from there, build and play with your code all at once. >> >>Thanks, >>Roman.
