On Thu, Aug 4, 2016, at 15:08, James Aylett wrote:> On Wed, Aug 03, 2016 at 08:17:05PM +0200, rsto at paranoia.at wrote: > > I'll notify you when the CJK pull request passes Travis. > > That's great, thanks!Alright, after lots of fiddling with .travis.yml I finally made the pull request build on Travis' trusty image: https://github.com/xapian/xapian/pull/114 I have kept ICU/pkg-config mandatory. Most probably that will have to change before merge into master, but I'd rather first wait for feedback on the code. The .travis.yml-related commits could then be squashed as well. Looking forward to your feedback. Cheers, Robert
The CJK word segmentation and snippet pull requests both pass Travis since middle/end of last week. Did you find time to look at them? Or is their merge pending decisions on other features/pull requests? I can keep the PRs rebased against master, but I will wait for your feedback on the current tranche before migrating remaining features on top of them. Thanks, Robert P.S. They are at https://github.com/xapian/xapian/pull/114 and https://github.com/xapian/xapian/pull/117
Hi, On Thu, Aug 11, 2016, at 13:19, rsto at paranoia.at wrote:> The CJK word segmentation and snippet pull requests both pass Travis > since middle/end of last week. Did you find time to look at them?just checking in if you found time to look at the PRs? It'd be nice to know a tentative timeline, so I can plan if to build next features on top of our local fork or the upstream PRs. Thanks, Robert