Re: Review Request 45827: GEODE-902: Remove incorrect src header; update LICENSE and NOTICE
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45827/#review127851 --- Ship it! Ship It! - Joey McAllister On April 6, 2016, 6:07 p.m., Anthony Baker wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45827/ > --- > > (Updated April 6, 2016, 6:07 p.m.) > > > Review request for geode, Joey McAllister and Dan Smith. > > > Repository: geode > > > Description > --- > > GEODE-902: Remove incorrect src header > > GEODE-902: Update LICENSE / NOTICE for MX4JModelMBean.java > > Added a pointer to the MX4J license and an entry in the NOTICE > file for both the source and binary distributions. This update > is an interim change as we are reworking the files for the M2 > release. > > > Diffs > - > > LICENSE 31f4473161272ff3e1aa01f2ed9eecbee013531c > NOTICE 5396eaa37a8ab5e54f689b60d28292604c2b3a88 > geode-assembly/src/main/dist/LICENSE > 2f1374b190fa2fd9d42610ad597c0980e22d5b24 > geode-assembly/src/main/dist/NOTICE > b42971539456ffb5778c60d124c9fe17e7152b31 > > geode-core/src/main/java/com/gemstone/gemfire/admin/jmx/internal/MX4JModelMBean.java > 1add1ecc56912991f23d441b6677b85e5f2867db > > Diff: https://reviews.apache.org/r/45827/diff/ > > > Testing > --- > > > Thanks, > > Anthony Baker > >
Re: Review Request 45827: GEODE-902: Remove incorrect src header; update LICENSE and NOTICE
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45827/#review127455 --- Ship it! Ship It! - Mark Bretl On April 6, 2016, 11:07 a.m., Anthony Baker wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45827/ > --- > > (Updated April 6, 2016, 11:07 a.m.) > > > Review request for geode, Joey McAllister and Dan Smith. > > > Repository: geode > > > Description > --- > > GEODE-902: Remove incorrect src header > > GEODE-902: Update LICENSE / NOTICE for MX4JModelMBean.java > > Added a pointer to the MX4J license and an entry in the NOTICE > file for both the source and binary distributions. This update > is an interim change as we are reworking the files for the M2 > release. > > > Diffs > - > > LICENSE 31f4473161272ff3e1aa01f2ed9eecbee013531c > NOTICE 5396eaa37a8ab5e54f689b60d28292604c2b3a88 > geode-assembly/src/main/dist/LICENSE > 2f1374b190fa2fd9d42610ad597c0980e22d5b24 > geode-assembly/src/main/dist/NOTICE > b42971539456ffb5778c60d124c9fe17e7152b31 > > geode-core/src/main/java/com/gemstone/gemfire/admin/jmx/internal/MX4JModelMBean.java > 1add1ecc56912991f23d441b6677b85e5f2867db > > Diff: https://reviews.apache.org/r/45827/diff/ > > > Testing > --- > > > Thanks, > > Anthony Baker > >
Re: Review Request 45827: GEODE-902: Remove incorrect src header; update LICENSE and NOTICE
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45827/#review127394 --- Ship it! Ship It! - Dan Smith On April 6, 2016, 6:07 p.m., Anthony Baker wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45827/ > --- > > (Updated April 6, 2016, 6:07 p.m.) > > > Review request for geode, Joey McAllister and Dan Smith. > > > Repository: geode > > > Description > --- > > GEODE-902: Remove incorrect src header > > GEODE-902: Update LICENSE / NOTICE for MX4JModelMBean.java > > Added a pointer to the MX4J license and an entry in the NOTICE > file for both the source and binary distributions. This update > is an interim change as we are reworking the files for the M2 > release. > > > Diffs > - > > LICENSE 31f4473161272ff3e1aa01f2ed9eecbee013531c > NOTICE 5396eaa37a8ab5e54f689b60d28292604c2b3a88 > geode-assembly/src/main/dist/LICENSE > 2f1374b190fa2fd9d42610ad597c0980e22d5b24 > geode-assembly/src/main/dist/NOTICE > b42971539456ffb5778c60d124c9fe17e7152b31 > > geode-core/src/main/java/com/gemstone/gemfire/admin/jmx/internal/MX4JModelMBean.java > 1add1ecc56912991f23d441b6677b85e5f2867db > > Diff: https://reviews.apache.org/r/45827/diff/ > > > Testing > --- > > > Thanks, > > Anthony Baker > >
Review Request 45827: GEODE-902: Remove incorrect src header; update LICENSE and NOTICE
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45827/ --- Review request for geode and Dan Smith. Repository: geode Description --- GEODE-902: Remove incorrect src header GEODE-902: Update LICENSE / NOTICE for MX4JModelMBean.java Added a pointer to the MX4J license and an entry in the NOTICE file for both the source and binary distributions. This update is an interim change as we are reworking the files for the M2 release. Diffs - LICENSE 31f4473161272ff3e1aa01f2ed9eecbee013531c NOTICE 5396eaa37a8ab5e54f689b60d28292604c2b3a88 geode-assembly/src/main/dist/LICENSE 2f1374b190fa2fd9d42610ad597c0980e22d5b24 geode-assembly/src/main/dist/NOTICE b42971539456ffb5778c60d124c9fe17e7152b31 geode-core/src/main/java/com/gemstone/gemfire/admin/jmx/internal/MX4JModelMBean.java 1add1ecc56912991f23d441b6677b85e5f2867db Diff: https://reviews.apache.org/r/45827/diff/ Testing --- Thanks, Anthony Baker
Re: LICENSE and NOTICE
I think Anthony meant 'feature/GEODE-610' branch: https://github.com/apache/incubator-geode/tree/feature/GEODE-610 - Nitin From: Michael Stolz <mst...@pivotal.io> Sent: Monday, January 4, 2016 9:55 AM To: dev@geode.incubator.apache.org Subject: Re: LICENSE and NOTICE I don't see a GEODE-210 branch. Am I missing something? -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: 631-835-4771 On Mon, Jan 4, 2016 at 11:19 AM, Anthony Baker <aba...@pivotal.io> wrote: > Look on the feature/GEODE-210 branch. > > The source distribution versions are in the root directory. > The binary distribution versions are in gemfire-assembly/src/main/dist/ > > Each distribution has different requirements since different external > components are bundled. > > Anthony > > > On Jan 4, 2016, at 8:17 AM, Dave Barnes <dbar...@pivotal.io> wrote: > > > > Where can I find these files? > > > > On Mon, Jan 4, 2016 at 7:16 AM, Anthony Baker <aba...@pivotal.io> wrote: > > > >> I’ve made a start on our LICENSE and NOTICE files (for both source and > >> binary distributions) on the feature/GEODE-610 branch. Thanks to Niall > for > >> the excellent analysis to get this started. If you’d like to > contribute to > >> this you can: > >> > >> 1) Review the contents of the LICENSE files and see if they look right. > >> 2) Figure out what needs to go in the NOTICE files. > >> > >> This is needed for the alpha1 release. > >> > >> Thanks, > >> Anthony > >> > >> > >
Re: LICENSE and NOTICE
Oops! Thanks for catching the typo. Anthony > On Jan 4, 2016, at 10:43 AM, Nitin Lamba <ni...@ampool.io> wrote: > > I think Anthony meant 'feature/GEODE-610' branch: > https://github.com/apache/incubator-geode/tree/feature/GEODE-610 > > - Nitin > > > From: Michael Stolz <mst...@pivotal.io> > Sent: Monday, January 4, 2016 9:55 AM > To: dev@geode.incubator.apache.org > Subject: Re: LICENSE and NOTICE > > I don't see a GEODE-210 branch. Am I missing something? > > -- > Mike Stolz > Principal Engineer, GemFire Product Manager > Mobile: 631-835-4771 > > On Mon, Jan 4, 2016 at 11:19 AM, Anthony Baker <aba...@pivotal.io> wrote: > >> Look on the feature/GEODE-210 branch. >> >> The source distribution versions are in the root directory. >> The binary distribution versions are in gemfire-assembly/src/main/dist/ >> >> Each distribution has different requirements since different external >> components are bundled. >> >> Anthony >> >>> On Jan 4, 2016, at 8:17 AM, Dave Barnes <dbar...@pivotal.io> wrote: >>> >>> Where can I find these files? >>> >>> On Mon, Jan 4, 2016 at 7:16 AM, Anthony Baker <aba...@pivotal.io> wrote: >>> >>>> I’ve made a start on our LICENSE and NOTICE files (for both source and >>>> binary distributions) on the feature/GEODE-610 branch. Thanks to Niall >> for >>>> the excellent analysis to get this started. If you’d like to >> contribute to >>>> this you can: >>>> >>>> 1) Review the contents of the LICENSE files and see if they look right. >>>> 2) Figure out what needs to go in the NOTICE files. >>>> >>>> This is needed for the alpha1 release. >>>> >>>> Thanks, >>>> Anthony >>>> >>>> >> >> signature.asc Description: Message signed with OpenPGP using GPGMail
LICENSE and NOTICE
I’ve made a start on our LICENSE and NOTICE files (for both source and binary distributions) on the feature/GEODE-610 branch. Thanks to Niall for the excellent analysis to get this started. If you’d like to contribute to this you can: 1) Review the contents of the LICENSE files and see if they look right. 2) Figure out what needs to go in the NOTICE files. This is needed for the alpha1 release. Thanks, Anthony signature.asc Description: Message signed with OpenPGP using GPGMail
Re: LICENSE and NOTICE
Where can I find these files? On Mon, Jan 4, 2016 at 7:16 AM, Anthony Baker <aba...@pivotal.io> wrote: > I’ve made a start on our LICENSE and NOTICE files (for both source and > binary distributions) on the feature/GEODE-610 branch. Thanks to Niall for > the excellent analysis to get this started. If you’d like to contribute to > this you can: > > 1) Review the contents of the LICENSE files and see if they look right. > 2) Figure out what needs to go in the NOTICE files. > > This is needed for the alpha1 release. > > Thanks, > Anthony > >
Re: LICENSE and NOTICE
I don't see a GEODE-210 branch. Am I missing something? -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: 631-835-4771 On Mon, Jan 4, 2016 at 11:19 AM, Anthony Baker <aba...@pivotal.io> wrote: > Look on the feature/GEODE-210 branch. > > The source distribution versions are in the root directory. > The binary distribution versions are in gemfire-assembly/src/main/dist/ > > Each distribution has different requirements since different external > components are bundled. > > Anthony > > > On Jan 4, 2016, at 8:17 AM, Dave Barnes <dbar...@pivotal.io> wrote: > > > > Where can I find these files? > > > > On Mon, Jan 4, 2016 at 7:16 AM, Anthony Baker <aba...@pivotal.io> wrote: > > > >> I’ve made a start on our LICENSE and NOTICE files (for both source and > >> binary distributions) on the feature/GEODE-610 branch. Thanks to Niall > for > >> the excellent analysis to get this started. If you’d like to > contribute to > >> this you can: > >> > >> 1) Review the contents of the LICENSE files and see if they look right. > >> 2) Figure out what needs to go in the NOTICE files. > >> > >> This is needed for the alpha1 release. > >> > >> Thanks, > >> Anthony > >> > >> > >
Re: LICENSE and NOTICE
Look on the feature/GEODE-210 branch. The source distribution versions are in the root directory. The binary distribution versions are in gemfire-assembly/src/main/dist/ Each distribution has different requirements since different external components are bundled. Anthony > On Jan 4, 2016, at 8:17 AM, Dave Barnes <dbar...@pivotal.io> wrote: > > Where can I find these files? > > On Mon, Jan 4, 2016 at 7:16 AM, Anthony Baker <aba...@pivotal.io> wrote: > >> I’ve made a start on our LICENSE and NOTICE files (for both source and >> binary distributions) on the feature/GEODE-610 branch. Thanks to Niall for >> the excellent analysis to get this started. If you’d like to contribute to >> this you can: >> >> 1) Review the contents of the LICENSE files and see if they look right. >> 2) Figure out what needs to go in the NOTICE files. >> >> This is needed for the alpha1 release. >> >> Thanks, >> Anthony >> >> signature.asc Description: Message signed with OpenPGP using GPGMail
