Fix PSQL Console plugin for OS X

Поиск
Список
Период
Сортировка
От Nicola Vitacolonna
Тема Fix PSQL Console plugin for OS X
Дата
Msg-id 75CDF5B1-6EC5-4553-B8DF-7A28D00FA9C8@gmail.com
обсуждение исходный текст
Ответы Re: Fix PSQL Console plugin for OS X  (Dave Page <dpage@pgadmin.org>)
Список pgadmin-hackers
The PSQL Console command has been broken for a long time on OS X. Could you please fix it?

This is how the command should look like:

Command=/usr/bin/osascript -e 'ignoring application responses' -e 'tell application "Terminal"' -e 'activate' -e 'do
script""' -e 'do script "\'$$PGBINDIR/psql\' --host \'$$HOSTNAME\' --port $$PORT --username \'$$USERNAME\'
\'$$DATABASE\'&& exit || sleep 5 && exit" in window 1' -e 'end tell' -e 'end ignoring' 

For humans:

ignoring application responses
    tell application "Terminal"
        do script "" -- open a new Terminal window
        do script "$$PGBINDIR/psql --host '$$HOSTNAME' --port $$PORT --username '$$USERNAME' '$$DATABASE' && exit ||
sleep5 && exit" in window 1 
        activate
    end tell
end ignoring

The changes are as follows:

1) add `do script “”` to open a new Terminal window. This is necessary because in the current window there may be a
runningprocess. 
2) Remove `with command` (not wrong, but redundant).
3) Add `in window 1`. This is essential, otherwise the command fails.
4) Move `activate` at the end. This change is not strictly necessary, it is just a matter personal preference.

Thanks,
Nicola

PS: Is there a way to contribute to the code directly (à la Github)?


Вложения

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Thom Brown
Дата:
Сообщение: Speed up node animation in pgAdmin4
Следующее
От: Surinder Kumar
Дата:
Сообщение: Re: [PATCH] Enhancement in sql-field control [pgAdmin4]