Thread: [Pydev-code] [PATCH] Patches to store pydev project parameter in the project + "Default Interpreter
Brought to you by:
fabioz
From: Gergely K. <ger...@gm...> - 2006-10-22 22:25:16
|
Hello, I created 2 patches for PyDev: 1. Add a "Default Interpreter" option to the launch configuration settings. This makes it possible to share launch configurations among developers: https://sourceforge.net/tracker/index.php?func=detail&aid=1582551&group_id=85796 2. Store pydev project parameters in the project itself. This makes it possible to include these settings (PYTHONPATH and Python Version) in version control systems. https://sourceforge.net/tracker/index.php?func=detail&aid=1582557&group_id=85796 Please consider these patches for inclusion. I also created 2 small changes in the releng build xml files: https://sourceforge.net/tracker/index.php?func=detail&aid=1582560&group_id=85796(Upgrade Retroweaver task) https://sourceforge.net/tracker/index.php?func=detail&aid=1582561&group_id=85796(forceContextQualifier support) Best Regards, Gergely |
From: Fabio Z. <fa...@gm...> - 2006-10-23 01:07:54
|
On 10/22/06, Gergely Kis <ger...@gm...> wrote: > > Hello, > > I created 2 patches for PyDev: > > 1. Add a "Default Interpreter" option to the launch configuration > settings. This makes it possible to share launch configurations among > developers: > > https://sourceforge.net/tracker/index.php?func=detail&aid=1582551&group_id=85796 > > 2. Store pydev project parameters in the project itself. This makes it > possible to include these settings (PYTHONPATH and Python Version) in > version control systems. > > https://sourceforge.net/tracker/index.php?func=detail&aid=1582557&group_id=85796 > > Please consider these patches for inclusion. > > I also created 2 small changes in the releng build xml files: > https://sourceforge.net/tracker/index.php?func=detail&aid=1582560&group_id=85796 > (Upgrade Retroweaver task) > https://sourceforge.net/tracker/index.php?func=detail&aid=1582561&group_id=85796 > (forceContextQualifier support) > Hi Gergely, Thanks for the patches, I'll try to take a look at them tomorrow and will keep you informed (through the sourceforge tracker). Cheers, Fabio |
From: Gergely K. <ger...@gm...> - 2006-10-23 13:55:47
|
Hi, On 10/23/06, Fabio Zadrozny <fa...@gm...> wrote: > > > > Hi Gergely, > > Thanks for the patches, > > I'll try to take a look at them tomorrow and will keep you informed > (through the sourceforge tracker). > > Cheers, > > Fabio > Thanks Fabio. It also looks like that I can only attach files to tickets that I created. Is there a sourceforge.net setting to change this? I wanted to attach my PYTHONPATH patch to the related feature request ticket, but failed. Best Regards, Gergely |
From: Fabio Z. <fa...@gm...> - 2006-10-24 01:07:39
|
On 10/23/06, Gergely Kis <ger...@gm...> wrote: > > Hi, > > On 10/23/06, Fabio Zadrozny <fa...@gm...> wrote: > > > > > > > > Hi Gergely, > > > > Thanks for the patches, > > > > I'll try to take a look at them tomorrow and will keep you informed > > (through the sourceforge tracker). > > > > Cheers, > > > > Fabio > > > Thanks Fabio. It also looks like that I can only attach files to tickets > that I created. Is there a sourceforge.net setting to change this? I > wanted to attach my PYTHONPATH patch to the related feature request ticket, > but failed. > > Humm... I didn't find any way to make that possible in the tracker... Still, doing a new request for that doesn't seem that bad (putting a link seems a nice thing in this case). BTW: I've put some comments and have left that request in pending state (but already patched it). Cheers, Fabio |
From: Gergely K. <ger...@gm...> - 2006-10-24 20:45:44
|
Hi, Thanks for committing the patches. I will post the change you requested. I plan to change the .pydevproject property format to the following: <pydev_project> <pydev_property name="x.y.z">text</pydev_property> <pydev_property name="a.b.c"> <path>p/q</path> <path>w/z</path> </ |
From: Gergely K. <ger...@gm...> - 2006-10-24 20:56:57
|
Hello, Well, Gmail sent it before I was ready...sorry for that. So the changed format would be: <pydev_project> <pydev_property name=" x.y.z">text</pydev_property> <pydev_pathproperty name="a.b.c"> <path>p/q</path> <path>w/z</path> </pydev_pathproperty> </pydev_project> I also plan to extend the interface of the PythonNatureStore: getPathProperty(QualifiedName key) setPathProperty(QualifiedName key, String value) These methods would deal with the second property type. The PythonPathNature would use these path properties. Later (when someone has the time :-) ) additional PathProperty accessors could be defined so that the internal PyDev code doesn't need to deal with the splitting / joining of strings: String[] getPathProperty(QualifiedName key) setPathProperty(QualifiedName key, String[] paths) What do you think? Best Regards, Gergely |
From: Fabio Z. <fa...@gm...> - 2006-10-24 23:33:22
|
> > > So the changed format would be: > <pydev_project> > <pydev_property name=" x.y.z">text</pydev_property> > <pydev_pathproperty name="a.b.c"> > <path>p/q</path> > <path>w/z</path> > </pydev_pathproperty> > </pydev_project> > > I also plan to extend the interface of the PythonNatureStore: > > getPathProperty(QualifiedName key) > > setPathProperty(QualifiedName key, String value) Seems a nice format to me ;-) These methods would deal with the second property type. The PythonPathNature > would use these path properties. > Later (when someone has the time :-) ) additional PathProperty accessors > could be defined so that the internal PyDev code doesn't need to deal with > the splitting / joining of strings: > > String[] getPathProperty(QualifiedName key) > > setPathProperty(QualifiedName key, String[] paths) > > What do you think? I think it's a good idea (but as you said, it can wait for someone to have the time to do it) I'll also wait for your patches regarding the xml project format, for doing the next release, so that we don't have problems on doing backward-compatibility for that. Cheers, Fabio |
From: Gergely K. <ger...@gm...> - 2006-10-25 21:01:46
|
Hello, Uploaded the implementation: https://sourceforge.net/tracker/index.php?func=detail&aid=1582557&group_id=85796&atid=577332 I hope you will like it. Best Regards, Gergely |