As I see it was done only for <disable> tags, but not for <projectIndexer>. 

Regards, 
Alex 

----- Original Message ----- 
From: "Roy Ganor" <r...@zend.com> 
To: a...@xored.com 
Cc: "DLTK Developer Discussions" <dltk-...@eclipse.org>, pdt-dev@eclipse.org 
Sent: Monday, August 17, 2009 2:41:42 PM GMT +06:00 Almaty, Novosibirsk 
Subject: RE: [Dltk-dev] Re: index2 




This is exactly what I did in org.eclipse.php.core/plugin.xml, maybe you’re not 
sync? 



Thanks, 

Btw- you can remove the comment in the dltkcore plugin about the 
ProjectIndexer2. 







From: a...@xored.com [mailto:a...@xored.com] 
Sent: Monday, August 17, 2009 10:24 AM 
To: Roy Ganor 
Cc: DLTK Developer Discussions; pdt-dev@eclipse.org; Michael Spector 
Subject: Re: [Dltk-dev] Re: index2 




Hi Roy, 

Could you please enable index2 only for PHP, i.e.: 

### Eclipse Workspace Patch 1.0 
#P org.eclipse.php.core 
Index: plugin.xml 
=================================================================== 
RCS file: 
/cvsroot/tools/org.eclipse.pdt/plugins/org.eclipse.php.core/plugin.xml,v 
retrieving revision 1.67 
diff -u -r1.67 plugin.xml 
--- plugin.xml 16 Aug 2009 07:52:14 -0000 1.67 
+++ plugin.xml 17 Aug 2009 06:56:08 -0000 
@@ -305,7 +306,7 @@ 
point="org.eclipse.dltk.core.projectIndexer"> 
<projectIndexer 
class="org.eclipse.dltk.core.index2.ProjectIndexer2" 
- nature="#"> 
+ nature="org.eclipse.php.core.PHPNature"> 
</projectIndexer> 
</extension> 

Thank you, 
Alex 

----- Original Message ----- 
From: "Roy Ganor" <r...@zend.com> 
To: a...@xored.com 
Cc: "DLTK Developer Discussions" <dltk-...@eclipse.org>, pdt-dev@eclipse.org, 
"Michael Spector" <mich...@zend.com> 
Sent: Thursday, August 13, 2009 11:22:57 PM GMT +06:00 Almaty, Novosibirsk 
Subject: RE: [Dltk-dev] Re: index2 


the index2.ProjectIndexer2 is now enabled for php nature (following Alex’s 
changes) 



Thanks for clarifying, 

Roy 



From: a...@xored.com [mailto:a...@xored.com] 
Sent: Tuesday, August 11, 2009 7:52 PM 
To: Roy Ganor 
Cc: DLTK Developer Discussions; pdt-dev@eclipse.org 
Subject: Re: [Dltk-dev] Re: index2 




Hi Roy, 

It looks like we have some misunderstanding here. 
h2-1.1.116.jar is already a plugin. 

Regards, 
Alex 

----- Original Message ----- 
From: "Roy Ganor" <r...@zend.com> 
To: a...@xored.com 
Cc: "DLTK Developer Discussions" <dltk-...@eclipse.org>, pdt-dev@eclipse.org 
Sent: Tuesday, August 11, 2009 11:02:24 PM GMT +06:00 Almaty, Novosibirsk 
Subject: RE: [Dltk-dev] Re: index2 


Hi Alex, 

Problem is that until h2 is included in orbit, PDT users need to create a 
plugin manually that includes the h2 library and then load it somehow. I think 
it is too complicated... 

Any chance to make this plugin internal to core or as Michael created initially 
as an external one? 



Thank you for your time, 

Roy 



From: a...@xored.com [mailto:a...@xored.com] 
Sent: Tuesday, August 11, 2009 5:55 PM 
To: Roy Ganor 
Cc: DLTK Developer Discussions; pdt-dev@eclipse.org 
Subject: Re: [Dltk-dev] Re: index2 




Hi Roy, 

I think we should not include (optional) external dependencies into DLTK 
builds. 
Users can download h2 from the link below until it is included in Orbit. 

What do you think? 

And DLTK includes org.eclipse.dltk.index.sql.* plugins actually using h2. 

Regards, 
Alex 

----- Original Message ----- 
From: "Roy Ganor" <r...@zend.com> 
To: "DLTK Developer Discussions" <dltk-...@eclipse.org>, "Alex Panchenko" 
<a...@xored.com>, pdt-dev@eclipse.org 
Sent: Tuesday, August 11, 2009 9:36:57 PM GMT +06:00 Almaty, Novosibirsk 
Subject: RE: [Dltk-dev] Re: index2 


Hi Alex, 

Eclipse PDT was built against the latest integration DLTK build already, thanks 
for providing this build. I’ll be able to make the adjustments later on this 
week. 

Can you explain why org.h2.core was not included in the core DLTK package? 



Best Regards, 

Roy 



Btw - Michael - thanks for forwarding, have fun J 


From: dltk-dev-boun...@eclipse.org [mailto:dltk-dev-boun...@eclipse.org] On 
Behalf Of Michael Spector 
Sent: Tuesday, August 11, 2009 4:56 PM 
To: Alex Panchenko; DLTK Developer Discussions; pdt-dev@eclipse.org 
Subject: [Dltk-dev] Re: index2 




Hi Alex, 





I'm forwarding this mail to DLTK/PDT mailing lists, since I'm unable to check 
this (I'm on holiday). 





Thanks, 


Michael 


On Sat, Aug 8, 2009 at 12:23 AM, Alex Panchenko < a...@xored.com > wrote: 

Hi Michael, 

1) I have built h2.jar by their own build script, it is available at 
http://www.eclipse.org/downloads/download.php?file=/technology/dltk/externals/h2-1.1.116.jar
 

It should not be downloaded as part of DLTK plugins, since it's external 
dependency and users should download it explicitly. 

2) I have made code adjustments, so Ruby & TCL work as before even if new 
plugins are available. 
(However, I believe we need additional checks & testing in this area). 
Please make adjustments to your extensions (contribute indexer2 and disable 
other indexers only for your nature) and check if it works for you. 

If you have any questions - let's discuss them. 

When everything is fine for you & us - I'll make new build and upload it to 
eclipse. 
I really don't want to make buggy builds publicly available. 

Thank you, 
Alex 

_______________________________________________
pdt-dev mailing list
pdt-dev@eclipse.org
https://dev.eclipse.org/mailman/listinfo/pdt-dev

Reply via email to