Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[pdt-dev] RE: [Dltk-dev] Re: index2

I removed the redundant extension, thanks!

 

Roy

From: alex@xxxxxxxxx [mailto:alex@xxxxxxxxx]
Sent: Monday, August 17, 2009 1:35 PM
To: Roy Ganor
Cc: DLTK Developer Discussions; pdt-dev@xxxxxxxxxxx
Subject: Re: [Dltk-dev] Re: index2

 

Hm, it turns out you have ProjectIndexer2 mentioned twice, so here is updated patch:

### 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 10:28:57 -0000
@@ -1,5 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<?eclipse version="3.0"?>
<!--
+<?eclipse version="3.0"?>
+<!--
     Copyright (c) 2009 IBM Corporation and others.
     All rights reserved. This program and the accompanying materials
     are made available under the terms of the Eclipse Public License v1.0
@@ -303,15 +304,6 @@
   
    <extension
          point="org.eclipse.dltk.core.projectIndexer">
-      <projectIndexer
-            class="org.eclipse.dltk.core.index2.ProjectIndexer2"
-            nature="#">
-      </projectIndexer>
-    </extension>
-  
-  
-   <extension
-         point="org.eclipse.dltk.core.projectIndexer">
       <disable
               nature="org.eclipse.php.core.PHPNature"
             class="org.eclipse.dltk.internal.core.structure.StructureProjectIndexer">


Regards,
Alex

----- Original Message -----
From: "Roy Ganor" <roy@xxxxxxxx>
To: alex@xxxxxxxxx
Cc: "DLTK Developer Discussions" <dltk-dev@xxxxxxxxxxx>, pdt-dev@xxxxxxxxxxx
Sent: Monday, August 17, 2009 5:26:40 PM GMT +06:00 Almaty, Novosibirsk
Subject: RE: [Dltk-dev] Re: index2

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: alex@xxxxxxxxx [mailto:alex@xxxxxxxxx]
Sent: Monday, August 17, 2009 12:36 PM
To: Roy Ganor
Cc: DLTK Developer Discussions; pdt-dev@xxxxxxxxxxx
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" <roy@xxxxxxxx>
To: alex@xxxxxxxxx
Cc: "DLTK Developer Discussions" <dltk-dev@xxxxxxxxxxx>, pdt-dev@xxxxxxxxxxx
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: alex@xxxxxxxxx [mailto:alex@xxxxxxxxx]
Sent: Monday, August 17, 2009 10:24 AM
To: Roy Ganor
Cc: DLTK Developer Discussions; pdt-dev@xxxxxxxxxxx; 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" <roy@xxxxxxxx>
To: alex@xxxxxxxxx
Cc: "DLTK Developer Discussions" <dltk-dev@xxxxxxxxxxx>, pdt-dev@xxxxxxxxxxx, "Michael Spector" <michael@xxxxxxxx>
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: alex@xxxxxxxxx [mailto:alex@xxxxxxxxx]
Sent: Tuesday, August 11, 2009 7:52 PM
To: Roy Ganor
Cc: DLTK Developer Discussions; pdt-dev@xxxxxxxxxxx
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" <roy@xxxxxxxx>
To: alex@xxxxxxxxx
Cc: "DLTK Developer Discussions" <dltk-dev@xxxxxxxxxxx>, pdt-dev@xxxxxxxxxxx
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: alex@xxxxxxxxx [mailto:alex@xxxxxxxxx]
Sent: Tuesday, August 11, 2009 5:55 PM
To: Roy Ganor
Cc: DLTK Developer Discussions; pdt-dev@xxxxxxxxxxx
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" <roy@xxxxxxxx>
To: "DLTK Developer Discussions" <dltk-dev@xxxxxxxxxxx>, "Alex Panchenko" <alex@xxxxxxxxx>, pdt-dev@xxxxxxxxxxx
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-bounces@xxxxxxxxxxx [mailto:dltk-dev-bounces@xxxxxxxxxxx] On Behalf Of Michael Spector
Sent: Tuesday, August 11, 2009 4:56 PM
To: Alex Panchenko; DLTK Developer Discussions; pdt-dev@xxxxxxxxxxx
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 <alex@xxxxxxxxx> 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

 


Back to the top