You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@airavata.apache.org by Suresh Marru <sm...@apache.org> on 2012/08/01 15:46:17 UTC
Re: [VOTE] Apache Airavata 0.4-Incubating RC1
Hi All,
The VOTE is called for a lazy consensus and is close to 72 hours. Just in case if there are any further comments, I will leave the vote open for 6 more hours. If you have any concerns or comments with this release please voice your opinions and vote now.
Thanks,
Suresh
On Jul 31, 2012, at 10:09 AM, Alexei Fedotov wrote:
> Suresh,
>
> I am not a lawyer, and cannot yet decide if any of issues is serious
> enough. Let mentors decide.
>
> I'm glad to see that you have cleaned the trunk.
>
> --
> With best regards / с наилучшими пожеланиями,
> Alexei Fedotov / Алексей Федотов,
> http://dataved.ru/
> +7 916 562 8095
>
>
> On Tue, Jul 31, 2012 at 5:59 PM, Suresh Marru <sm...@apache.org> wrote:
>> Hi Alexei,
>>
>> Thank you for taking time to review the release. Please see comments below:
>>
>> On Jul 29, 2012, at 4:15 PM, Alexei Fedotov wrote:
>>
>>> Hello Suresh,
>>> hope the following questions could make the release better.
>>>
>>> 1. Why root NOTICE and LICENSE files are nearly empty, while the files at
>>> modules/distribution/src/main/resources contain all required info on
>>> licenses? Why not to move files to the root?
>>
>> The root NOTICE & LICENSE are for source code and the ones in modules/distribution/src/main/resources are for binary release. Since the source code does not have any third party codes, you will see it have only APL V2 where as the binary ones include all L&D of all the bundled jars.
>>
>>> 2. I have noticed import com.sun.tools.doclets.internal.toolkit.MethodWriter at
>>> modules/ws-messenger/samples/messagebroker/wse-multiple-producers-consumers/src/org/apache/airavata/wsmg/samples/wse/Consumer.java
>>>
>>> MethodWriter license seems to be GPL, see below. If the link below is
>>> correct, we get linking to GPL code.
>>> http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/7-b147/com/sun/tools/doclets/internal/toolkit/MethodWriter.java?av=h
>>>
>>> It seems the class is not used anyway. Why not to remove it?
>>
>> Thanks for this catch, too bad to have this unused import linger through in a stale sample code. Since it was an unused import and it was not linked to any code, is it a blocker for the release?, I removed it in the trunk though (r1367537).
>>>
>>> 3. I wonder if the parts of work
>>> (modules/xbaya-gui/src/main/java/org/apache/airavata/xbaya/ui/graph/system/DifferedInputNodeGUI.java)
>>> containing APL along with Indiana University Extreme! Lab Software
>>> License can be just licensed under Apache License in the release (for
>>> usage simplicity). The initial authors seem to be the same as Apache
>>> committers.
>>
>> Yes your assertion is right, during incubation the IP was donated from Indiana University to Apache and headers were properly replaced. Tracking back on the file you pointed out (and couple of others) were added to the trunk from donation area and added the APL header but a legacy snipped was left out at the bottom of the files, I removed them now. The RAT check passes on all the code since all java have APL headers and probably ignored these stale snippets at the bottom.
>>
>> Appreciate your attention to detail. Do you think we should call a new RC or 2 and 3 are non-blockers for the release?
>>
>> Thanks,
>> Suresh
>>
>>
>>> --
>>> With best regards / с наилучшими пожеланиями,
>>> Alexei Fedotov / Алексей Федотов,
>>> http://dataved.ru/
>>> +7 916 562 8095
>>>
>>>
>>> On Sun, Jul 29, 2012 at 6:37 PM, Suresh Marru <sm...@apache.org> wrote:
>>>> Apache Airavata (Incubating) is pleased to call for a vote on the following
>>>> Apache Airavata 0.4-incubating release candidate artifacts:
>>>>
>>>> We are requesting a lazy consensus vote, as we have already received 3
>>>> binding IPMC +1 votes during the release voting on airavata-dev:
>>>>
>>>> Community VOTE & RESULT Thread: http://markmail.org/thread/4nbaxvi5byjpvhgq
>>>>
>>>> Detailed change log/release notes:
>>>> https://svn.apache.org/repos/asf/incubator/airavata/tags/airavata-0.4-incubating/RELEASE_NOTES
>>>>
>>>> All Release Artifacts:
>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/
>>>>
>>>> PGP release keys (signed using 617DDBAD):
>>>> https://svn.apache.org/repos/asf/incubator/airavata/KEYS
>>>>
>>>> Specific URL's:
>>>>
>>>> SVN source tag (1364995):
>>>> https://svn.apache.org/repos/asf/incubator/airavata/tags/airavata-0.4-incubating/
>>>>
>>>> Source release:
>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/airavata-0.4-incubating-source-release.zip
>>>>
>>>> Binary Artifacts:
>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/apache-airavata-0.4-incubating-bin.tar.gz
>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/apache-airavata-0.4-incubating-bin.zip
>>>>
>>>> Maven staging repo:
>>>> https://repository.apache.org/content/repositories/orgapacheairavata-072/
>>>>
>>>> Please verify the artifacts and vote. The vote will be open for 72 hours.
>>>>
>>>> [ ] +1 approve
>>>> [ ] +0 no opinion
>>>> [ ] -1 disapprove (and reason why)
>>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>>> For additional commands, e-mail: general-help@incubator.apache.org
>>>
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>
Re: [VOTE] Apache Airavata 0.4-Incubating RC1
Posted by Suresh Marru <sm...@apache.org>.
Thank you very much Ate for taking time to review and feedback and votes. I will close the Vote now and post results.
Suresh
On Aug 1, 2012, at 10:35 AM, Ate Douma wrote:
> Hi Suresh,
>
> I looked at the two issues raised and IMO these do not pose as blockers for this release, so +1 from me on this release candidate (binding and Mentor)
>
> More feedback below.
>
> Regards, Ate
>
> On 08/01/2012 03:46 PM, Suresh Marru wrote:
>> Hi All,
>>
>> The VOTE is called for a lazy consensus and is close to 72 hours. Just in case if there are any further comments, I will leave the vote open for 6 more hours. If you have any concerns or comments with this release please voice your opinions and vote now.
>>
>> Thanks,
>> Suresh
>>
>> On Jul 31, 2012, at 10:09 AM, Alexei Fedotov wrote:
>>
>>> Suresh,
>>>
>>> I am not a lawyer, and cannot yet decide if any of issues is serious
>>> enough. Let mentors decide.
>>>
>>> I'm glad to see that you have cleaned the trunk.
>>>
>>> --
>>> With best regards / с наилучшими пожеланиями,
>>> Alexei Fedotov / Алексей Федотов,
>>> http://dataved.ru/
>>> +7 916 562 8095
>>>
>>>
>>> On Tue, Jul 31, 2012 at 5:59 PM, Suresh Marru <sm...@apache.org> wrote:
>>>> Hi Alexei,
>>>>
>>>> Thank you for taking time to review the release. Please see comments below:
>>>>
>>>> On Jul 29, 2012, at 4:15 PM, Alexei Fedotov wrote:
>>>>
>>>>> Hello Suresh,
>>>>> hope the following questions could make the release better.
>>>>>
>>>>> 1. Why root NOTICE and LICENSE files are nearly empty, while the files at
>>>>> modules/distribution/src/main/resources contain all required info on
>>>>> licenses? Why not to move files to the root?
>>>>
>>>> The root NOTICE & LICENSE are for source code and the ones in modules/distribution/src/main/resources are for binary release. Since the source code does not have any third party codes, you will see it have only APL V2 where as the binary ones include all L&D of all the bundled jars.
>>>>
>>>>> 2. I have noticed import com.sun.tools.doclets.internal.toolkit.MethodWriter at
>>>>> modules/ws-messenger/samples/messagebroker/wse-multiple-producers-consumers/src/org/apache/airavata/wsmg/samples/wse/Consumer.java
>>>>>
>>>>> MethodWriter license seems to be GPL, see below. If the link below is
>>>>> correct, we get linking to GPL code.
>>>>> http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/7-b147/com/sun/tools/doclets/internal/toolkit/MethodWriter.java?av=h
>>>>>
>>>>> It seems the class is not used anyway. Why not to remove it?
>>>>
>>>> Thanks for this catch, too bad to have this unused import linger through in a stale sample code. Since it was an unused import and it was not linked to any code, is it a blocker for the release?, I removed it in the trunk though (r1367537).
>
> While an annoyance I wouldn't call this 'linking' to GPL code. It only means you'll need a Sun JDK to compile the project, but most likely you don't even need it at runtime if the compiler is smart enough to drop this unused import.
>
> At any rate this really is only a mistake without intended GPL linking nor any actual usage. And fixed already. So, really not an issue at all IMO.
>
>>>>>
>>>>> 3. I wonder if the parts of work
>>>>> (modules/xbaya-gui/src/main/java/org/apache/airavata/xbaya/ui/graph/system/DifferedInputNodeGUI.java)
>>>>> containing APL along with Indiana University Extreme! Lab Software
>>>>> License can be just licensed under Apache License in the release (for
>>>>> usage simplicity). The initial authors seem to be the same as Apache
>>>>> committers.
>>>>
>>>> Yes your assertion is right, during incubation the IP was donated from Indiana University to Apache and headers were properly replaced. Tracking back on the file you pointed out (and couple of others) were added to the trunk from donation area and added the APL header but a legacy snipped was left out at the bottom of the files, I removed them now. The RAT check passes on all the code since all java have APL headers and probably ignored these stale snippets at the bottom.
>
> Same thing here. The appropriate license header was already in place, the old one at the bottom doesn't 'negate' the one on top. It wasn't a problem to begin with, even less so now after fixing it in trunk.
>
>>>>
>>>> Appreciate your attention to detail. Do you think we should call a new RC or 2 and 3 are non-blockers for the release?
>>>>
>>>> Thanks,
>>>> Suresh
>>>>
>>>>
>>>>> --
>>>>> With best regards / с наилучшими пожеланиями,
>>>>> Alexei Fedotov / Алексей Федотов,
>>>>> http://dataved.ru/
>>>>> +7 916 562 8095
>>>>>
>>>>>
>>>>> On Sun, Jul 29, 2012 at 6:37 PM, Suresh Marru <sm...@apache.org> wrote:
>>>>>> Apache Airavata (Incubating) is pleased to call for a vote on the following
>>>>>> Apache Airavata 0.4-incubating release candidate artifacts:
>>>>>>
>>>>>> We are requesting a lazy consensus vote, as we have already received 3
>>>>>> binding IPMC +1 votes during the release voting on airavata-dev:
>>>>>>
>>>>>> Community VOTE & RESULT Thread: http://markmail.org/thread/4nbaxvi5byjpvhgq
>>>>>>
>>>>>> Detailed change log/release notes:
>>>>>> https://svn.apache.org/repos/asf/incubator/airavata/tags/airavata-0.4-incubating/RELEASE_NOTES
>>>>>>
>>>>>> All Release Artifacts:
>>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/
>>>>>>
>>>>>> PGP release keys (signed using 617DDBAD):
>>>>>> https://svn.apache.org/repos/asf/incubator/airavata/KEYS
>>>>>>
>>>>>> Specific URL's:
>>>>>>
>>>>>> SVN source tag (1364995):
>>>>>> https://svn.apache.org/repos/asf/incubator/airavata/tags/airavata-0.4-incubating/
>>>>>>
>>>>>> Source release:
>>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/airavata-0.4-incubating-source-release.zip
>>>>>>
>>>>>> Binary Artifacts:
>>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/apache-airavata-0.4-incubating-bin.tar.gz
>>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/apache-airavata-0.4-incubating-bin.zip
>>>>>>
>>>>>> Maven staging repo:
>>>>>> https://repository.apache.org/content/repositories/orgapacheairavata-072/
>>>>>>
>>>>>> Please verify the artifacts and vote. The vote will be open for 72 hours.
>>>>>>
>>>>>> [ ] +1 approve
>>>>>> [ ] +0 no opinion
>>>>>> [ ] -1 disapprove (and reason why)
>>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>>>>> For additional commands, e-mail: general-help@incubator.apache.org
>>>>>
>>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>>> For additional commands, e-mail: general-help@incubator.apache.org
>>>
>
>
Re: [VOTE] Apache Airavata 0.4-Incubating RC1
Posted by Suresh Marru <sm...@apache.org>.
Thank you very much Ate for taking time to review and feedback and votes. I will close the Vote now and post results.
Suresh
On Aug 1, 2012, at 10:35 AM, Ate Douma wrote:
> Hi Suresh,
>
> I looked at the two issues raised and IMO these do not pose as blockers for this release, so +1 from me on this release candidate (binding and Mentor)
>
> More feedback below.
>
> Regards, Ate
>
> On 08/01/2012 03:46 PM, Suresh Marru wrote:
>> Hi All,
>>
>> The VOTE is called for a lazy consensus and is close to 72 hours. Just in case if there are any further comments, I will leave the vote open for 6 more hours. If you have any concerns or comments with this release please voice your opinions and vote now.
>>
>> Thanks,
>> Suresh
>>
>> On Jul 31, 2012, at 10:09 AM, Alexei Fedotov wrote:
>>
>>> Suresh,
>>>
>>> I am not a lawyer, and cannot yet decide if any of issues is serious
>>> enough. Let mentors decide.
>>>
>>> I'm glad to see that you have cleaned the trunk.
>>>
>>> --
>>> With best regards / с наилучшими пожеланиями,
>>> Alexei Fedotov / Алексей Федотов,
>>> http://dataved.ru/
>>> +7 916 562 8095
>>>
>>>
>>> On Tue, Jul 31, 2012 at 5:59 PM, Suresh Marru <sm...@apache.org> wrote:
>>>> Hi Alexei,
>>>>
>>>> Thank you for taking time to review the release. Please see comments below:
>>>>
>>>> On Jul 29, 2012, at 4:15 PM, Alexei Fedotov wrote:
>>>>
>>>>> Hello Suresh,
>>>>> hope the following questions could make the release better.
>>>>>
>>>>> 1. Why root NOTICE and LICENSE files are nearly empty, while the files at
>>>>> modules/distribution/src/main/resources contain all required info on
>>>>> licenses? Why not to move files to the root?
>>>>
>>>> The root NOTICE & LICENSE are for source code and the ones in modules/distribution/src/main/resources are for binary release. Since the source code does not have any third party codes, you will see it have only APL V2 where as the binary ones include all L&D of all the bundled jars.
>>>>
>>>>> 2. I have noticed import com.sun.tools.doclets.internal.toolkit.MethodWriter at
>>>>> modules/ws-messenger/samples/messagebroker/wse-multiple-producers-consumers/src/org/apache/airavata/wsmg/samples/wse/Consumer.java
>>>>>
>>>>> MethodWriter license seems to be GPL, see below. If the link below is
>>>>> correct, we get linking to GPL code.
>>>>> http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/7-b147/com/sun/tools/doclets/internal/toolkit/MethodWriter.java?av=h
>>>>>
>>>>> It seems the class is not used anyway. Why not to remove it?
>>>>
>>>> Thanks for this catch, too bad to have this unused import linger through in a stale sample code. Since it was an unused import and it was not linked to any code, is it a blocker for the release?, I removed it in the trunk though (r1367537).
>
> While an annoyance I wouldn't call this 'linking' to GPL code. It only means you'll need a Sun JDK to compile the project, but most likely you don't even need it at runtime if the compiler is smart enough to drop this unused import.
>
> At any rate this really is only a mistake without intended GPL linking nor any actual usage. And fixed already. So, really not an issue at all IMO.
>
>>>>>
>>>>> 3. I wonder if the parts of work
>>>>> (modules/xbaya-gui/src/main/java/org/apache/airavata/xbaya/ui/graph/system/DifferedInputNodeGUI.java)
>>>>> containing APL along with Indiana University Extreme! Lab Software
>>>>> License can be just licensed under Apache License in the release (for
>>>>> usage simplicity). The initial authors seem to be the same as Apache
>>>>> committers.
>>>>
>>>> Yes your assertion is right, during incubation the IP was donated from Indiana University to Apache and headers were properly replaced. Tracking back on the file you pointed out (and couple of others) were added to the trunk from donation area and added the APL header but a legacy snipped was left out at the bottom of the files, I removed them now. The RAT check passes on all the code since all java have APL headers and probably ignored these stale snippets at the bottom.
>
> Same thing here. The appropriate license header was already in place, the old one at the bottom doesn't 'negate' the one on top. It wasn't a problem to begin with, even less so now after fixing it in trunk.
>
>>>>
>>>> Appreciate your attention to detail. Do you think we should call a new RC or 2 and 3 are non-blockers for the release?
>>>>
>>>> Thanks,
>>>> Suresh
>>>>
>>>>
>>>>> --
>>>>> With best regards / с наилучшими пожеланиями,
>>>>> Alexei Fedotov / Алексей Федотов,
>>>>> http://dataved.ru/
>>>>> +7 916 562 8095
>>>>>
>>>>>
>>>>> On Sun, Jul 29, 2012 at 6:37 PM, Suresh Marru <sm...@apache.org> wrote:
>>>>>> Apache Airavata (Incubating) is pleased to call for a vote on the following
>>>>>> Apache Airavata 0.4-incubating release candidate artifacts:
>>>>>>
>>>>>> We are requesting a lazy consensus vote, as we have already received 3
>>>>>> binding IPMC +1 votes during the release voting on airavata-dev:
>>>>>>
>>>>>> Community VOTE & RESULT Thread: http://markmail.org/thread/4nbaxvi5byjpvhgq
>>>>>>
>>>>>> Detailed change log/release notes:
>>>>>> https://svn.apache.org/repos/asf/incubator/airavata/tags/airavata-0.4-incubating/RELEASE_NOTES
>>>>>>
>>>>>> All Release Artifacts:
>>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/
>>>>>>
>>>>>> PGP release keys (signed using 617DDBAD):
>>>>>> https://svn.apache.org/repos/asf/incubator/airavata/KEYS
>>>>>>
>>>>>> Specific URL's:
>>>>>>
>>>>>> SVN source tag (1364995):
>>>>>> https://svn.apache.org/repos/asf/incubator/airavata/tags/airavata-0.4-incubating/
>>>>>>
>>>>>> Source release:
>>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/airavata-0.4-incubating-source-release.zip
>>>>>>
>>>>>> Binary Artifacts:
>>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/apache-airavata-0.4-incubating-bin.tar.gz
>>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/apache-airavata-0.4-incubating-bin.zip
>>>>>>
>>>>>> Maven staging repo:
>>>>>> https://repository.apache.org/content/repositories/orgapacheairavata-072/
>>>>>>
>>>>>> Please verify the artifacts and vote. The vote will be open for 72 hours.
>>>>>>
>>>>>> [ ] +1 approve
>>>>>> [ ] +0 no opinion
>>>>>> [ ] -1 disapprove (and reason why)
>>>>>>
>>>>>
>>>>> ---------------------------------------------------------------------
>>>>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>>>>> For additional commands, e-mail: general-help@incubator.apache.org
>>>>>
>>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>>> For additional commands, e-mail: general-help@incubator.apache.org
>>>
>
>
Re: [VOTE] Apache Airavata 0.4-Incubating RC1
Posted by Ate Douma <at...@douma.nu>.
Hi Suresh,
I looked at the two issues raised and IMO these do not pose as blockers for this
release, so +1 from me on this release candidate (binding and Mentor)
More feedback below.
Regards, Ate
On 08/01/2012 03:46 PM, Suresh Marru wrote:
> Hi All,
>
> The VOTE is called for a lazy consensus and is close to 72 hours. Just in case if there are any further comments, I will leave the vote open for 6 more hours. If you have any concerns or comments with this release please voice your opinions and vote now.
>
> Thanks,
> Suresh
>
> On Jul 31, 2012, at 10:09 AM, Alexei Fedotov wrote:
>
>> Suresh,
>>
>> I am not a lawyer, and cannot yet decide if any of issues is serious
>> enough. Let mentors decide.
>>
>> I'm glad to see that you have cleaned the trunk.
>>
>> --
>> With best regards / с наилучшими пожеланиями,
>> Alexei Fedotov / Алексей Федотов,
>> http://dataved.ru/
>> +7 916 562 8095
>>
>>
>> On Tue, Jul 31, 2012 at 5:59 PM, Suresh Marru <sm...@apache.org> wrote:
>>> Hi Alexei,
>>>
>>> Thank you for taking time to review the release. Please see comments below:
>>>
>>> On Jul 29, 2012, at 4:15 PM, Alexei Fedotov wrote:
>>>
>>>> Hello Suresh,
>>>> hope the following questions could make the release better.
>>>>
>>>> 1. Why root NOTICE and LICENSE files are nearly empty, while the files at
>>>> modules/distribution/src/main/resources contain all required info on
>>>> licenses? Why not to move files to the root?
>>>
>>> The root NOTICE & LICENSE are for source code and the ones in modules/distribution/src/main/resources are for binary release. Since the source code does not have any third party codes, you will see it have only APL V2 where as the binary ones include all L&D of all the bundled jars.
>>>
>>>> 2. I have noticed import com.sun.tools.doclets.internal.toolkit.MethodWriter at
>>>> modules/ws-messenger/samples/messagebroker/wse-multiple-producers-consumers/src/org/apache/airavata/wsmg/samples/wse/Consumer.java
>>>>
>>>> MethodWriter license seems to be GPL, see below. If the link below is
>>>> correct, we get linking to GPL code.
>>>> http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/7-b147/com/sun/tools/doclets/internal/toolkit/MethodWriter.java?av=h
>>>>
>>>> It seems the class is not used anyway. Why not to remove it?
>>>
>>> Thanks for this catch, too bad to have this unused import linger through in a stale sample code. Since it was an unused import and it was not linked to any code, is it a blocker for the release?, I removed it in the trunk though (r1367537).
While an annoyance I wouldn't call this 'linking' to GPL code. It only means
you'll need a Sun JDK to compile the project, but most likely you don't even
need it at runtime if the compiler is smart enough to drop this unused import.
At any rate this really is only a mistake without intended GPL linking nor any
actual usage. And fixed already. So, really not an issue at all IMO.
>>>>
>>>> 3. I wonder if the parts of work
>>>> (modules/xbaya-gui/src/main/java/org/apache/airavata/xbaya/ui/graph/system/DifferedInputNodeGUI.java)
>>>> containing APL along with Indiana University Extreme! Lab Software
>>>> License can be just licensed under Apache License in the release (for
>>>> usage simplicity). The initial authors seem to be the same as Apache
>>>> committers.
>>>
>>> Yes your assertion is right, during incubation the IP was donated from Indiana University to Apache and headers were properly replaced. Tracking back on the file you pointed out (and couple of others) were added to the trunk from donation area and added the APL header but a legacy snipped was left out at the bottom of the files, I removed them now. The RAT check passes on all the code since all java have APL headers and probably ignored these stale snippets at the bottom.
Same thing here. The appropriate license header was already in place, the old
one at the bottom doesn't 'negate' the one on top. It wasn't a problem to begin
with, even less so now after fixing it in trunk.
>>>
>>> Appreciate your attention to detail. Do you think we should call a new RC or 2 and 3 are non-blockers for the release?
>>>
>>> Thanks,
>>> Suresh
>>>
>>>
>>>> --
>>>> With best regards / с наилучшими пожеланиями,
>>>> Alexei Fedotov / Алексей Федотов,
>>>> http://dataved.ru/
>>>> +7 916 562 8095
>>>>
>>>>
>>>> On Sun, Jul 29, 2012 at 6:37 PM, Suresh Marru <sm...@apache.org> wrote:
>>>>> Apache Airavata (Incubating) is pleased to call for a vote on the following
>>>>> Apache Airavata 0.4-incubating release candidate artifacts:
>>>>>
>>>>> We are requesting a lazy consensus vote, as we have already received 3
>>>>> binding IPMC +1 votes during the release voting on airavata-dev:
>>>>>
>>>>> Community VOTE & RESULT Thread: http://markmail.org/thread/4nbaxvi5byjpvhgq
>>>>>
>>>>> Detailed change log/release notes:
>>>>> https://svn.apache.org/repos/asf/incubator/airavata/tags/airavata-0.4-incubating/RELEASE_NOTES
>>>>>
>>>>> All Release Artifacts:
>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/
>>>>>
>>>>> PGP release keys (signed using 617DDBAD):
>>>>> https://svn.apache.org/repos/asf/incubator/airavata/KEYS
>>>>>
>>>>> Specific URL's:
>>>>>
>>>>> SVN source tag (1364995):
>>>>> https://svn.apache.org/repos/asf/incubator/airavata/tags/airavata-0.4-incubating/
>>>>>
>>>>> Source release:
>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/airavata-0.4-incubating-source-release.zip
>>>>>
>>>>> Binary Artifacts:
>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/apache-airavata-0.4-incubating-bin.tar.gz
>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/apache-airavata-0.4-incubating-bin.zip
>>>>>
>>>>> Maven staging repo:
>>>>> https://repository.apache.org/content/repositories/orgapacheairavata-072/
>>>>>
>>>>> Please verify the artifacts and vote. The vote will be open for 72 hours.
>>>>>
>>>>> [ ] +1 approve
>>>>> [ ] +0 no opinion
>>>>> [ ] -1 disapprove (and reason why)
>>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>>>> For additional commands, e-mail: general-help@incubator.apache.org
>>>>
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>> For additional commands, e-mail: general-help@incubator.apache.org
>>
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org
Re: [VOTE] Apache Airavata 0.4-Incubating RC1
Posted by Ate Douma <at...@douma.nu>.
Hi Suresh,
I looked at the two issues raised and IMO these do not pose as blockers for this
release, so +1 from me on this release candidate (binding and Mentor)
More feedback below.
Regards, Ate
On 08/01/2012 03:46 PM, Suresh Marru wrote:
> Hi All,
>
> The VOTE is called for a lazy consensus and is close to 72 hours. Just in case if there are any further comments, I will leave the vote open for 6 more hours. If you have any concerns or comments with this release please voice your opinions and vote now.
>
> Thanks,
> Suresh
>
> On Jul 31, 2012, at 10:09 AM, Alexei Fedotov wrote:
>
>> Suresh,
>>
>> I am not a lawyer, and cannot yet decide if any of issues is serious
>> enough. Let mentors decide.
>>
>> I'm glad to see that you have cleaned the trunk.
>>
>> --
>> With best regards / с наилучшими пожеланиями,
>> Alexei Fedotov / Алексей Федотов,
>> http://dataved.ru/
>> +7 916 562 8095
>>
>>
>> On Tue, Jul 31, 2012 at 5:59 PM, Suresh Marru <sm...@apache.org> wrote:
>>> Hi Alexei,
>>>
>>> Thank you for taking time to review the release. Please see comments below:
>>>
>>> On Jul 29, 2012, at 4:15 PM, Alexei Fedotov wrote:
>>>
>>>> Hello Suresh,
>>>> hope the following questions could make the release better.
>>>>
>>>> 1. Why root NOTICE and LICENSE files are nearly empty, while the files at
>>>> modules/distribution/src/main/resources contain all required info on
>>>> licenses? Why not to move files to the root?
>>>
>>> The root NOTICE & LICENSE are for source code and the ones in modules/distribution/src/main/resources are for binary release. Since the source code does not have any third party codes, you will see it have only APL V2 where as the binary ones include all L&D of all the bundled jars.
>>>
>>>> 2. I have noticed import com.sun.tools.doclets.internal.toolkit.MethodWriter at
>>>> modules/ws-messenger/samples/messagebroker/wse-multiple-producers-consumers/src/org/apache/airavata/wsmg/samples/wse/Consumer.java
>>>>
>>>> MethodWriter license seems to be GPL, see below. If the link below is
>>>> correct, we get linking to GPL code.
>>>> http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/7-b147/com/sun/tools/doclets/internal/toolkit/MethodWriter.java?av=h
>>>>
>>>> It seems the class is not used anyway. Why not to remove it?
>>>
>>> Thanks for this catch, too bad to have this unused import linger through in a stale sample code. Since it was an unused import and it was not linked to any code, is it a blocker for the release?, I removed it in the trunk though (r1367537).
While an annoyance I wouldn't call this 'linking' to GPL code. It only means
you'll need a Sun JDK to compile the project, but most likely you don't even
need it at runtime if the compiler is smart enough to drop this unused import.
At any rate this really is only a mistake without intended GPL linking nor any
actual usage. And fixed already. So, really not an issue at all IMO.
>>>>
>>>> 3. I wonder if the parts of work
>>>> (modules/xbaya-gui/src/main/java/org/apache/airavata/xbaya/ui/graph/system/DifferedInputNodeGUI.java)
>>>> containing APL along with Indiana University Extreme! Lab Software
>>>> License can be just licensed under Apache License in the release (for
>>>> usage simplicity). The initial authors seem to be the same as Apache
>>>> committers.
>>>
>>> Yes your assertion is right, during incubation the IP was donated from Indiana University to Apache and headers were properly replaced. Tracking back on the file you pointed out (and couple of others) were added to the trunk from donation area and added the APL header but a legacy snipped was left out at the bottom of the files, I removed them now. The RAT check passes on all the code since all java have APL headers and probably ignored these stale snippets at the bottom.
Same thing here. The appropriate license header was already in place, the old
one at the bottom doesn't 'negate' the one on top. It wasn't a problem to begin
with, even less so now after fixing it in trunk.
>>>
>>> Appreciate your attention to detail. Do you think we should call a new RC or 2 and 3 are non-blockers for the release?
>>>
>>> Thanks,
>>> Suresh
>>>
>>>
>>>> --
>>>> With best regards / с наилучшими пожеланиями,
>>>> Alexei Fedotov / Алексей Федотов,
>>>> http://dataved.ru/
>>>> +7 916 562 8095
>>>>
>>>>
>>>> On Sun, Jul 29, 2012 at 6:37 PM, Suresh Marru <sm...@apache.org> wrote:
>>>>> Apache Airavata (Incubating) is pleased to call for a vote on the following
>>>>> Apache Airavata 0.4-incubating release candidate artifacts:
>>>>>
>>>>> We are requesting a lazy consensus vote, as we have already received 3
>>>>> binding IPMC +1 votes during the release voting on airavata-dev:
>>>>>
>>>>> Community VOTE & RESULT Thread: http://markmail.org/thread/4nbaxvi5byjpvhgq
>>>>>
>>>>> Detailed change log/release notes:
>>>>> https://svn.apache.org/repos/asf/incubator/airavata/tags/airavata-0.4-incubating/RELEASE_NOTES
>>>>>
>>>>> All Release Artifacts:
>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/
>>>>>
>>>>> PGP release keys (signed using 617DDBAD):
>>>>> https://svn.apache.org/repos/asf/incubator/airavata/KEYS
>>>>>
>>>>> Specific URL's:
>>>>>
>>>>> SVN source tag (1364995):
>>>>> https://svn.apache.org/repos/asf/incubator/airavata/tags/airavata-0.4-incubating/
>>>>>
>>>>> Source release:
>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/airavata-0.4-incubating-source-release.zip
>>>>>
>>>>> Binary Artifacts:
>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/apache-airavata-0.4-incubating-bin.tar.gz
>>>>> http://people.apache.org/builds/incubator/airavata/0.4-incubating/RC1/apache-airavata-0.4-incubating-bin.zip
>>>>>
>>>>> Maven staging repo:
>>>>> https://repository.apache.org/content/repositories/orgapacheairavata-072/
>>>>>
>>>>> Please verify the artifacts and vote. The vote will be open for 72 hours.
>>>>>
>>>>> [ ] +1 approve
>>>>> [ ] +0 no opinion
>>>>> [ ] -1 disapprove (and reason why)
>>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>>>> For additional commands, e-mail: general-help@incubator.apache.org
>>>>
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>> For additional commands, e-mail: general-help@incubator.apache.org
>>