Can we replace all of these hardcoded 1.0.0 and 1.0.0-incubating strings
with a replacement token that is updated in the artifacts for a release?


On Fri, Feb 10, 2017 at 2:46 PM, Karen Miller <kmil...@apache.org> wrote:

> +0 for me as well.  I had also noted that the documentation has a couple of
> prominent
> and incorrect instances of 1.0.0-incubating.  If there were to be a release
> candidate 3,
> let's for sure get as many of the incorrect instances of 1.0.0-incubating
> fixed as possible.
> However, the 2 within the reference manual (user guide) do not change the
> functionality
> of the SW:
>
> geode-docs/about_geode.html.md.erb:This documentation describes product
> concepts and provides complete setup instructions for Apache Geode
> 1.0.0-incubating. Source files are ava
> geode-docs/prereq_and_install.html.md.erb:Each host of Apache Geode
> 1.0.0-incubating that meets a small set of prerequisites may follow the
> provided installation instructions.
>
>
> On Fri, Feb 10, 2017 at 2:30 PM, Hitesh Khamesra <
> hitesh...@yahoo.com.invalid> wrote:
>
> > Here are all the reference of incubating in release branch 1.1.0. Please
> > look into this..
> >
> > -bash-4.2$ git grep 'incubat'
> > BUILDING.md:    v1.0.0-incubating
> > BUILDING.md:    v1.0.0-incubating
> > dev-tools/docker/compile/start-compile-docker.sh:  -w
> > "/home/${USER_NAME}/incubator-geode" \
> > dev-tools/docker/compile/start-compile-docker.sh:  -v
> > "$PWD:/home/${USER_NAME}/incubator-geode" \
> > dev-tools/docker/docs/build-docs.sh:  -w "/home/${USER_NAME}/incubator-
> geode/geode-book"
> > \
> > dev-tools/docker/docs/build-docs.sh:  -v "$PWD:/home/${USER_NAME}/
> incubator-geode"
> > \
> > dev-tools/docker/docs/view-docs.sh:  -w "/home/${USER_NAME}/incubator-
> > geode/geode-book/final_app/public" \
> > dev-tools/docker/docs/view-docs.sh:  -v "$PWD:/home/${USER_NAME}/
> incubator-geode"
> > \
> > docker/Dockerfile:      && git clone https://github.com/apache/
> > incubator-geode.git \
> > docker/Dockerfile:      && cd incubator-geode \
> > docker/Dockerfile:      && git checkout rel/v1.0.0-incubating \
> > docker/Dockerfile:      && ls /incubator-geode | grep -v geode-assembly |
> > xargs rm -rf \
> > docker/Dockerfile:      && rm -rf /incubator-geode/geode-
> assembly/build/distributions/
> > \
> > docker/Dockerfile:ENV GEODE_HOME /incubator-geode/geode-
> > assembly/build/install/apache-geode
> > docker/README.md:docker build -t apachegeode/geode:1.0.0-incubating .
> > docker/README.md:docker push apachegeode/geode:1.0.0-incubating
> > docker/README.md:docker run -it -p 10334:10334 -p 7575:7575 -p 1099:1099
> > apache/geode:1.0.0-incubating
> > geode-core/src/main/java/org/apache/geode/internal/cache/
> AbstractLRURegionMap.java:
> > * modification for LIFO Logic incubation
> > geode-docs/about_geode.html.md.erb:This documentation describes product
> > concepts and provides complete setup instructions for Apache Geode
> > 1.0.0-incubating. Source files are ava
> > geode-docs/prereq_and_install.html.md.erb:Each host of Apache Geode
> > 1.0.0-incubating that meets a small set of prerequisites may follow the
> > provided installation instructions.
> > geode-old-versions/build.gradle:addTestSource('test100',
> > '1.0.0-incubating')
> > geode-site/website/content/docs/guide/10/about_geode.html:      Apache
> > Geode 1.0.0-incubating Documentation |
> > geode-site/website/content/docs/guide/10/about_geode.html:    Apache
> > Geode 1.0.0-incubating Documentation
> > geode-site/website/content/docs/guide/10/about_geode.html:<p>This
> > documentation describes product concepts and provides complete setup
> > instructions for Apache Geode 1.0.0-incuba
> > geode-site/website/content/docs/guide/10/getting_started/
> book_intro.html:<p>Each
> > host of Apache Geode 1.0.0-incubating that meets a small set of
> > prerequisites may follow the pro
> > geode-site/website/content/docs/guide/10/getting_started/
> > installation/install_standalone.html:<li><p>Download the project source
> > from the Releases page found at <a href="http://
> > geode-site/website/content/docs/guide/10/getting_started/
> > installation/install_standalone.html:v1.0.0-incubating
> > geode-site/website/content/docs/guide/10/getting_started/
> > installation/install_standalone.html:<li><p>Download the project source
> > from the Releases page found at <a href="http://
> > geode-site/website/content/docs/guide/10/getting_started/
> > installation/install_standalone.html:v1.0.0-incubating
> > geode-site/website/content/docs/guide/10/getting_started/
> > installation/install_standalone.html:<li> Download the .zip or .tar file
> > from the Releases page found at <a href="http:/
> > geode-site/website/content/docs/guide/10/getting_started/
> > installation/install_standalone.html:<pre class="highlight
> > plaintext"><code>$ unzip apache-geode-1.0.0-incubating.zip -d
> > geode-site/website/content/docs/guide/10/getting_started/
> > installation/install_standalone.html:<pre class="highlight
> > plaintext"><code>$ tar -xvf apache-geode-1.0.0-incubating.tar
> > geode-site/website/content/docs/guide/10/getting_started/
> > installation/install_standalone.html:v1.0.0-incubating
> > geode-site/website/content/docs/guide/10/prereq_and_install.html:<p>Each
> > host of Apache Geode 1.0.0-incubating that meets a small set of
> > prerequisites may follow the provided in
> > geode-site/website/content/releases/index.html:        <h3><span
> > class="icon-flag-checkered"> 1.0.0-incubating</span></h3>
> > geode-site/website/content/releases/index.html:              [ <a href="
> > http://apache.org/dyn/closer.cgi/geode/1.0.0-
> > incubating/apache-geode-1.0.0-incubating.zip">ZIP</a>, <a hr
> > geode-site/website/content/releases/index.html:              <a href="
> > https://dist.apache.org/repos/dist/release/geode/1.0.
> > 0-incubating/apache-geode-1.0.0-incubating.zip.asc">PG
> > geode-site/website/content/releases/index.html:              [<a href="
> > http://apache.org/dyn/closer.cgi/geode/1.0.0-
> > incubating/apache-geode-1.0.0-incubating.tar.gz">TAR.GZ</a>,
> > geode-site/website/content/releases/index.html:              <a href="
> > https://dist.apache.org/repos/dist/release/geode/1.0.
> > 0-incubating/apache-geode-1.0.0-incubating.tar.gz.asc"
> > geode-site/website/content/releases/index.html:               [ <a
> href="
> > http://apache.org/dyn/closer.cgi/geode/1.0.0-
> > incubating/apache-geode-src-1.0.0-incubating.zip">ZIP</a>,
> > geode-site/website/content/releases/index.html:               <a href="
> > https://dist.apache.org/repos/dist/release/geode/1.0.
> > 0-incubating/apache-geode-src-1.0.0-incubating.zip.as
> > geode-site/website/content/releases/index.html:               [<a href="
> > http://apache.org/dyn/closer.cgi/geode/1.0.0-
> > incubating/apache-geode-src-1.0.0-incubating.tar.gz">TAR.GZ<
> > geode-site/website/content/releases/index.html:               <a href="
> > https://dist.apache.org/repos/dist/release/geode/1.0.
> > 0-incubating/apache-geode-src-1.0.0-incubating.tar.gz
> > geode-site/website/content/releases/index.html:            <strong><a
> > href="https://cwiki.apache.org/confluence/display/GEODE/
> > Release+Notes#ReleaseNotes-1.0.0-incubating">Releas
> > geode-site/website/content/releases/index.html:      compile
> > 'org.apache.geode:geode-core:1.0.0-incubating'
> > geode-site/website/content/releases/index.html:
> > &lt;version&gt;1.0.0-incubating&lt;/version&gt;
> > geode-site/website/content/releases/index.html:      <p>If you need
> > access to older releases they can be found in the <a href="
> > https://archive.apache.org/dist/incubator/geode/";>
> > geode-spark-connector/project/Dependencies.scala:    val geode =
> > "org.apache.geode" % "geode-core" % "1.0.0-incubating"
> > excludeAll(ExclusionRule(organization = "org.jboss.netty"
> >
> >
> >       From: Galen M O'Sullivan <gosulli...@pivotal.io>
> >  To: dev@geode.apache.org
> > Cc: Hitesh Khamesra <hitesh...@yahoo.com>
> >  Sent: Friday, February 10, 2017 1:59 PM
> >  Subject: Re: [VOTE] RC2: Apache Geode release - v1.1.0
> >
> > Good catch, Ken. It looks like the Dockerfile and Docker scripts refer to
> > 'incubat*', and a few places in the docs do as well.
> >
> > On Fri, Feb 10, 2017 at 1:35 PM, Kenneth Howe <kh...@pivotal.io> wrote:
> >
> > > +0
> > > I’m not going to give an outright down-vote, but BUILDING.md needs to
> be
> > > updated to refer to the correct versions.
> > >
> > > The build steps show the expected response to
> > >  gfsh version
> > > to be "v1.0.0-incubating”
> > >
> > > It’s not great to show the wrong version, but since this is the first
> > > release since graduation showing an incorrect “incubating” version tag
> is
> > > particularly disturbing. I’d hate to confuse any new users kicking the
> > > tires on geode now that it’s an Apache TLP.
> > >
> > > Ken
> > >
> > > > On Feb 9, 2017, at 12:55 PM, Hitesh Khamesra
> > <hitesh...@yahoo.com.INVALID>
> > > wrote:
> > > >
> > > > All,
> > > >
> > > > This is the second release candidate of the first release for Apache
> > > Geode, version 1.1.0.
> > > > Thanks to all the community members.
> > > >
> > > > It fixes the following issues:
> > > >    https://issues.apache.org/jira/secure/ReleaseNote.jspa?
> > > projectId=12318420&version=12338352
> > > >
> > > >
> > > > *** Please download, test and vote by Tuesday, February 14, 0800 hrs
> US
> > > Pacific.
> > > >
> > > > Note that we are voting upon the source (tag):
> > > >    rel/v1.1.0.RC2
> > > >    https://git-wip-us.apache.org/repos/asf?p=geode.git;a=tag;h=
> > > refs/tags/rel/v1.1.0.RC2
> > > >
> > > > Commit ID:    2286fd064a52173eab8fdcfadfb89a01e81ef728
> > > >
> > > > Source and binary files:
> > > >    https://dist.apache.org/repos/dist/dev/geode/1.1.0.RC2/
> > > >
> > > > Maven staging repo:
> > > >    https://repository.apache.org/content/repositories/
> > > orgapachegeode-1016/
> > > >
> > > > Geode's KEYS file containing PGP keys we use to sign the release:
> > > >    https://github.com/apache/geode/blob/release/1.1.0/KEYS
> > > >
> > > > pub  4096R/AC6AB8ED 2017-01-18
> > > >      Key fingerprint = 048F B40F AD7D 9C15 54AD  D447 2CA9 90A2 AC6A
> > > B8ED
> > > >
> > > > Thanks,
> > > > Hitesh
> > >
> > >
> >
> >
> >
>

Reply via email to