On Wed, Apr 12, 2017 at 7:07 PM, Sarah McAlear <smcal...@pivotal.io> wrote:

> Hi Hackers!
>
Hi Sarah,

>
> Is there an update on this?
>
We will look in to it end of this week.
I was not rushing to it, because - Dave was preparing for the 1.4 release.

-- Thanks, Ashesh

>
> Thanks,
> Sarah & Joao
>
> On Fri, Apr 7, 2017 at 10:27 AM, Sarah McAlear <smcal...@pivotal.io>
> wrote:
>
>> Hi Ashesh!
>>
>> Good catch. Looks like there was an override of the input function that
>> didn't get moved to the new file, causing the input with the @ to fail. We
>> also added headers to the files that were missing them. This new patch
>> should work.
>>
>> Thanks!
>> Joao & Sarah
>>
>>
>>
>> On Mon, Apr 3, 2017 at 8:41 AM, Ashesh Vashi <
>> ashesh.va...@enterprisedb.com> wrote:
>>
>>> On Mon, Apr 3, 2017 at 12:09 PM, Ashesh Vashi <
>>> ashesh.va...@enterprisedb.com> wrote:
>>>
>>>> Hi Jaoao, Sarah,
>>>>
>>>> I've tried to run on fresh machine, it failed with the below error:
>>>>
>>> And - I have noticed - the headers are missing in new files.
>>>
>>> --
>>>
>>> Thanks & Regards,
>>>
>>> Ashesh Vashi
>>> EnterpriseDB INDIA: Enterprise PostgreSQL Company
>>> <http://www.enterprisedb.com/>
>>>
>>>
>>> *http://www.linkedin.com/in/asheshvashi*
>>> <http://www.linkedin.com/in/asheshvashi>
>>>
>>>>
>>>> *$ python setup.py*
>>>> *NOTE: Configuring authentication for SERVER mode.*
>>>>
>>>> *Enter the email address and password to use for the initial pgAdmin
>>>> user account:*
>>>>
>>>> *Email address: ashesh.va...@enterprisedb.com
>>>> <ashesh.va...@enterprisedb.com>*
>>>> *Traceback (most recent call last):*
>>>> *  File "setup.py", line 52, in <module>*
>>>> *    db_upgrade(app)*
>>>> *  File
>>>> "/Users/asheshvashi/Developments/Projects/pgAdmin4/web/pgadmin/setup/db_upgrade.py",
>>>> line 25, in db_upgrade*
>>>> *    flask_migrate.upgrade(migration_folder)*
>>>> *  File
>>>> "/Users/asheshvashi/Developments/WorkPlace/pgAdmin4/lib/python2.7/site-packages/flask_migrate/__init__.py",
>>>> line 244, in upgrade*
>>>> *    command.upgrade(config, revision, sql=sql, tag=tag)*
>>>> *  File
>>>> "/Users/asheshvashi/Developments/WorkPlace/pgAdmin4/lib/python2.7/site-packages/alembic/command.py",
>>>> line 254, in upgrade*
>>>> *    script.run_env()*
>>>> *  File
>>>> "/Users/asheshvashi/Developments/WorkPlace/pgAdmin4/lib/python2.7/site-packages/alembic/script/base.py",
>>>> line 416, in run_env*
>>>> *    util.load_python_file(self.dir, 'env.py')*
>>>> *  File
>>>> "/Users/asheshvashi/Developments/WorkPlace/pgAdmin4/lib/python2.7/site-packages/alembic/util/pyfiles.py",
>>>> line 93, in load_python_file*
>>>> *    module = load_module_py(module_id, path)*
>>>> *  File
>>>> "/Users/asheshvashi/Developments/WorkPlace/pgAdmin4/lib/python2.7/site-packages/alembic/util/compat.py",
>>>> line 75, in load_module_py*
>>>> *    mod = imp.load_source(module_id, path, fp)*
>>>> *  File
>>>> "/Users/asheshvashi/Developments/Projects/pgAdmin4/web/pgadmin/setup/../../migrations/env.py",
>>>> line 85, in <module>*
>>>> *    run_migrations_online()*
>>>> *  File
>>>> "/Users/asheshvashi/Developments/Projects/pgAdmin4/web/pgadmin/setup/../../migrations/env.py",
>>>> line 78, in run_migrations_online*
>>>> *    context.run_migrations()*
>>>> *  File "<string>", line 8, in run_migrations*
>>>> *  File
>>>> "/Users/asheshvashi/Developments/WorkPlace/pgAdmin4/lib/python2.7/site-packages/alembic/runtime/environment.py",
>>>> line 817, in run_migrations*
>>>> *    self.get_context().run_migrations(**kw)*
>>>> *  File
>>>> "/Users/asheshvashi/Developments/WorkPlace/pgAdmin4/lib/python2.7/site-packages/alembic/runtime/migration.py",
>>>> line 323, in run_migrations*
>>>> *    step.migration_fn(**kw)*
>>>> *  File
>>>> "/Users/asheshvashi/Developments/Projects/pgAdmin4/web/migrations/versions/fdc58d9bd449_.py",
>>>> line 84, in upgrade*
>>>> *    email, password = user_info()*
>>>> *  File
>>>> "/Users/asheshvashi/Developments/Projects/pgAdmin4/web/pgadmin/setup/user_info.py",
>>>> line 50, in user_info*
>>>> *    email = input("Email address: ")*
>>>> *  File "<string>", line 1*
>>>> *    ashesh.va...@enterprisedb.com <ashesh.va...@enterprisedb.com>*
>>>> *                ^*
>>>> *SyntaxError: invalid syntax*
>>>>
>>>>
>>>> --
>>>>
>>>> Thanks & Regards,
>>>>
>>>> Ashesh Vashi
>>>> EnterpriseDB INDIA: Enterprise PostgreSQL Company
>>>> <http://www.enterprisedb.com>
>>>>
>>>>
>>>> *http://www.linkedin.com/in/asheshvashi*
>>>> <http://www.linkedin.com/in/asheshvashi>
>>>>
>>>> On Fri, Mar 31, 2017 at 8:17 PM, Murtuza Zabuawala <
>>>> murtuza.zabuaw...@enterprisedb.com> wrote:
>>>>
>>>>> Hi,
>>>>>
>>>>> PFA minor add-on patch for README.
>>>>>
>>>>> --
>>>>> Regards,
>>>>> Murtuza Zabuawala
>>>>> EnterpriseDB: http://www.enterprisedb.com
>>>>> The Enterprise PostgreSQL Company
>>>>>
>>>>> On Fri, Mar 31, 2017 at 8:04 PM, Murtuza Zabuawala <
>>>>> murtuza.zabuaw...@enterprisedb.com> wrote:
>>>>>
>>>>>> Hi Ashesh,
>>>>>>
>>>>>> Patch looks good to me.
>>>>>>
>>>>>> --
>>>>>> Regards,
>>>>>> Murtuza Zabuawala
>>>>>> EnterpriseDB: http://www.enterprisedb.com
>>>>>> The Enterprise PostgreSQL Company
>>>>>>
>>>>>> On Fri, Mar 31, 2017 at 1:10 PM, Ashesh Vashi <
>>>>>> ashesh.va...@enterprisedb.com> wrote:
>>>>>>
>>>>>>> Hi Joao & Sarah,
>>>>>>>
>>>>>>> I have asked Murtuza to review the patch today.
>>>>>>> He will update me by EOD.
>>>>>>>
>>>>>>> If all goes well, I will commit the patch.
>>>>>>>
>>>>>>> --
>>>>>>>
>>>>>>> Thanks & Regards,
>>>>>>>
>>>>>>> Ashesh Vashi
>>>>>>> EnterpriseDB INDIA: Enterprise PostgreSQL Company
>>>>>>> <http://www.enterprisedb.com>
>>>>>>>
>>>>>>>
>>>>>>> *http://www.linkedin.com/in/asheshvashi*
>>>>>>> <http://www.linkedin.com/in/asheshvashi>
>>>>>>>
>>>>>>> On Thu, Mar 30, 2017 at 8:36 PM, Joao Pedro De Almeida Pereira <
>>>>>>> jdealmeidapere...@pivotal.io> wrote:
>>>>>>>
>>>>>>>> Hello Dave and Ashesh,
>>>>>>>>
>>>>>>>> Do you still need us to provide more information about this patch
>>>>>>>> or is it ready to be merged?
>>>>>>>>
>>>>>>>> Thanks
>>>>>>>> Joao
>>>>>>>>
>>>>>>>> On Thu, Mar 23, 2017 at 12:00 PM, Joao Pedro De Almeida Pereira <
>>>>>>>> jdealmeidapere...@pivotal.io> wrote:
>>>>>>>>
>>>>>>>>> Hello Hackers,
>>>>>>>>>
>>>>>>>>> We found out a issue using Python 3 related to importing modules
>>>>>>>>> that we corrected in the patch that is now attached.
>>>>>>>>>
>>>>>>>>> Also we would like to know the status of this.
>>>>>>>>>
>>>>>>>>> Thanks
>>>>>>>>> Joao & Sarah
>>>>>>>>>
>>>>>>>>> On Fri, Mar 17, 2017 at 10:32 AM, Sarah McAlear <
>>>>>>>>> smcal...@pivotal.io> wrote:
>>>>>>>>>
>>>>>>>>>> Hi!
>>>>>>>>>>
>>>>>>>>>> We realized that this change was causing the tests to fail
>>>>>>>>>> because the folder for the sqlite databases was not being created. 
>>>>>>>>>> We also
>>>>>>>>>> updated the files to contain the missing headers.
>>>>>>>>>>
>>>>>>>>>> Thanks!
>>>>>>>>>> Joao & Sarah
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> On Thu, Mar 16, 2017 at 9:31 AM, Dave Page <dp...@pgadmin.org>
>>>>>>>>>> wrote:
>>>>>>>>>>
>>>>>>>>>>> Ashesh, can you review/commit this please? One thing I notice on
>>>>>>>>>>> a
>>>>>>>>>>> quick look through is that the file headers are missing
>>>>>>>>>>> everywhere.
>>>>>>>>>>> They should be present in all source files, except where they
>>>>>>>>>>> would
>>>>>>>>>>> bloat the data transfer from client to server.
>>>>>>>>>>>
>>>>>>>>>>> On Wed, Mar 15, 2017 at 8:09 PM, Sarah McAlear <
>>>>>>>>>>> smcal...@pivotal.io> wrote:
>>>>>>>>>>> > Hi Hackers!
>>>>>>>>>>> >
>>>>>>>>>>> > It looks like our previous patch messed up some logging.
>>>>>>>>>>> Please use this one
>>>>>>>>>>> > instead.
>>>>>>>>>>> >
>>>>>>>>>>> > Thanks,
>>>>>>>>>>> > Joao & Sarah
>>>>>>>>>>> >
>>>>>>>>>>> >
>>>>>>>>>>> >
>>>>>>>>>>> > On Wed, Mar 15, 2017 at 2:46 PM, Sarah McAlear <
>>>>>>>>>>> smcal...@pivotal.io> wrote:
>>>>>>>>>>> >>
>>>>>>>>>>> >> Hi Hackers!
>>>>>>>>>>> >>
>>>>>>>>>>> >> Here's a patch to move to current db migration system to use
>>>>>>>>>>> Alembic.
>>>>>>>>>>> >> Instructions to create new migrations are in the README.
>>>>>>>>>>> >>
>>>>>>>>>>> >> Thanks!
>>>>>>>>>>> >> Joao & Sarah
>>>>>>>>>>> >>
>>>>>>>>>>> >
>>>>>>>>>>> >
>>>>>>>>>>> >
>>>>>>>>>>> > --
>>>>>>>>>>> > Sent via pgadmin-hackers mailing list (
>>>>>>>>>>> pgadmin-hackers@postgresql.org)
>>>>>>>>>>> > To make changes to your subscription:
>>>>>>>>>>> > http://www.postgresql.org/mailpref/pgadmin-hackers
>>>>>>>>>>> >
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> --
>>>>>>>>>>> Dave Page
>>>>>>>>>>> Blog: http://pgsnake.blogspot.com
>>>>>>>>>>> Twitter: @pgsnake
>>>>>>>>>>>
>>>>>>>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>>>>>>>> The Enterprise PostgreSQL Company
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>
>

Reply via email to