Re: Review Request 40012: Libprocess: Corrected style of Makefiles.

2015-11-07 Thread Alexander Rukletsov

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

(Updated Nov. 7, 2015, 12:50 p.m.)


Review request for mesos and Till Toenshoff.


Bugs: MESOS-3849
https://issues.apache.org/jira/browse/MESOS-3849


Repository: mesos


Description
---

This is not a full standartization of Makefile format, however, it is a step 
towards consistency. This patch focuses mainly on the order of files and 
corrects obvious issues. These are guidelines behind the change:

- Files in the same folder should appear in lexicographical order.
- Subfolders should appear in lexicographical order either after files or mixed 
with them (depending on the current consensus in the file).
- Though tabs are preferred, spaces are not replaced if they are current 
consensus in the file or current section.


Diffs
-

  3rdparty/libprocess/3rdparty/Makefile.am 
53e83d4905945593e174601a0b791d01824dc34b 
  3rdparty/libprocess/Makefile.am 404a7438e7cd53d9295fe6025b5af5fb83df939b 
  3rdparty/libprocess/include/Makefile.am 
fcc62e99b92b9d2e7ab344e561a06dd6de1fef7e 

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


Testing
---

make check (Mac OS X 10.10.4)
make distcheck (CentOS 7.1.1503)


Thanks,

Alexander Rukletsov



Review Request 40012: Libprocess: Corrected style of Makefiles.

2015-11-06 Thread Alexander Rukletsov

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

Review request for mesos and Till Toenshoff.


Repository: mesos


Description
---

This is not a full standartization of Makefile format, however, it is a step 
towards consistency. This patch focuses mainly on the order of files and 
corrects obvious issues. These are guidelines behind the change:

- Files in the same folder should appear in lexicographical order.
- Subfolders should appear in lexicographical order either after files or mixed 
with them (depending on the current consensus in the file).
- Though tabs are preferred, spaces are not replaced if they are current 
consensus in the file or current section.


Diffs
-

  3rdparty/libprocess/3rdparty/Makefile.am 
53e83d4905945593e174601a0b791d01824dc34b 
  3rdparty/libprocess/Makefile.am 404a7438e7cd53d9295fe6025b5af5fb83df939b 
  3rdparty/libprocess/include/Makefile.am 
fcc62e99b92b9d2e7ab344e561a06dd6de1fef7e 

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


Testing
---

make check (Mac OS X 10.10.4)


Thanks,

Alexander Rukletsov



Re: Review Request 40012: Libprocess: Corrected style of Makefiles.

2015-11-06 Thread Alexander Rukletsov

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

(Updated Nov. 6, 2015, 4:22 p.m.)


Review request for mesos and Till Toenshoff.


Changes
---

Rebased, run distcheck.


Repository: mesos


Description
---

This is not a full standartization of Makefile format, however, it is a step 
towards consistency. This patch focuses mainly on the order of files and 
corrects obvious issues. These are guidelines behind the change:

- Files in the same folder should appear in lexicographical order.
- Subfolders should appear in lexicographical order either after files or mixed 
with them (depending on the current consensus in the file).
- Though tabs are preferred, spaces are not replaced if they are current 
consensus in the file or current section.


Diffs (updated)
-

  3rdparty/libprocess/3rdparty/Makefile.am 
53e83d4905945593e174601a0b791d01824dc34b 
  3rdparty/libprocess/Makefile.am 404a7438e7cd53d9295fe6025b5af5fb83df939b 
  3rdparty/libprocess/include/Makefile.am 
fcc62e99b92b9d2e7ab344e561a06dd6de1fef7e 

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


Testing (updated)
---

make check (Mac OS X 10.10.4)
make distcheck (CentOS 7.1.1503)


Thanks,

Alexander Rukletsov



Re: Review Request 40012: Libprocess: Corrected style of Makefiles.

2015-11-06 Thread Till Toenshoff

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

Ship it!


Ship It!

- Till Toenshoff


On Nov. 6, 2015, 11:07 a.m., Alexander Rukletsov wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40012/
> ---
> 
> (Updated Nov. 6, 2015, 11:07 a.m.)
> 
> 
> Review request for mesos and Till Toenshoff.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> This is not a full standartization of Makefile format, however, it is a step 
> towards consistency. This patch focuses mainly on the order of files and 
> corrects obvious issues. These are guidelines behind the change:
> 
> - Files in the same folder should appear in lexicographical order.
> - Subfolders should appear in lexicographical order either after files or 
> mixed with them (depending on the current consensus in the file).
> - Though tabs are preferred, spaces are not replaced if they are current 
> consensus in the file or current section.
> 
> 
> Diffs
> -
> 
>   3rdparty/libprocess/3rdparty/Makefile.am 
> 53e83d4905945593e174601a0b791d01824dc34b 
>   3rdparty/libprocess/Makefile.am 404a7438e7cd53d9295fe6025b5af5fb83df939b 
>   3rdparty/libprocess/include/Makefile.am 
> fcc62e99b92b9d2e7ab344e561a06dd6de1fef7e 
> 
> Diff: https://reviews.apache.org/r/40012/diff/
> 
> 
> Testing
> ---
> 
> make check (Mac OS X 10.10.4)
> 
> 
> Thanks,
> 
> Alexander Rukletsov
> 
>