On Thu, 5 Dec 2013, Jeff Law wrote: > On 12/05/13 03:18, Richard Biener wrote: > > > > This fixes the issue in linux/vt.h that appears in SUSE SLE11 kernel > > headers which contain a pre-release variant that is broken and not > > compatible with C++ (using the 'new' keyword). > > > > The following fix simply replaces that (and only that) field with > > 'newev', the upstream choice. > > > > fixinclude-tested and also built libsanitizer on a broken system. > > > > Ok for trunk? > > > > Thanks, > > Richard. > > > > 2013-12-05 Richard Biener <rguent...@suse.de> > > > > fixincludes/ > > * inclhack.def (suse_linux_vt_cxx): New fix for linux/vt.h > > being not compatible with C++. > > > > * tests/base/linux/vt.h: New test. > Looks correct to me. The README requests that we try to keep the hacks sorted > by their hackname and seems to do so with the exception of the last few hacks. > So you probably want to move this to the right spot in the file.
Done and committed. Richard.
