Re: Review Request 40214: Cleaned up protobuf generation rules in src/Makefile.am.

2015-11-12 Thread Vinod Kone 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40214/#review106271
---

Ship it!



src/Makefile.am (lines 163 - 169)


just curious why we had LOG_PROTOS and STATE_PROTOS pointing to the C files 
instead of LOG_PROTO and STATE_PROTO pointing to the proto files?


- Vinod Kone


On Nov. 12, 2015, 1:31 a.m., Jie Yu wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40214/
> ---
> 
> (Updated Nov. 12, 2015, 1:31 a.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Cleaned up protobuf generation rules in src/Makefile.am.
> 
> Probobuf generation rules in src/Makefile.am is a mess. As we introduce more 
> and more protobuf files in the code base, this will soon become 
> unmaintainable. This is the first phase of the clean up. The next step is to 
> consolidate all the protobuf generation rules (instead of copy and paste).
> 
> 
> Diffs
> -
> 
>   src/Makefile.am d40378f8dc85a3b98d826e6516e17c05cd7ce174 
> 
> Diff: https://reviews.apache.org/r/40214/diff/
> 
> 
> Testing
> ---
> 
> ./configure
> make check
> make install
> 
> 
> Thanks,
> 
> Jie Yu
> 
>

Re: Review Request 40214: Cleaned up protobuf generation rules in src/Makefile.am.

2015-11-12 Thread Jie Yu 


> On Nov. 12, 2015, 5:26 p.m., Vinod Kone wrote:
> > src/Makefile.am, lines 171-177
> > 
> >
> > just curious why we had LOG_PROTOS and STATE_PROTOS pointing to the C 
> > files instead of LOG_PROTO and STATE_PROTO pointing to the proto files?

liblog and libstate need to include those cc and h files. I'll do another 
cleanup shortly to consolidate these rules.


- Jie


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40214/#review106271
---


On Nov. 12, 2015, 1:31 a.m., Jie Yu wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40214/
> ---
> 
> (Updated Nov. 12, 2015, 1:31 a.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Cleaned up protobuf generation rules in src/Makefile.am.
> 
> Probobuf generation rules in src/Makefile.am is a mess. As we introduce more 
> and more protobuf files in the code base, this will soon become 
> unmaintainable. This is the first phase of the clean up. The next step is to 
> consolidate all the protobuf generation rules (instead of copy and paste).
> 
> 
> Diffs
> -
> 
>   src/Makefile.am d40378f8dc85a3b98d826e6516e17c05cd7ce174 
> 
> Diff: https://reviews.apache.org/r/40214/diff/
> 
> 
> Testing
> ---
> 
> ./configure
> make check
> make install
> 
> 
> Thanks,
> 
> Jie Yu
> 
>

Re: Review Request 40214: Cleaned up protobuf generation rules in src/Makefile.am.

2015-11-11 Thread Mesos ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40214/#review106193
---


Patch looks great!

Reviews applied: [40214]

All tests passed.

- Mesos ReviewBot


On Nov. 12, 2015, 1:31 a.m., Jie Yu wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40214/
> ---
> 
> (Updated Nov. 12, 2015, 1:31 a.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Cleaned up protobuf generation rules in src/Makefile.am.
> 
> Probobuf generation rules in src/Makefile.am is a mess. As we introduce more 
> and more protobuf files in the code base, this will soon become 
> unmaintainable. This is the first phase of the clean up. The next step is to 
> consolidate all the protobuf generation rules (instead of copy and paste).
> 
> 
> Diffs
> -
> 
>   src/Makefile.am d40378f8dc85a3b98d826e6516e17c05cd7ce174 
> 
> Diff: https://reviews.apache.org/r/40214/diff/
> 
> 
> Testing
> ---
> 
> ./configure
> make check
> make install
> 
> 
> Thanks,
> 
> Jie Yu
> 
>

Review Request 40214: Cleaned up protobuf generation rules in src/Makefile.am.

2015-11-11 Thread Jie Yu 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40214/
---

Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.


Repository: mesos


Description
---

Cleaned up protobuf generation rules in src/Makefile.am.

Probobuf generation rules in src/Makefile.am is a mess. As we introduce more 
and more protobuf files in the code base, this will soon become unmaintainable. 
This is the first phase of the clean up. The next step is to consolidate all 
the protobuf generation rules (instead of copy and paste).


Diffs
-

  src/Makefile.am d40378f8dc85a3b98d826e6516e17c05cd7ce174 

Diff: https://reviews.apache.org/r/40214/diff/


Testing
---

./configure
make check
make install


Thanks,

Jie Yu

Re: Review Request 40214: Cleaned up protobuf generation rules in src/Makefile.am.

2015-11-11 Thread Mesos ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40214/#review106192
---


Patch looks great!

Reviews applied: [40214]

All tests passed.

- Mesos ReviewBot


On Nov. 12, 2015, 1:31 a.m., Jie Yu wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40214/
> ---
> 
> (Updated Nov. 12, 2015, 1:31 a.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Cleaned up protobuf generation rules in src/Makefile.am.
> 
> Probobuf generation rules in src/Makefile.am is a mess. As we introduce more 
> and more protobuf files in the code base, this will soon become 
> unmaintainable. This is the first phase of the clean up. The next step is to 
> consolidate all the protobuf generation rules (instead of copy and paste).
> 
> 
> Diffs
> -
> 
>   src/Makefile.am d40378f8dc85a3b98d826e6516e17c05cd7ce174 
> 
> Diff: https://reviews.apache.org/r/40214/diff/
> 
> 
> Testing
> ---
> 
> ./configure
> make check
> make install
> 
> 
> Thanks,
> 
> Jie Yu
> 
>