Re: Review Request 39991: fetcher should extract from .tar archive

2015-11-16 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39991/#review106629 --- Ship it! I believe the copy-n-paste should be fine here until we

Re: Review Request 39991: fetcher should extract from .tar archive

2015-11-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39991/#review106195 --- Patch looks great! Reviews applied: [39991] All tests passed. -

Re: Review Request 39991: fetcher should extract from .tar archive

2015-11-11 Thread Bhuvan Arumugam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39991/ --- (Updated Nov. 12, 2015, 1:47 a.m.) Review request for mesos and Bernd

Re: Review Request 39991: fetcher should extract from .tar archive

2015-11-06 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39991/#review105404 --- src/tests/fetcher_tests.cpp (line 542)

Re: Review Request 39991: fetcher should extract from .tar archive

2015-11-06 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39991/#review105455 --- src/tests/fetcher_tests.cpp (line 548)

Re: Review Request 39991: fetcher should extract from .tar archive

2015-11-06 Thread Bernd Mathiske
> On Nov. 6, 2015, 2:48 a.m., Adam B wrote: > > src/tests/fetcher_tests.cpp, lines 553-554 > > > > > > Where did you pull this TODO from? Should it remain a benh TODO or do > > you want to own it? This is from way

Re: Review Request 39991: fetcher should extract from .tar archive

2015-11-05 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39991/#review105311 --- Patch looks great! Reviews applied: [39991] All tests passed. -