Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [wtp-dev] Questions about JSDT development

Thanks for the warm welcome everyone.

I finally got patches for these two issues pushed into Gerrit.  Actually, I filed a new Bugzilla ticket for the latter issue since there wasn't one yet.

I've never used Gerrit before.  I assume it will send an email when action is needed (code defects, etc.)?

363580 - RhinoRemote debugger can not find the thread because it's too quick and it's finished before the RhinoRequest arrives.
https://bugs.eclipse.org/bugs/show_bug.cgi?id=363580

485301 - Files shown in 'External _javascript_Source' can be the wrong file
https://git.eclipse.org/r/63785


On Fri, Dec 25, 2015 at 11:18 AM, Victor V. Rubezhny <vrubezhny@xxxxxxxxxx> wrote:
Philip,

That's great! Thank you for your fix proposals!

Have good holidays!

Kind Regards,
Victor Rubezhny,
JSDT


On 12/25/2015 07:13 PM, Philip Lowman wrote:
Mickael,

Thanks for the pointers, compiling with Java 8 solved the problem.

I'll signup for an Eclipse account and push my patches via Gerrit soon.  One is for bug 363580 to prevent the debugger from resuming all threads when Context.enter() & exit is called without starting a script thread (a very common use case for us).  The other is a bug having to do with collisions in generated filenames in the External _javascript_ Source folder (which prevents opening the correct source file when it happens).


On Wed, Dec 23, 2015 at 4:55 AM, Mickael Istria <mistria@xxxxxxxxxx> wrote:
On 12/23/2015 04:40 AM, Philip Lowman wrote:
Hi,
Hi,
I've developed a couple of fixes for the _javascript_ debugger
Great!
Thanks and welcome!

1. If I need to be concerned with the following build error for the org.eclipse.wst.jsdt.doc plugin[1] which I get after following the directions at the URL below.  The error message sounds like I should be compiling with the Java EE SDK.  Should I be compiling with the Java EE SDK or is it acceptable to just disable this plugin and develop with the Java SE SDK?
osgi.ee means "OSGi Execution Environment", which is configured per bundle in the Bundle-RequiredExecutionEnvironment directlve in the MANIFEST.MF. It's not related to Java EE.
Those errors seem to highlight that you're using Java 7, whereas most recent Eclipse bundles require Java 8. So I believe you should simply install Java 8 and make sure you have it well configured for the build to use it. "mvn --help" can be used to tell you which version of Java your build is using.

2. Directions on how to contribute fixes.  What is the preferred process for JSDT?  Is it to sign the agreement referenced on this page and just attach patches to Bugzilla (filing bugs when necessary)?
The most efficient and recommended workflow is to push your patches via Gerrit: https://wiki.eclipse.org/JSDT/Development#Pushing_a_new_patch_for_review

HTH
--
Mickael Istria
Eclipse developer at JBoss, by Red Hat
My blog - My Tweets

_______________________________________________
wtp-dev mailing list
wtp-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/wtp-dev



--
Philip Lowman


_______________________________________________
wtp-dev mailing list
wtp-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/wtp-dev


CONFIDENTIALITY NOTICE: This email and files attached to it are confidential. If you are not the intended recipient you are hereby notified that using, copying, distributing or taking any action in reliance on the contents of this information is strictly prohibited. If you have received this email in error please notify the sender and delete this email.


_______________________________________________
wtp-dev mailing list
wtp-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/wtp-dev



--
Philip Lowman

Back to the top