Oh, and here's the patch by the way:

=== modified file 'config/pandora-plugin'
--- config/pandora-plugin    2011-02-28 18:00:50 +0000
+++ config/pandora-plugin    2011-04-01 04:49:03 +0000
@@ -20,10 +20,19 @@

 # Find plugins in the tree and add them to the build system

-import ConfigParser, os, sys
+import os, sys
 import datetime, time
 import subprocess

+# This assures compatibility between Python 2.x and 3.x
+try:    # Python 2.x
+  import ConfigParser
+  os_path_walk = os.path.walk
+except ImportError:    # Python 3.x
+  import configparser
+  ConfigParser = configparser
+  os_path_walk = os.walk
+
 plugin_am_file=None
 plugin_ac_file=None
 plugin_doc_index=None
@@ -54,7 +63,7 @@
     new_content = self.new_file.read()
     self.new_file.close()
     try:
-        old_file = file(self.real_fname, 'r')
+        old_file = open(self.real_fname, 'r')
         old_content = old_file.read()
         old_file.close()
     except IOError:
@@ -182,7 +191,7 @@
     if plugin['writing_status'] == 'done':
       return
     else:
-      print "Dependency loop detected with %s" % plugin['name']
+      sys.stdout.write("Dependency loop detected with %s" % plugin['name'])
       exit(1)

   plugin['writing_status'] = 'dependencies'
@@ -196,7 +205,7 @@
         write_plugin(find_plugin, plugin_ini_list)
         break
     if found is False:
-      print "Could not find dependency %s: %s" % (plugin['name'],
dependency)
+      sys.stdout.write("Could not find dependency %s: %s" %
(plugin['name'], dependency))
       exit(1)

   write_plugin_ac(plugin, plugin_ac_file)
@@ -341,7 +350,7 @@

 def expand_plugin_ini(plugin):
     if plugin['name'] == "**OUT-OF-TREE**":
-      print "Out of tree plugins require the name field to be specified in
plugin.ini"
+      sys.stdout.write("Out of tree plugins require the name field to be
specified in plugin.ini")
       sys.exit(1)

     if plugin['plugin_dir'] == ".":
@@ -440,11 +449,11 @@
     plugin['plugin_dir'] = plugin_dir
     if plugin_dir == '.':
       if not plugin.has_key('url'):
-        print "External Plugins are required to specifiy a url"
+        sys.stdout.write("External Plugins are required to specifiy a url")
         plugin['url']= 'http://launchpad.net/%(name)s' % plugin
         sys.exit(1)
       if plugin_dir == '.' and not plugin.has_key('version'):
-        print "External Plugins are required to specifiy a version"
+        sys.stdout.write("External Plugins are required to specifiy a
version")
         sys.exit(1)
     if not plugin.has_key('version'):
       plugin['version'] = config['default_plugin_version']
@@ -652,7 +661,7 @@
   if config['plugin_ini_fname'] in fnames:
     arg.append(dirname)

-os.path.walk(os.path.join(config['top_srcdir'],
+os_path_walk(os.path.join(config['top_srcdir'],
                           config['root_plugin_dir']),
              accumulate_plugins,
              plugin_list)
@@ -738,7 +747,7 @@
   plugin_name_list = [plugin['libname'] for plugin in plugin_ini_list]
   for plugin in plugin_ini_list:
     if plugin_name_list.count(plugin['libname']) != 1:
-      print "Duplicate module name %s" % plugin['libname']
+      sys.stdout.write("Duplicate module name %s" % plugin['libname'])
       exit(1)

   for plugin in plugin_ini_list:



On Thu, Mar 31, 2011 at 11:28 PM, Juan Antonio Osorio
<[email protected]>wrote:

> Well, I did a patch for the pandora-plugin compatibility issue, and and had
> no problems
> when I ran the autorun.sh script, but when using the makefile I get the
> following error:
>
> drizzled/module/loader.cc:63:30: error: expected unqualified-id before
> ‘__null’
> drizzled/module/loader.cc:64:30: error: expected unqualified-id before
> ‘__null’
> make[2]: *** [drizzled/module/loader.o] Error 1
> make[2]: Leaving directory `/home/ozz/drizzle/trunk'
> make[1]: *** [all-recursive] Error 1
> make[1]: Leaving directory `/home/ozz/drizzle/trunk'
>
> which points to this:
>
> extern drizzled_builtin_list PANDORA_BUILTIN_SYMBOLS_LIST;
> extern drizzled_builtin_list PANDORA_BUILTIN_LOAD_SYMBOLS_LIST;
>
> What could be the problem?
>
>
>
> On Thu, Mar 31, 2011 at 11:15 AM, Juan Antonio Osorio <[email protected]
> > wrote:
>
>> thanks, also, from what I'm seeing, the ones in mysql_protocol are in
>> python 3 synthax O_o.
>>
>>
>>
>> On Thu, Mar 31, 2011 at 11:04 AM, Monty Taylor <[email protected]>wrote:
>>
>>> No, there is also config/pandora-plugin. That is the one I would work on
>>> first, since it is the one which will prevent you from building drizzle
>>> at all if it doesn't work. The others are just related to testing.
>>>
>>> On 03/31/2011 12:50 PM, Juan Antonio Osorio wrote:
>>> > Well, I could give it a try, are these the only python files in the
>>> > source code?
>>> >
>>> > ./plugin/mysql_protocol/prototest/prototest/mysql/command.py
>>> > ./plugin/mysql_protocol/prototest/prototest/mysql/result.py
>>> > ./plugin/mysql_protocol/prototest/prototest/mysql/bitfield.py
>>> > ./plugin/mysql_protocol/prototest/prototest/mysql/handshake.py
>>> > ./plugin/mysql_protocol/prototest/prototest/mysql/__init__.py
>>> > ./plugin/mysql_protocol/prototest/prototest/mysql/column.py
>>> > ./plugin/mysql_protocol/prototest/prototest/mysql/packet.py
>>> > ./plugin/mysql_protocol/prototest/prototest/__init__.py
>>> > ./tests/lib/test_run_options.py
>>> > ./tests/lib/test_mgmt/test_execution.py
>>> > ./tests/lib/test_mgmt/test_management.py
>>> > ./tests/lib/test_mgmt/__init__.py
>>> > ./tests/lib/test_mgmt/execution_management.py
>>> > ./tests/lib/__init__.py
>>> > ./tests/lib/uuid.py
>>> > ./tests/lib/sys_mgmt/logging_management.py
>>> > ./tests/lib/sys_mgmt/port_management.py
>>> > ./tests/lib/sys_mgmt/__init__.py
>>> > ./tests/lib/sys_mgmt/time_management.py
>>> > ./tests/lib/sys_mgmt/codeTree.py
>>> > ./tests/lib/sys_mgmt/system_management.py
>>> > ./tests/lib/test_mode.py
>>> > ./tests/lib/drizzle_test_run/dtr_test_execution.py
>>> > ./tests/lib/drizzle_test_run/dtr_test_management.py
>>> > ./tests/lib/drizzle_test_run/__init__.py
>>> > ./tests/lib/server_mgmt/server.py
>>> > ./tests/lib/server_mgmt/__init__.py
>>> > ./tests/lib/server_mgmt/drizzled.py
>>> > ./tests/lib/server_mgmt/server_management.py
>>> > ./tests/lib/randgen/randgen_test_execution.py
>>> > ./tests/lib/randgen/__init__.py
>>> > ./tests/lib/randgen/randgen_test_management.py
>>> > ./tests/dbqp.py
>>> > ./docs/conf.py
>>> > ./docs/pyext/dbtable.py
>>> > ./docs/pyext/confval.py
>>> > ./docs/pyext/__init__.py
>>> > ./docs/pyext/options.py
>>> >
>>> >
>>> > On Thu, Mar 31, 2011 at 2:35 AM, Monty Taylor <[email protected]
>>> > <mailto:[email protected]>> wrote:
>>> >
>>> >     On 03/31/2011 02:20 AM, Andrew Hutchings wrote:
>>> >     > Hi Juan,
>>> >     >
>>> >     > On 31/03/11 02:22, Juan Antonio Osorio wrote:
>>> >     >> My name is Juan Antonio and I'm interested in applying to work
>>> in the
>>> >     >> project in summer
>>> >     >> through the GSoC, but when I tried to compile the source code I
>>> >     got this:
>>> >     >>
>>> >     >> I'm guessing that plugin depends on python2.7 and I'm using
>>> >     python3.2.
>>> >     >> Does the code depend
>>> >     >> on a lot of python code? maybe I could port it or try to make it
>>> >     syntax
>>> >     >> independent.
>>> >     >
>>> >     > That is the Pandora build system, and yes it depends upon Python
>>> >     2.x.  I
>>> >     > think the only other two things we use python for is docs (using
>>> >     > python-sphinx) and testing (tests/dbqp).
>>> >     >
>>> >     > As for porting Pandora I would recommend talking to Monty Taylor
>>> >     > (mtaylor on IRC) since he heads that project.
>>> >
>>> >     I would totally accept patches to allow pandora-build to run in
>>> python3,
>>> >     as long as they will also work with python2 (since most of the
>>> distros
>>> >     have not moved yet)
>>> >
>>> >     Monty
>>> >
>>> >     _______________________________________________
>>> >     Mailing list: https://launchpad.net/~drizzle-discuss
>>> >     <https://launchpad.net/%7Edrizzle-discuss>
>>> >     Post to     : [email protected]
>>> >     <mailto:[email protected]>
>>> >     Unsubscribe : https://launchpad.net/~drizzle-discuss
>>> >     <https://launchpad.net/%7Edrizzle-discuss>
>>> >     More help   : https://help.launchpad.net/ListHelp
>>> >
>>> >
>>> >
>>> >
>>> > --
>>> > Juan Antonio Osorio R.
>>> > e-mail: [email protected] <mailto:[email protected]>
>>> >
>>> > "All truly great thoughts are conceived by walking."
>>> > - F.N.
>>> >
>>> >
>>>
>>>
>>
>>
>> --
>> Juan Antonio Osorio R.
>> e-mail: [email protected]
>>
>> "All truly great thoughts are conceived by walking."
>> - F.N.
>>
>>
>>
>
>
> --
> Juan Antonio Osorio R.
> e-mail: [email protected]
>
> "All truly great thoughts are conceived by walking."
> - F.N.
>
>
>


-- 
Juan Antonio Osorio R.
e-mail: [email protected]

"All truly great thoughts are conceived by walking."
- F.N.
_______________________________________________
Mailing list: https://launchpad.net/~drizzle-discuss
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~drizzle-discuss
More help   : https://help.launchpad.net/ListHelp

Reply via email to