Hi Kees, You added some support for Minix to LLVM's copy of googletest: http://llvm.org/viewvc/llvm-project?view=rev&revision=100895 Is it still being used? If so, would you mind pushing these patches upstream to the googletest project: http://code.google.com/p/googletest/ Otherwise we may drop them, because it makes it harder to import new releases of googletest into the LLVM repository. Thanks, Jay.
Hello Jay, At the time I submitted those patches I also submitted them to googletest. As far as I know they got accepted, although I didn't verify that. I am no longer working on the Minix project, but since the Minix project is most definitely using llvm, I am cc:-ing this to the Minix developers mailing list. Perhaps they want to respond to this. -- Kees On 27 Jul 2011, at 11:43, Jay Foad wrote:> Hi Kees, > > You added some support for Minix to LLVM's copy of googletest: > > http://llvm.org/viewvc/llvm-project?view=rev&revision=100895 > > Is it still being used? If so, would you mind pushing these patches > upstream to the googletest project: > > http://code.google.com/p/googletest/ > > Otherwise we may drop them, because it makes it harder to import new > releases of googletest into the LLVM repository. > > Thanks, > Jay.
All, Jay - and Kees -, thank you for notifying us of this move. Yes, as Kees wrote we certainly are using llvm/gtest, but are prepared to submit our changes to gtest directly if they haven't taken them yet. I'll take it up with my GSOC student, who is working on submitting many other minix-specific patches to maintainers of packages in our third-party package repository. =Ben
On 27 July 2011 11:20, dr. ir. Kees van Reeuwijk <reeuwijk at few.vu.nl> wrote:> At the time I submitted those patches I also submitted them to googletest. As far as I know they got accepted, although I didn't verify that.OK, I see from this thread: http://groups.google.com/group/googletestframework/browse_thread/thread/3081a14965764366/332ce9869f4bbe31 ... that rather than add explicit support for Minix, they suggested that you build with CPPFLAGS="-DGTEST_HAS_POSIX_REGEX=1 -DGTEST_HAS_STD_WSTRING=0". I'm not sure what (if anything) we have to do in LLVM to support that. Jay.
Apparently Analagous Threads
- [LLVMdev] Minix support in googletest
- [LLVMdev] Minix support in googletest
- [LLVMdev] [Minix-office] Re: Minix support in googletest
- [LLVMdev] [PATCH][RFC] upgrade to googletest 1.6.0 (was Re: Build of latest llvm gives warning and error)
- [LLVMdev] [PATCH][RFC] upgrade to googletest 1.6.0 (was Re: Build of latest llvm gives warning and error)