---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45622/#review127231
---
Ship it!
Ship It!
- Ben Mahler
On April 2, 2016, 9:35 p.m., Kevin Klues wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45622/
> ---
>
> (Updated April 2, 2016, 9:35 p.m.)
>
>
> Review request for mesos, Ben Mahler and switched to 'mcypark'.
>
>
> Bugs: MESOS-5082
> https://issues.apache.org/jira/browse/MESOS-5082
>
>
> Repository: mesos
>
>
> Description
> ---
>
> There appears to be a discrepancy between clang and gcc, which allows
> clang to accept `using` declarations of the form `using ns_name::name;`
> that contain nested classes, structs, and enums after the `name` field
> in the declaration (e.g. `using ns_name::name::enum;`).
>
> The language for describing this functionality is ambiguous in the
> C++11 specification as referenced here:
> http://en.cppreference.com/w/cpp/language/namespace#Using-declarations
>
> This patch fixes a bug where we relied on this functionality in clang
> and had build errors in gcc.
>
>
> Diffs
> -
>
> src/linux/cgroups.hpp 53397fc1da11a71259df1c38526a613676a60301
> src/slave/containerizer/mesos/isolators/cgroups/devices/gpus/nvidia.cpp
> c2cdc8fde7a85741be6494ea664d3719d1f13a43
>
> Diff: https://reviews.apache.org/r/45622/diff/
>
>
> Testing
> ---
>
> Rebuilt with nvidia gpu support for both clang and gcc.
>
>
> Thanks,
>
> Kevin Klues
>
>