Attached you can find an updated restrict_arr patch which instead of just whitelisting sun, it adds a configure check for features.h file. See bug #80287 to see why that whitelisting was added. There's also a patch which adds a check for libintl which is needed to complete the linking on systems where libintl is not included in libc. Finally, the patch for the ebuild adds the two patches, adds an autogen script, and as I was there, I've removed uclibc useflag and used elibc_uclibc to achieve the same thing. Thanks, Diego
Created attachment 58864 [details, diff] Ebuild patch
Created attachment 58865 [details, diff] grep-2.5.1-restrict_arr.patch
Created attachment 58866 [details, diff] grep-2.5.1-libintl.patch
are you going to send these fixes upstream ?
Not sure about restrict_arr because I don't started with it, I just fixed it to work on G/fbsd... About libintl, I think I'll try in the next days.
i meant the features.h part of the restrict_arr patch ... the other part with the uclibc define we'll just keep in Gentoo
The features.h part is not in plain grep, it was on a gentoo patch...
Created attachment 61125 [details, diff] grep-2.5.1-libintl.patch please try this patch
Comment on attachment 58865 [details, diff] grep-2.5.1-restrict_arr.patch i updated the patch to simply undefine __restrict_arr and then define it again as null ... don't need to include features.h in this case
It re-creates configure after the patch but seems to work fine anyway.
so drop the configure.in hunk of the patch and try again
Works, but that's not good anyway because it will have problems if in the future configure is regenerated for other patches.
added to 2.5.1-r8