This is what I have:

   <extension point="org.eclipse.dltk.core.projectIndexer">

      <disable nature="org.eclipse.php.core.PHPNature" 
class="org.eclipse.dltk.internal.core.structure.StructureProjectIndexer"></disable>

      <disable nature="org.eclipse.php.core.PHPNature" 
class="org.eclipse.dltk.internal.core.mixin.MixinProjectIndexer"></disable>

      <projectIndexer  nature="org.eclipse.php.core.PHPNature" 
class="org.eclipse.dltk.core.index2.ProjectIndexer2"></projectIndexer>

   </extension>

 

As you can see everything was designated for org.eclipse.php.core.PHPNature

 

Are you synced?

Roy

 

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

 

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