[GitHub] incubator-netbeans pull request #113: [NETBEANS-54] Module Review libs.xerce...

2017-10-09 Thread vieiro
GitHub user vieiro opened a pull request: https://github.com/apache/incubator-netbeans/pull/113 [NETBEANS-54] Module Review libs.xerces - Updated maven coordinates for Xerces 2.8.0 (Apache 2). - Added xerces notice from

Re: Library not available in maven (jvyamlb)

2017-10-09 Thread Antonio
See https://issues.apache.org/jira/browse/NETBEANS-82 On 09/10/17 22:04, Antonio wrote: Hi all, The "libs.jvyamlb" module wraps Ola Bini's JvYAMLb library, version 0.2.7, but in maven central only version 0.2.6 is available. Ola Bini keeps a github page [1] with the library, and it seems he

Re: [Mentors] Exclude .template files?

2017-10-09 Thread Junichi Yamamoto
Hi, > In fact this is already the case in the codebase. The review of the > hibernate module done by Illya Kysil revealed one such file: FYI: The templates of html module also have it. https://github.com/apache/incubator-netbeans/tree/master/html/src/org/netbeans/modules/html/templates I

[GitHub] incubator-netbeans issue #73: [NETBEANS-54] Module Review profiler.j2se

2017-10-09 Thread junichi11
Github user junichi11 commented on the issue: https://github.com/apache/incubator-netbeans/pull/73 @varaghar If you come from @emilianbold 's repo (https://github.com/emilianbold/incubator-netbeans), please send your PR there. i.e. not `apache:master` but `emilianbold:master`. ---

Re: [legal] [mentors] Incorporating dendencies into the final JAR

2017-10-09 Thread Craig Russell
> On Oct 9, 2017, at 9:45 AM, Emilian Bold wrote: > > Hello, > > For websvc.saas.api/ I see that 2 xsd files, which have the original GPL w/ > CPE + CDDL NetBeans dual license have been moved into an external > dependency,

Library not available in maven (jvyamlb)

2017-10-09 Thread Antonio
Hi all, The "libs.jvyamlb" module wraps Ola Bini's JvYAMLb library, version 0.2.7, but in maven central only version 0.2.6 is available. Ola Bini keeps a github page [1] with the library, and it seems he pushed v0.2.7 to the "codehaus-jruby-repository" maven repository. So, what should we

Re: [legal] [mentors] Incorporating dendencies into the final JAR

2017-10-09 Thread Wade Chandler
If they are NB specific files that seems exactly like what should happen…the files be donated. Is that what you are saying Emi? Wade > On Oct 9, 2017, at 12:45, Emilian Bold wrote: > > Hello, > > For websvc.saas.api/ I see that 2 xsd files, which have the original

[GitHub] incubator-netbeans pull request #112: [NETBEANS-54] Module Review libs.lucen...

2017-10-09 Thread vieiro
GitHub user vieiro opened a pull request: https://github.com/apache/incubator-netbeans/pull/112 [NETBEANS-54] Module Review libs.lucene - Added lucene notice file from https://github.com/apache/lucene-solr/blob/releases/lucene-solr/3.5.0/lucene/NOTICE.txt - Added the maven

Re: Heads up: Oracle internal urls (& git repos?) for unit tests

2017-10-09 Thread Antonio
On 09/10/17 08:47, Jaroslav Tulach wrote: Unit tests shall be unit tests - e.g. not require special network setup to succeed. That's a sentence that should be engraved in many foreheads... :-) I don't know what the NetBeans qa/functional testing infrastructure would require. Maybe some

[GitHub] incubator-netbeans issue #107: Removed failing tests dependent on nbm

2017-10-09 Thread matthiasblaesing
Github user matthiasblaesing commented on the issue: https://github.com/apache/incubator-netbeans/pull/107 Wouldn't it be better to disable the test instead of deleting them? Even if they are not executed right now, the next one working on the modules might be supported by the code

[GitHub] incubator-netbeans pull request #103: [NETBEANS-54] Module Review libs.junit...

2017-10-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-netbeans/pull/103 ---

[GitHub] incubator-netbeans pull request #54: added initial .travis.yml

2017-10-09 Thread krichter722
Github user krichter722 closed the pull request at: https://github.com/apache/incubator-netbeans/pull/54 ---

[GitHub] incubator-netbeans issue #54: added initial .travis.yml

2017-10-09 Thread krichter722
Github user krichter722 commented on the issue: https://github.com/apache/incubator-netbeans/pull/54 > @krichter722 thank you for your interest. I'm sorry, but this PR was overlooked, and another travis.yml was already merged. Please close this PR, but stay with the project, there

[GitHub] incubator-netbeans pull request #104: [NETBEANS-54] Module Review libs.osgi

2017-10-09 Thread matthiasblaesing
Github user matthiasblaesing commented on a diff in the pull request: https://github.com/apache/incubator-netbeans/pull/104#discussion_r143545978 --- Diff: libs.osgi/external/binaries-list --- @@ -14,5 +14,5 @@ # KIND, either express or implied. See the License for the #

[GitHub] incubator-netbeans issue #54: added initial .travis.yml

2017-10-09 Thread matthiasblaesing
Github user matthiasblaesing commented on the issue: https://github.com/apache/incubator-netbeans/pull/54 @krichter722 thank you for your interest. I'm sorry, but this PR was overlooked, and another travis.yml was already merged. Please close this PR, but stay with the project, there

[GitHub] incubator-netbeans issue #111: Removed failing tests dependent on deleted re...

2017-10-09 Thread matthiasblaesing
Github user matthiasblaesing commented on the issue: https://github.com/apache/incubator-netbeans/pull/111 A different approach to deleting the test would be to fix it. The keystore and the jars could be created at test time and then used. In anycase I would not delete it -

Re: JDK 10/JDK 18.3 support

2017-10-09 Thread Michael Nascimento
On Mon, Oct 9, 2017 at 3:02 PM, Jan Lahoda wrote: > > > > Jan, I suggest naming it 18.3 for now. As much as I don't like it, I > don't > > see it becoming 10... :-( > > > > I was mostly trying to avoid dealing with a '.' in the branch name :-). So > jdk18_3, OK? Ah, I lost

[GitHub] incubator-netbeans issue #97: Add support for specifying classifier and exte...

2017-10-09 Thread matthiasblaesing
Github user matthiasblaesing commented on the issue: https://github.com/apache/incubator-netbeans/pull/97 Merged - thanks for looking into this. @vieiro @geertjanw you both asked for it - the change is now in master. ---

[GitHub] incubator-netbeans pull request #97: Add support for specifying classifier a...

2017-10-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-netbeans/pull/97 ---

Re: JDK 10/JDK 18.3 support

2017-10-09 Thread Jan Lahoda
On Mon, Oct 9, 2017 at 5:56 PM, Michael Nascimento wrote: > Well, IDEA already has support for it. So I guess we really need a release > Right: http://mail.openjdk.java.net/pipermail/amber-dev/2017-September/002150.html for 9 so we can be the first stable IDE with support

[GitHub] incubator-netbeans pull request #70: WIP custom ant task to produce html rep...

2017-10-09 Thread ebarboni
Github user ebarboni commented on a diff in the pull request: https://github.com/apache/incubator-netbeans/pull/70#discussion_r143516562 --- Diff: nbbuild/build.xml --- @@ -1946,30 +1946,41 @@ It is possible to use -Ddebug.port=3234 -Ddebug.pause=y to start the system in d

Re: Depending on GitHub & Travis was: FYI: Enabling Travis CI for incubator-netbeans repository

2017-10-09 Thread Emilian Bold
No problem, I understand all this and the GitHub Travis integration is pretty handy! What I found odd was that you had to ask INFRA about Travis. But I guess it has something to do with they premium plan Apache is under vs the free plan which has more limits. --emi On Mon, Oct 9, 2017 at 7:01

[GitHub] incubator-netbeans pull request #70: WIP custom ant task to produce html rep...

2017-10-09 Thread JaroslavTulach
Github user JaroslavTulach commented on a diff in the pull request: https://github.com/apache/incubator-netbeans/pull/70#discussion_r143515489 --- Diff: nbbuild/antsrc/org/netbeans/nbbuild/RatHtmlReportTask.java --- @@ -0,0 +1,345 @@ +/** + * Licensed to the Apache Software

Re: Enabling Travis CI for incubator-netbeans repository

2017-10-09 Thread Richard Lowery
I tend to agree… GitLab being very GitHub-ish in nature, but runnable on your own infrastructure. GitLab-CI is very similar to Travis-CI, but yet on-par with the reporting and visualizations you can get from Jenkins. Just my opinion though. -- Richard On 10/9/17, 10:45 AM, "Emilian Bold"

Re: FYI: Enabling Travis CI for incubator-netbeans repository

2017-10-09 Thread Emilian Bold
Well, I also find it odd how much Apache depends on GitHub. I'm OK with Hudson / Jenkins since it's open source and you may (and I have) run it yourself. AFAIK Travis is a pure service, no way to run it on my own server and compile it from source code. --emi On Mon, Oct 9, 2017 at 5:40 PM, Wade

[GitHub] incubator-netbeans pull request #110: [NETBEANS-54] Module Review j2ee.metad...

2017-10-09 Thread juneau001
GitHub user juneau001 opened a pull request: https://github.com/apache/incubator-netbeans/pull/110 [NETBEANS-54] Module Review j2ee.metadata.model.support - no external libraries - checked Rat report, added license header on Table.javax which unfortunately also changed

Re: [VOTE] Release Apache NetBeans HTML/Java API version 1.5

2017-10-09 Thread Ryan Cuprak
+1 > On Oct 8, 2017, at 9:06 PM, John D. Ament wrote: > > Graduation is a measure of the community, not any specific repository. The > entire Apache Netbeans community graduates together. > > The key things we look for: > > - Understand the Apache Way > - Understand

[GitHub] incubator-netbeans pull request #70: WIP custom ant task to produce html rep...

2017-10-09 Thread ebarboni
Github user ebarboni commented on a diff in the pull request: https://github.com/apache/incubator-netbeans/pull/70#discussion_r143476986 --- Diff: nbbuild/antsrc/org/netbeans/nbbuild/reporting/RatHtmlReportTask.java --- @@ -0,0 +1,295 @@ +/** + * Licensed to the Apache

[GitHub] incubator-netbeans pull request #70: WIP custom ant task to produce html rep...

2017-10-09 Thread ebarboni
Github user ebarboni commented on a diff in the pull request: https://github.com/apache/incubator-netbeans/pull/70#discussion_r143460566 --- Diff: nbbuild/antsrc/org/netbeans/nbbuild/reporting/ModuleInfo.java --- @@ -0,0 +1,52 @@ +/** + * Licensed to the Apache Software

[GitHub] incubator-netbeans pull request #109: Not all tests can be compiled right no...

2017-10-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-netbeans/pull/109 ---

[GitHub] incubator-netbeans issue #109: Not all tests can be compiled right now. Comp...

2017-10-09 Thread jlahoda
Github user jlahoda commented on the issue: https://github.com/apache/incubator-netbeans/pull/109 Seems OK. ---

[GitHub] incubator-netbeans pull request #105: [NETBEANS-54] Module Review - Resolves...

2017-10-09 Thread martypowell
Github user martypowell closed the pull request at: https://github.com/apache/incubator-netbeans/pull/105 ---

[GitHub] incubator-netbeans issue #105: [NETBEANS-54] Module Review - Resolves #65 - ...

2017-10-09 Thread martypowell
Github user martypowell commented on the issue: https://github.com/apache/incubator-netbeans/pull/105 @junichi11 thanks wasn't paying attention...will do ---

[DRAFT] Incubator PMC Board Report - October 2017

2017-10-09 Thread John D. Ament
All, Below is the current draft of the board report. Much of the header info is missing, still in flight. We are missing two podling reports and four reports are presently not signed off on. Podling's on CC please make sure you've reviewed. Incubator PMC report for October 2017 The Apache

[GitHub] incubator-netbeans pull request #109: Not all tests can be compiled right no...

2017-10-09 Thread JaroslavTulach
GitHub user JaroslavTulach opened a pull request: https://github.com/apache/incubator-netbeans/pull/109 Not all tests can be compiled right now. Compile them as part of travis run. I've noticed that there are some modules with too wide test dependencies. Such modules cannot be

[GitHub] incubator-netbeans pull request #108: Enable travis

2017-10-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-netbeans/pull/108 ---

Re: FYI: Enabling Travis CI for incubator-netbeans repository

2017-10-09 Thread Jaroslav Tulach
Guys, if you like Travis, please approve: github.com/apache/incubator-netbeans/pull/108 On pondělí 9. října 2017 12:17:41 CEST Emilian Bold wrote: > Is this part of builds.apache.org or is Travis doing the build in some > remote corner of the Cloud? > > I really like the GitHub integration and

Re: FYI: Enabling Travis CI for incubator-netbeans repository

2017-10-09 Thread Emilian Bold
Is this part of builds.apache.org or is Travis doing the build in some remote corner of the Cloud? I really like the GitHub integration and the sanity check on PRs but I'm not entirely cool with it being a commercial service after all. --emi On Mon, Oct 9, 2017 at 11:40 AM, Jaroslav Tulach

FYI: Enabling Travis CI for incubator-netbeans repository

2017-10-09 Thread Jaroslav Tulach
Hi. I've just requested issues.apache.org/jira/browse/INFRA-15249 It should help us verify essential aspects of NetBeans are preserved. For example that the code will remain buildable after merging a PR. -jt

Re: [VOTE] Release Apache NetBeans HTML/Java API version 1.5

2017-10-09 Thread Geertjan Wielenga
I have tested with 10.2.6 and JDK 1.8.0_121 and Maven 3.5.0. The build hangs after this, i.e., the build does not continue and does not complete: Running TestSuite Configuring TestNG with: TestNG652Configurator java.lang.Exception: Error at

Re: Development version

2017-10-09 Thread Jaroslav Tulach
On neděle 8. října 2017 11:27:57 CEST Geertjan Wielenga wrote: > Yes, I saw that -- reported it and it's being looked at. > > Note though that this is unrelated to Apache, i.e., if you want to use > Apache NetBeans, use the Apache NetBeans GitHub repo: > github.com/apache/incubator-netbeans

[GitHub] incubator-netbeans issue #97: Add support for specifying classifier and exte...

2017-10-09 Thread JaroslavTulach
Github user JaroslavTulach commented on the issue: https://github.com/apache/incubator-netbeans/pull/97 Feel free to proceed with integration. +1 ---

[GitHub] incubator-netbeans pull request #97: Add support for specifying classifier a...

2017-10-09 Thread JaroslavTulach
Github user JaroslavTulach commented on a diff in the pull request: https://github.com/apache/incubator-netbeans/pull/97#discussion_r143400960 --- Diff: nbbuild/antsrc/org/netbeans/nbbuild/extlibs/DownloadBinaries.java --- @@ -415,16 +406,113 @@ private String hash(InputStream is)

Re: NetBeans Mentoring/Training Suggestion

2017-10-09 Thread Geertjan Wielenga
I think something deeper than that is needed, much more detailed, on what the modules each provide and how they relate to each other. Gj On Mon, 9 Oct 2017 at 07:56, Jaroslav Tulach wrote: > We've been doing this >

Re: NetBeans Mentoring/Training Suggestion

2017-10-09 Thread Jaroslav Tulach
We've been doing this https://netbeans.org/features/platform/certified-training.html for years. We certainly have a lot material ready. -jt Dne 6. 10. 2017 8:46 Ryan Cuprak napsal(a): > > Hello, > I have been poking around the NetBeans code and had a request/idea.NetBeans

Re: [VOTE] Release Apache NetBeans HTML/Java API version 1.5

2017-10-09 Thread Anton Epple
Hello Jaroslav, +1 I have tested with 10.2.6 and JDK8 u131 and the build succeeds here as well. --Toni Am 09.10.17, 05:37 schrieb "Jaroslav Tulach" : Hello William. Thanks for testing. >> William Beebe : 08.10.17 @ 21:27 <<