Re: Introducing myself & stepping forward

[Antonio Vieiro]

Thanks, Emilian! Thanks all!

Un abrazo,
Antonio

2017-10-02 7:52 GMT+02:00 Emilian Bold :
> Welcome Antonio! You have been missed!
>
> --emi
>
>
> On Mon, Oct 2, 2017 at 7:53 AM, Antonio Vieiro  wrote:
>> Great to see you all as well! :-)
>>
>>
>>> El 2 oct 2017, a las 1:17, Wade Chandler  escribió:
>>>
>>> Great to see you here Antonio!
>>>
>>> Wade
>>>
>>>
>>> On Oct 1, 2017 2:04 PM, "Antonio Vieiro"  wrote:
>>>
 Hi all,

 This is Antonio Vieiro, an old time NetBeans enthusiast based in Madrid,
 Spain.

 I've been keeping an eye on the NetBeans incubator process for a
 while. It seems there's a looong list of modules to review at

 https://cwiki.apache.org/confluence/display/NETBEANS/
 List+of+Modules+to+Review

 I was wondering if you need a hand on the review process and, if so,
 I'd appreciate some further instructions on how I should proceed.

 Kind regards,
 Antonio

 P.S.: Yep, I was a NetBeans Dream Team Member a few years ago.

>>

Re: Introducing myself & stepping forward

[Emilian Bold]

Welcome Antonio! You have been missed!

--emi


On Mon, Oct 2, 2017 at 7:53 AM, Antonio Vieiro  wrote:
> Great to see you all as well! :-)
>
>
>> El 2 oct 2017, a las 1:17, Wade Chandler  escribió:
>>
>> Great to see you here Antonio!
>>
>> Wade
>>
>>
>> On Oct 1, 2017 2:04 PM, "Antonio Vieiro"  wrote:
>>
>>> Hi all,
>>>
>>> This is Antonio Vieiro, an old time NetBeans enthusiast based in Madrid,
>>> Spain.
>>>
>>> I've been keeping an eye on the NetBeans incubator process for a
>>> while. It seems there's a looong list of modules to review at
>>>
>>> https://cwiki.apache.org/confluence/display/NETBEANS/
>>> List+of+Modules+to+Review
>>>
>>> I was wondering if you need a hand on the review process and, if so,
>>> I'd appreciate some further instructions on how I should proceed.
>>>
>>> Kind regards,
>>> Antonio
>>>
>>> P.S.: Yep, I was a NetBeans Dream Team Member a few years ago.
>>>
>

Re: Hello

[Craig Russell]

Hi,

You might find this of value: https://netbeans.org/community/lists/index.html

Regards,

Craig

> On Sep 30, 2017, at 5:45 PM, William Beebe  wrote:
> 
> I'm interested in following this mailing list.

Craig L Russell
c...@apache.org

Re: Introducing myself & stepping forward

[Wade Chandler]

Great to see you here Antonio!

Wade


On Oct 1, 2017 2:04 PM, "Antonio Vieiro"  wrote:

> Hi all,
>
> This is Antonio Vieiro, an old time NetBeans enthusiast based in Madrid,
> Spain.
>
> I've been keeping an eye on the NetBeans incubator process for a
> while. It seems there's a looong list of modules to review at
>
> https://cwiki.apache.org/confluence/display/NETBEANS/
> List+of+Modules+to+Review
>
> I was wondering if you need a hand on the review process and, if so,
> I'd appreciate some further instructions on how I should proceed.
>
> Kind regards,
> Antonio
>
> P.S.: Yep, I was a NetBeans Dream Team Member a few years ago.
>

Re: Commit author rules?

[Junichi Yamamoto]

Hello,

I agree with that.

Thanks,
Junichi

2017-10-01 14:21 GMT+09:00 Emilian Bold :
> Hello,
>
> I believe it would be best to use our full name and @apache alias when
> committing to the main repository.
>
> Right now I see a mix of company, personal and .local email addresses
> as well as complete name or just (local machine) usernames.
>
> Actually I don't care about the email address that much, but it must
> be something reachable.
>
> The full name though seems important.
>
> How do you see things?
>
> --emi

[GitHub] incubator-netbeans pull request #36: [NETBEANS-54] Module Review api.xml

[junichi11]

GitHub user junichi11 opened a pull request:

https://github.com/apache/incubator-netbeans/pull/36

[NETBEANS-54] Module Review api.xml

- no external library
- checked Rat report: add the license header in two *.dtd. ignore three 
package-list[1] files, org-netbeans-api-xml.sig
- A unit test fails even if the license header is not added. 
(ClassNotFoundException: org.apache.crimson.parser.XMLReaderImpl)

[1] 
http://docs.oracle.com/javase/8/docs/technotes/tools/windows/javadoc.html#JSWOR663


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/junichi11/incubator-netbeans 
module-review-api.xml

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-netbeans/pull/36.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #36


commit d94a8cbde4c1cde564e4635751d1799c76a70718
Author: Junichi Yamamoto 
Date:   2017-10-01T22:22:01Z

[NETBEANS-54] Module Review api.xml

- no external library
- checked Rat report: add the license header in two *.dtd. ignore three 
package-list files, org-netbeans-api-xml.sig
- A unit test fails even if the license header is not added.




---

[GitHub] incubator-netbeans issue #7: [NETBEANS-54] Module Review api.annotations.com...

[dschoorl]

Github user dschoorl commented on the issue:

https://github.com/apache/incubator-netbeans/pull/7
  
Hi Jan,

Thanks for reviewing again.

In no way, I am a legal expert. But I think that by providing the 
jcip-license.txt file, we meet the requirements of the license stated in 
article 4b (https://creativecommons.org/licenses/by/2.5/legalcode) and I think 
no notice is needed? Why do you think needs the jcip license text be merged in 
the jsr305 one? I believe this is also not done by the findbugs-community who 
repackaged it with jsr305.jar.

Are these -license.txt files somehow processed automatically and do they 
need a certain structure, such as having a License-tag? Is that described 
somewhere? 



---

[GitHub] incubator-netbeans pull request #35: -external library Apache felix. Changed...

[jlahoda]

GitHub user jlahoda opened a pull request:

https://github.com/apache/incubator-netbeans/pull/35

-external library Apache felix. Changed coordinates to Maven. File na…

…me changed, adjusted. License Apache License v2.0. -notice.txt added 
based on NOTICE from Felix.

-checked Rat report; no license header in binaries-list (added)

-fixed version in -license.txt

-skimmed through the module, did not notice additional problems

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jlahoda/incubator-netbeans libs.felix-review

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-netbeans/pull/35.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #35


commit 4c5e4f31c8f8e4a296d4031236fc14ef5bd86aa4
Author: Jan Lahoda 
Date:   2017-09-24T05:52:49Z

-external library Apache felix. Changed coordinates to Maven. File name 
changed, adjusted. License Apache License v2.0. -notice.txt added based on 
NOTICE from Felix.

-checked Rat report; no license header in binaries-list (added)

-fixed version in -license.txt

-skimmed through the module, did not notice additional problems




---

[GitHub] incubator-netbeans pull request #34: -external library jna-4.2.2.jar: dual-l...

[jlahoda]

GitHub user jlahoda opened a pull request:

https://github.com/apache/incubator-netbeans/pull/34

-external library jna-4.2.2.jar: dual-licensed by Apache License v2.0…

… and GPLv2, picking Apache License. NOTICE entry does not seem to be 
needed. Maven coordinates added.

-checked Rat report; no license header in binaries-list (added)

-fixed version in -license.txt

-skimmed through the module, did not notice additional problems

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jlahoda/incubator-netbeans libs.jna-review

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-netbeans/pull/34.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #34


commit 2d0d215b5b8b81093ac97dc9a4ba382522679eeb
Author: Jan Lahoda 
Date:   2017-09-24T05:37:57Z

-external library jna-4.2.2.jar: dual-licensed by Apache License v2.0 and 
GPLv2, picking Apache License. NOTICE entry does not seem to be needed. Maven 
coordinates added.

-checked Rat report; no license header in binaries-list (added)

-fixed version in -license.txt

-skimmed through the module, did not notice additional problems




---

[GitHub] incubator-netbeans pull request #33: -external library jna-platform-4.2.2.ja...

[jlahoda]

GitHub user jlahoda opened a pull request:

https://github.com/apache/incubator-netbeans/pull/33

-external library jna-platform-4.2.2.jar: dual-licensed by Apache Lic…

…ense v2.0 and GPLv2, picking Apache License. NOTICE entry does not seem 
to be needed. Maven coordinates added.

-checked Rat report; no license header in binaries-list (added)

-fixed version in -license.txt

-skimmed through the module, did not notice additional problems

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jlahoda/incubator-netbeans 
libs.jna.platform-review

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-netbeans/pull/33.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #33


commit 59366e04675759c96458438e04669de896bbcd1a
Author: Jan Lahoda 
Date:   2017-10-01T20:18:47Z

-external library jna-platform-4.2.2.jar: dual-licensed by Apache License 
v2.0 and GPLv2, picking Apache License. NOTICE entry does not seem to be 
needed. Maven coordinates added.

-checked Rat report; no license header in binaries-list (added)

-fixed version in -license.txt

-skimmed through the module, did not notice additional problems




---

[GitHub] incubator-netbeans issue #32: [NETBEANS-54] Module Review server

[borxa]

Github user borxa commented on the issue:

https://github.com/apache/incubator-netbeans/pull/32
  
Settings updated


---

[GitHub] incubator-netbeans issue #7: [NETBEANS-54] Module Review api.annotations.com...

[jlahoda]

Github user jlahoda commented on the issue:

https://github.com/apache/incubator-netbeans/pull/7
  
Hi Dave,

Sorry for belated answer.

I think the jcip license text should be merged into the existing 
-license.txt file. Also the "License: " tag should probably be changed to 
something else than "New BSD" (e.g. "BSD+jcip"). Even if though this is 
probably not ever going to be used.

For notice - I suspect we will need different notices, one at the top level 
of the repository for things that are in the repository (which this is not), 
and a different one for binary builds (where the notice should contain exactly 
what is in the binary, not more, not less, in other words, this will probably 
differ between the platform and the Java IDE). And there needs to be enough 
data to construct such notices automatically, I don't think it is realistic to 
maintain them manually. Hence the request to add -notice.txt to external 
binaries if needed, so the build will be able to pick them up if needed. But it 
is not quite clear to me if anything like that is needed for this library.



---

[GitHub] incubator-netbeans issue #31: [NETBEANS-54] Module Review j2ee.core.utilitie...

[juneau001]

Github user juneau001 commented on the issue:

https://github.com/apache/incubator-netbeans/pull/31
  
Thanks for the feedback and sorry for missing those "Problems to be solved 
centrally".  


---

Re: Introducing myself & stepping forward

[Geertjan Wielenga]

Not sure but would be good in any case:
https://www.apache.org/licenses/icla.pdf

Gj

On Sun, Oct 1, 2017 at 11:29 AM, Antonio Vieiro  wrote:

> Yep.
>
> I was just wondering, should I sign the iCLA before submitting patches?
>
> Thanks,
> Antonio
>
> 2017-10-01 20:15 GMT+02:00 Geertjan Wielenga <
> geertjan.wiele...@googlemail.com>:
> > The instructions are at the top of that page and there's a YouTube clip
> > here: https://www.youtube.com/watch?v=Z4PBNSRp5g8
> >
> > Does that help?
> >
> > Gj
> >
> > On Sun, Oct 1, 2017 at 11:04 AM, Antonio Vieiro 
> wrote:
> >
> >> Hi all,
> >>
> >> This is Antonio Vieiro, an old time NetBeans enthusiast based in Madrid,
> >> Spain.
> >>
> >> I've been keeping an eye on the NetBeans incubator process for a
> >> while. It seems there's a looong list of modules to review at
> >>
> >> https://cwiki.apache.org/confluence/display/NETBEANS/
> >> List+of+Modules+to+Review
> >>
> >> I was wondering if you need a hand on the review process and, if so,
> >> I'd appreciate some further instructions on how I should proceed.
> >>
> >> Kind regards,
> >> Antonio
> >>
> >> P.S.: Yep, I was a NetBeans Dream Team Member a few years ago.
> >>
>

Re: Introducing myself & stepping forward

[Antonio Vieiro]

Yep.

I was just wondering, should I sign the iCLA before submitting patches?

Thanks,
Antonio

2017-10-01 20:15 GMT+02:00 Geertjan Wielenga :
> The instructions are at the top of that page and there's a YouTube clip
> here: https://www.youtube.com/watch?v=Z4PBNSRp5g8
>
> Does that help?
>
> Gj
>
> On Sun, Oct 1, 2017 at 11:04 AM, Antonio Vieiro  wrote:
>
>> Hi all,
>>
>> This is Antonio Vieiro, an old time NetBeans enthusiast based in Madrid,
>> Spain.
>>
>> I've been keeping an eye on the NetBeans incubator process for a
>> while. It seems there's a looong list of modules to review at
>>
>> https://cwiki.apache.org/confluence/display/NETBEANS/
>> List+of+Modules+to+Review
>>
>> I was wondering if you need a hand on the review process and, if so,
>> I'd appreciate some further instructions on how I should proceed.
>>
>> Kind regards,
>> Antonio
>>
>> P.S.: Yep, I was a NetBeans Dream Team Member a few years ago.
>>

Re: Introduction

[Geertjan Wielenga]

Excellent, great to hear.

Gj

On Sun, Oct 1, 2017 at 11:14 AM, William Beebe  wrote:

> Hello.
>
> My name is Bill Beebe living in Orlando, Florida.
>
> I'm a long time NetBeans user and occasional pre-release tester. I'm a
> newcomer to this particular development effort. I offer to do the tasks
> (documentation, reviews, bug fixing, and similar) that need to be done in
> order to learn more about this overall development effort, as well as
> NetBeans itself.
>
> Very respectfully,
>
> Bill
>

Introducing myself & stepping forward

[Antonio Vieiro]

Hi all,

This is Antonio Vieiro, an old time NetBeans enthusiast based in Madrid, Spain.

I've been keeping an eye on the NetBeans incubator process for a
while. It seems there's a looong list of modules to review at

https://cwiki.apache.org/confluence/display/NETBEANS/List+of+Modules+to+Review

I was wondering if you need a hand on the review process and, if so,
I'd appreciate some further instructions on how I should proceed.

Kind regards,
Antonio

P.S.: Yep, I was a NetBeans Dream Team Member a few years ago.

[GitHub] incubator-netbeans pull request #32: [NETBEANS-54] Module Review server

[borxa]

GitHub user borxa opened a pull request:

https://github.com/apache/incubator-netbeans/pull/32

[NETBEANS-54] Module Review server

- no external library
- checked Rat report; changed license headers in 2 files
- skimmed through the module, did not notice additional problems

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/borxa/incubator-netbeans master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-netbeans/pull/32.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #32


commit 664936aa2c6d60406818ef016052d36681efed00
Author: borxa 
Date:   2017-10-01T16:54:40Z

[NETBEANS-54] Module Review server




---

[GitHub] incubator-netbeans pull request #31: [NETBEANS-54] Module Review j2ee.core.u...

[juneau001]

GitHub user juneau001 opened a pull request:

https://github.com/apache/incubator-netbeans/pull/31

[NETBEANS-54] Module Review j2ee.core.utilities

Updated j2ee.core.utilities license headers, where needed.
- no external libraries
- checked Rat report and adjusted license headers accordingly on identified 
problem files
- skimmed module and did not see any other issues

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/juneau001/incubator-netbeans master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-netbeans/pull/31.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #31


commit f70a4a8de60bebfb6c563de40faa096745cf5fab
Author: Josh Juneau 
Date:   2017-10-01T13:45:12Z

[NETBEANS-54] Module Review j2ee.core.utilities




---

Podling Report Reminder - October 2017

[johndament]

Dear podling,

This email was sent by an automated system on behalf of the Apache
Incubator PMC. It is an initial reminder to give you plenty of time to
prepare your quarterly board report.

The board meeting is scheduled for Wed, 18 October 2017, 10:30 am PDT.
The report for your podling will form a part of the Incubator PMC
report. The Incubator PMC requires your report to be submitted 2 weeks
before the board meeting, to allow sufficient time for review and
submission (Wed, October 04).

Please submit your report with sufficient time to allow the Incubator
PMC, and subsequently board members to review and digest. Again, the
very latest you should submit your report is 2 weeks prior to the board
meeting.

Thanks,

The Apache Incubator PMC

Submitting your Report

--

Your report should contain the following:

*   Your project name
*   A brief description of your project, which assumes no knowledge of
the project or necessarily of its field
*   A list of the three most important issues to address in the move
towards graduation.
*   Any issues that the Incubator PMC or ASF Board might wish/need to be
aware of
*   How has the community developed since the last report
*   How has the project developed since the last report.
*   How does the podling rate their own maturity.

This should be appended to the Incubator Wiki page at:

https://wiki.apache.org/incubator/October2017

Note: This is manually populated. You may need to wait a little before
this page is created from a template.

Mentors
---

Mentors should review reports for their project(s) and sign them off on
the Incubator wiki page. Signing off reports shows that you are
following the project - projects that are not signed may raise alarms
for the Incubator PMC.

Incubator PMC

[GitHub] incubator-netbeans pull request #8: [NETBEANS-54] Module Review api.htmlui

[jlahoda]

Github user jlahoda closed the pull request at:

https://github.com/apache/incubator-netbeans/pull/8


---

[GitHub] incubator-netbeans issue #8: [NETBEANS-54] Module Review api.htmlui

[jlahoda]

Github user jlahoda commented on the issue:

https://github.com/apache/incubator-netbeans/pull/8
  
Pushed:

https://git-wip-us.apache.org/repos/asf?p=incubator-netbeans.git;a=commit;h=3880c5162f149eb0d3953aacac63c4c014cd101d



---

Re: README and Blog

[Geertjan Wielenga]

Done.

Gj

On Sat, Sep 30, 2017 at 7:39 PM, Tushar Joshi 
wrote:

> Done. My Roller username is "tusharjoshi"
>
> with regards
> Tushar
>

[GitHub] incubator-netbeans issue #21: Fixing form files license headers

[jlahoda]

Github user jlahoda commented on the issue:

https://github.com/apache/incubator-netbeans/pull/21
  
Ok, done:

https://git-wip-us.apache.org/repos/asf?p=incubator-netbeans.git;a=commit;h=cc6231c5f8fc1d99619b940387f1dcb51374631a

https://git-wip-us.apache.org/repos/asf?p=incubator-netbeans.git;a=commit;h=6f163f6ddcd8cd7e4c546ce5ea96115879da6c97

https://git-wip-us.apache.org/repos/asf?p=incubator-netbeans.git;a=commit;h=2861b9c8169c86e0e8a4d0b7c918b6c59849c398

https://git-wip-us.apache.org/repos/asf?p=incubator-netbeans.git;a=commit;h=bf9caf7aa689aee3ce7c759f378e2a70ce67d690

Thanks for the comments!


---

[GitHub] incubator-netbeans pull request #21: Fixing form files license headers

[jlahoda]

Github user jlahoda closed the pull request at:

https://github.com/apache/incubator-netbeans/pull/21


---

Re: Commit author rules?

[Daniel Gruno]

On 10/01/2017 09:54 AM, Matthias Bläsing wrote:
> Hi,
> 
> Am Sonntag, den 01.10.2017, 08:21 +0300 schrieb Emilian Bold:
>> I believe it would be best to use our full name and @apache alias
>> when committing to the main repository.
>>
>> Right now I see a mix of company, personal and .local email addresses
>> as well as complete name or just (local machine) usernames.
>>
>> Actually I don't care about the email address that much, but it must
>> be something reachable.
>>
>> The full name though seems important.
> 
> I agree with Emilians assessment, the author information is important
> and needs to be correct. It will also make it easier to see if work can
> be merged directly. If I'm not mistaken code can be merged under two
> conditions:
> 
>1. Either the change is deemed small/simple enough not to require an
>   ICLA
>2. The author needs to have an ICLA on file
> 
> For the first condition I'll rely on my gut feeling, for the second an
> apache email adresse as the author is a clear indication.
> 
> So I'd suggest to use the atpache address as the primary author
> address. 
> 
> Greetings
> 
> Matthias
> 

An email address is typically unique to whomever does the commit,
whereas multiple people can share the same name. I don't much care
whether you use your personal, corporate or apache.org email address,
but keeping it the same throughout really helps for those of us trying
to piece together who does what and when.

Re: Commit author rules?

[Matthias Bläsing]

Hi,

Am Sonntag, den 01.10.2017, 08:21 +0300 schrieb Emilian Bold:
> I believe it would be best to use our full name and @apache alias
> when committing to the main repository.
> 
> Right now I see a mix of company, personal and .local email addresses
> as well as complete name or just (local machine) usernames.
> 
> Actually I don't care about the email address that much, but it must
> be something reachable.
> 
> The full name though seems important.

I agree with Emilians assessment, the author information is important
and needs to be correct. It will also make it easier to see if work can
be merged directly. If I'm not mistaken code can be merged under two
conditions:

   1. Either the change is deemed small/simple enough not to require an
  ICLA
   2. The author needs to have an ICLA on file

For the first condition I'll rely on my gut feeling, for the second an
apache email adresse as the author is a clear indication.

So I'd suggest to use the atpache address as the primary author
address. 

Greetings

Matthias

[GitHub] incubator-netbeans issue #11: Fixes bug in FileChooserBuilderTest

[emilianbold]

Github user emilianbold commented on the issue:

https://github.com/apache/incubator-netbeans/pull/11
  
Committed.


---

[GitHub] incubator-netbeans pull request #11: Fixes bug in FileChooserBuilderTest

[emilianbold]

Github user emilianbold closed the pull request at:

https://github.com/apache/incubator-netbeans/pull/11


---

[GitHub] incubator-netbeans pull request #19: [NETBEANS-54] Module Review openide.nod...

[emilianbold]

Github user emilianbold closed the pull request at:

https://github.com/apache/incubator-netbeans/pull/19


---

[GitHub] incubator-netbeans issue #19: [NETBEANS-54] Module Review openide.nodes

[emilianbold]

Github user emilianbold commented on the issue:

https://github.com/apache/incubator-netbeans/pull/19
  
Committed.


---