I think the source headers and LICENSE are in good shape. I was able to follow BUILDING.md on macOS. At some point it would be nice to provide a simple a quick start guide (e.g. How to create a geode-native app in 5 simple steps). Perhaps that is already included in the docs though.
Anthony > On Nov 5, 2018, at 10:06 AM, Ernest Burghardt <eburgha...@pivotal.io> wrote: > > geode-native should be ready to go now > > On Tue, Oct 30, 2018 at 2:56 PM Dave Barnes <dbar...@pivotal.io> wrote: > >> unix_index.html and win_index.html cannot simply be removed, as they're >> referenced by Cmake. I created a ticket to address this: >> https://issues.apache.org/jira/browse/GEODE-5958. >> >> On Tue, Oct 30, 2018 at 11:48 AM Dave Barnes <dbar...@pivotal.io> wrote: >> >>> Re: the docs links mentioned in Anthony's message: >>> >>> docs/README.md - the reference to the Geode CONTRIBUTE.md file is useful. >>> I think we can assume that a user of the geode-native API will have a >> Geode >>> installation, too. >>> docs/api/unix_index.html and docs/api/win_index.html - These files (and >>> their bad links) are no longer needed and can be removed. >>> >>> On Tue, Oct 30, 2018 at 7:48 AM Anthony Baker <aba...@pivotal.io> wrote: >>> >>>> I ran rat manually and got this: >>>> >>>> Files with unapproved licenses: >>>> >>>> /geode/.cpackignore >>>> /geode/.lcovrc >>>> /geode/clicache/src/native_shared_ptr.hpp >>>> /geode/templates/security/CMakeLists.txt.forInstall >>>> >>>> Shouldn’t these failures be causing travis to fail? >>>> >>>> Many of the files rat that marks as binary have this the annoying BOM >> [1] >>>> from visual studio. Seems like we should just remove those. That will >>>> reveal a number of files that need a license header. >>>> >>>> What is this file? Seems to be binary content: >>>> clicache/test/native_shared_ptrTests.cpp >>>> >>>> I also noticed this text that should be updated to point to where we >>>> intend to host the documentation (on the geode website): >>>> >>>> docs/README.md:The Geode-Native repository provides the full source for >>>> the Apache Geode Native Client User Guide in markdown format (see >>>> _geode-project-dir_/geode-docs/CONTRIBUTE.md for more information on >> how to >>>> use markdown in this context). Users can build the markdown into an HTML >>>> user guide using [Bookbinder](https://github.com/pivotal-cf/bookbinder) >>>> and the instructions below. >>>> docs/api/unix_index.html:Access documentation at <a href=" >>>> http://docs-gemfire-nativeclient-develop.cfapps.io" >>>> target="_blank"><b>Pivotal GemFire Native Client Documentation</b></a>. >>>> docs/api/win_index.html:Access documentation at <a href=" >>>> http://docs-gemfire-nativeclient-develop.cfapps.io" >>>> target="_blank"><b>Pivotal GemFire Native Client Documentation</b></a>. >>>> >>>> >>>> Anthony >>>> >>>> [1] https://en.wikipedia.org/wiki/Byte_order_mark >>>> >>>>> On Oct 29, 2018, at 12:11 PM, Jacob Barrett <jbarr...@pivotal.io> >>>> wrote: >>>>> >>>>> There are a lot of files that rat is seeing as binary files and >> ignoring >>>>> the headers. Many of those files are not binary. Perhaps they have the >>>>> wrong metadata associated with them in git as a result of bad commits. >>>> We >>>>> should clean all those files up so that rat is checking everything >>>>> correctly. >>>>> >>>>> On Thu, Oct 18, 2018 at 3:41 PM Dan Smith <dsm...@pivotal.io> wrote: >>>>> >>>>>> Following up on this - is there anything we still need to do before >> we >>>> cut >>>>>> the 1.8 release branch in 2 weeks? >>>>>> >>>>>> Overall I think the native client source code looks like it's in good >>>> shape >>>>>> - we're running rat, the LICENSE and NOTICE look good, no binaries in >>>> the >>>>>> source, etc. >>>>>> >>>>>> For cutting the release branch, do we have criteria other than >> passing >>>>>> travis [1] ? >>>>>> What will our release steps should look like for the native client - >>>> just >>>>>> tar up the source and sign it with gpg? >>>>>> >>>>>> If we can have this figured out ahead of time and at least >>>> provisionally >>>>>> added to the release steps [2] I think it will help this release go a >>>>>> little smoother. >>>>>> >>>>>> [1] https://travis-ci.org/apache/geode-native/branches >>>>>> [2] >>>>>> >>>> >> https://cwiki.apache.org/confluence/display/GEODE/Releasing+Apache+Geode >>>>>> >>>>>> On Thu, Oct 11, 2018 at 3:23 PM Dan Smith <dsm...@pivotal.io> wrote: >>>>>> >>>>>>> +1 for a source release. Awesome! >>>>>>> >>>>>>> -Dan >>>>>>> >>>>>>> On Thu, Oct 11, 2018 at 2:32 PM Michael Oleske <mole...@pivotal.io> >>>>>> wrote: >>>>>>> >>>>>>>> Plus 1 for source release. Exciting times we live in! >>>>>>>> >>>>>>>> For verifying, plus one to a pipeline that's not just travis. >> Though >>>>>>>> they're instructions in the repo about how to run tests to get that >>>>>>>> baseline confidence. >>>>>>>> >>>>>>>> -michael >>>>>>>> >>>>>>>> On Wednesday, October 10, 2018, Anilkumar Gingade < >>>> aging...@pivotal.io> >>>>>>>> wrote: >>>>>>>> >>>>>>>>> Good work team. >>>>>>>>> +1 to get this as part of Geode 1.8 release. >>>>>>>>> It will be good to see community taking advantage of this. And >>>>>> building >>>>>>>> new >>>>>>>>> native client apps. >>>>>>>>> I assume it will have all the docs about client-server >> compatibility >>>>>>>>> version info. And framework for backward compatibility testing >> with >>>>>> new >>>>>>>>> geode releases. >>>>>>>>> >>>>>>>>> -Anil. >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> On Wed, Oct 10, 2018 at 12:02 PM Ernest Burghardt < >>>>>>>> eburgha...@pivotal.io> >>>>>>>>> wrote: >>>>>>>>> >>>>>>>>>> +1 for a source release >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> On Wed, Oct 10, 2018 at 12:59 PM Anthony Baker < >> aba...@pivotal.io> >>>>>>>>> wrote: >>>>>>>>>> >>>>>>>>>>> I think starting with a source-only release of the native client >>>>>> is >>>>>>>> a >>>>>>>>>> good >>>>>>>>>>> first step. That lets us focus on verifying that all the tasks >>>>>>>>> outlined >>>>>>>>>> in >>>>>>>>>>> [1] are complete and correct. >>>>>>>>>>> >>>>>>>>>>> Anthony >>>>>>>>>>> >>>>>>>>>>> [1] https://issues.apache.org/jira/browse/GEODE-1416 >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>>> On Oct 10, 2018, at 11:52 AM, Dan Smith <dsm...@pivotal.io> >>>>>>>> wrote: >>>>>>>>>>>> >>>>>>>>>>>> That is awesome! Let's get it in! >>>>>>>>>>>> >>>>>>>>>>>> I think there are some details to work out: >>>>>>>>>>>> - Do we need to build any automation for creating the native >>>>>>>> source >>>>>>>>>>>> release (similar to ./gradlew srcDist on the java side)? >>>>>>>>>>>> - Will we release binaries? Which platforms and how to does the >>>>>>>>> release >>>>>>>>>>>> manager build them? >>>>>>>>>>>> - How do we verify the NC code - can we create a public >>>>>> pipeline? >>>>>>>>>>>> >>>>>>>>>>>> Shipping these native APIs will be a great improvement! >>>>>>>>>>>> >>>>>>>>>>>> -Dan >>>>>>>>>>>> >>>>>>>>>>>> On Wed, Oct 10, 2018 at 8:41 AM Addison Huddy < >>>>>> ahu...@pivotal.io> >>>>>>>>>> wrote: >>>>>>>>>>>> >>>>>>>>>>>>> Hi, >>>>>>>>>>>>> >>>>>>>>>>>>> The Geode Native components ( >>>>>>>> https://github.com/apache/geode-native >>>>>>>>> ) >>>>>>>>>>> have >>>>>>>>>>>>> made tremendous progress since its original donation to >> Apache. >>>>>>>> The >>>>>>>>>>>>> project is nearing a release candidate and I propose that the >>>>>>>> *first >>>>>>>>>>>>> official release of Geode Native be included in Apache Geode >>>>>>>> 1.8.* >>>>>>>>>>>>> >>>>>>>>>>>>> Since donation, the project has >>>>>>>>>>>>> >>>>>>>>>>>>> - modernized its C++ API based on C++ 11 standards >>>>>>>>>>>>> - refactored away the cache singleton to allow for more >>>>>>>> flexible >>>>>>>>>>>>> architectures and client-side data modeling >>>>>>>>>>>>> - refactored the serializable interfaces (DataSerializable, >>>>>>>>>>>>> PdxSerializable, DataSerializableFixedId) to make object >>>>>>>>>> serialization >>>>>>>>>>>>> more straight-forward >>>>>>>>>>>>> - created several examples on how to use the client ( >>>>>>>>>>>>> https://github.com/apache/geode-native/tree/develop/examples >>>>>> ) >>>>>>>>>>>>> >>>>>>>>>>>>> In all, the project has closed 285 JIRA tickets since >> donation. >>>>>>>>>>>>> >>>>>>>>>>>>> If you want to learn more about the Geode Native, check out >>>>>> these >>>>>>>>> two >>>>>>>>>>>>> Apache Geode By Example videos. >>>>>>>>>>>>> >>>>>>>>>>>>> .NET: https://www.youtube.com/watch?v=-LQYNJNQ7B4&t=3s >>>>>>>>>>>>> >>>>>>>>>>>>> C++: https://www.youtube.com/watch?v=KJciEcFRdtY&t=1s >>>>>>>>>>>>> >>>>>>>>>>>>> Looking forward to hearing your input on including the first >>>>>> cut >>>>>>>> of >>>>>>>>>>> Geode >>>>>>>>>>>>> Native in Apache Geode 1.8. >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> Best, >>>>>>>>>>>>> Addison >>>>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>> >>>>>> >>>> >>>> >>