+1 to release, however the following additions need to happen (as called
out in this thread):

- Need a clear way to run RAT
- Need to clarify how to build each component ideally from the top level
- You must *not* create external websites that are referencing the release
of the project.  The usage of NOLA Marketing needs to be dug into a bit
further.

John

On Thu, Aug 10, 2017 at 12:35 PM Raymundo Panduro <raypand...@gmail.com>
wrote:

> Hello IPMC members,
>
> Here are the instructions on how to build the release candidate once you
> download it.
>
> http://nolamarketing.com/client/apache-spot/download/ (If you don't see
> the
>  *Validate Build* section use CTRL + F5 to refresh the page).
>
>
> Regarding the Licence comment, we updated the files in preparation for next
> release:
>
> https://github.com/apache/incubator-spot/commit/ebb564ed058570d72e3b84fc418f7f87780efd44
>
> If there are other comments please let us know.
>
>
>
>
>
>
>
>
>
> *Raymundo Panduro*
>
> On Fri, Aug 4, 2017 at 6:37 PM, Justin Mclean <jus...@classsoftware.com>
> wrote:
>
> > Hi,
> >
> > +1 binding
> >
> > I checked:
> > - name includes incubating
> > - hashes and signatures correct
> > - DISCLAIMER exists
> > - LICENSE is missing a few permissive licensed bundled items. Please fix
> > for the next release
> > - NOTICE is good
> > - no unexpected binary files
> > - all source files have ASF headers
> > - Could compile from source (by running sbt assembly in spot_ml)
> >
> > Please include instruction on how to compile/build the release candidate.
> >
> > With the license it maybe a good idea rather than referring to anther
> > license file it would be better to just include it if it is short OR when
> > referencing to the other license file(s) at least mention what license it
> > is.
> >
> > LICENSE is missing:
> > - MIT licensed code here [1]
> > - Public domain license material here [2][3]
> >
> > In the README the Maintainers section in the read me seems a little odd
> to
> > me. I would also put the dev and users mailing lists (which are missing)
> > above the slack channel for community support. The link for JIRA also
> needs
> > fixing (should be just https://issues.apache.org/jira/projects/SPOT).
> >
> > Thanks,
> > Justin
> >
> > 1. spot-oa/ui/flow/js/lib/ChordMapper.js
> > 2. spot-oa/ui/flow/world-110m.json
> > 3. http://www.naturalearthdata.com/about/terms-of-use/
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> > For additional commands, e-mail: general-h...@incubator.apache.org
> >
> >
>

Reply via email to