Python + getopt - проблемы с разбором - PullRequest
Новая
       6

Python + getopt - проблемы с разбором

0 голосов
/ 08 ноября 2011

У меня проблемы с написанием некоторых крючков Gerrit http://code.google.com/p/gerrit/.

http://gerrit.googlecode.com/svn/documentation/2.2.0/config-hooks.html

Если я анализирую командную строку для созданный набором патчей --change --change-url --project --branch --uploader --commit --patchset 

def main():

if (len(sys.argv) < 2):
    showUsage()
    exit()

if (sys.argv[1] == 'update-projects'):
    updateProjects()
    exit()

need = ['action=', 'change=', 'change-url=', 'commit=', 'project=', 'branch=', 'uploader=',
        'patchset=', 'abandoner=', 'reason=', 'submitter=', 'comment=', 'CRVW=', 'VRIF=' , 'patchset=' , 'restorer=', 'author=']
print  sys.argv[1:]
print '-----' 
optlist, args = getopt.getopt(sys.argv[1:], '', need)
id = url = hash = who = comment = reason =  codeReview = verified = restorer = ''
print optlist

for o, a in optlist:
    if o == '--change': id = a
    elif o == '--change-url': url = a
    elif o == '--commit': hash = a
    elif o == '--action': what = a
    elif o == '--uploader': who = a
    elif o == '--submitter': who = a
    elif o == '--abandoner': who = a
    elif o == '--author' : who = a
    elif o == '--branch': branch = a
    elif o == '--comment': comment = a
    elif o == '--CRVW' : codeReview = a
    elif o == '--VRIF' : verified = a
    elif o == '--patchset' : patchset = a
    elif o == '--restorer' : who = a
    elif o == '--reason' : reason = a

Ввод командной строки: 

--change I87f7802d438d5640779daa9ac8196aeb3eec8c2a
--change-url http://<hostname>:8080/308
--project private/bar
--branch master
--uploader xxxxxxx-xxxxx xxxxxxx (xxxxxxxxxxxxx.xxxxxxx@xxx-xxxx.xx)
--commit 49aae9befaf27a5fede51b498f0660199f47b899 --patchset 1

print sys.argv [1:] 

['--action', 'new',
'--change','I87f7802d438d5640779daa9ac8196aeb3eec8c2a',
'--change-url',
'http://<hostname>:8080/308',
'--project', 'private/bar',
'--branch', 'master',
'--uploader', 'xxxxxxx-xxxxx', 'xxxxxxx', '(xxxxxxxxxxxxx.xxxxxxx@xxx-xxxx.xx)',
'--commit', '49aae9befaf27a5fede51b498f0660199f47b899',
'--patchset', '1']

печать optlist 

[('--action', 'new'),
('--change', 'I87f7802d438d5640779daa9ac8196aeb3eec8c2a'),
('--change-url', 'http://<hostname>:8080/308'),
('--project', 'private/bar'),
('--branch', 'master'),
('--uploader', 'xxxxxxx-xxxxx')]

Я не знаю, почему скрипт генерирует 

'--uploader', 'xxxxxxx-xxxxx', 'xxxxxxx', '(xxxxxxxxxxxxx.xxxxxxx@xxx-xxxx.xx)'
and not
'--uploader', 'xxxxxxx-xxxxx xxxxxxx (xxxxxxxxxxxxx.xxxxxxx@xxx-xxxx.xx)'

потому что скрипт не анализируется --commit --patchset ...

Когда я анализирую добавленные комментарии, все работает:

Ввод командной строки: 

   -change I87f7802d438d5640779daa9ac8196aeb3eec8c2a
   --change-url http://<hostname>.intra:8080/308
   --project private/bar
   --branch master
   --author xxxxxxx-xxxxx xxxxxxx (xxxxxxxxxxxxx.xxxxxxx@xxx-xxxx.xx)
   --commit 49aae9befaf27a5fede51b498f0660199f47b899
   --comment asdf
   --CRVW 0 
   --VRIF 0

print sys.argv [1:] 

  '--action', 'comment',
    '--change', 'I87f7802d438d5640779daa9ac8196aeb3eec8c2a',
    '--change-url',
    'http://<hostname>:8080/308',
    '--project', 'private/bar',
    '--branch', 'master',
    '--author', 'xxxxxxx-xxxxx xxxxxxx (xxxxxxxxxxxxx.xxxxxxx@xxx-xxxx.xx)', <<< That's right!
    '--commit', '49aae9befaf27a5fede51b498f0660199f47b899',
    '--comment', 'asdf',
    '--CRVW', '0',
    '--VRIF', '0']

Ответы [ 3 ]

2 голосов
/ 08 ноября 2011

Поскольку имена параметров и значения разделены пробелами, вы должны поместить значения в кавычки, если они сами содержат пробелы.

Если вы напишите --uploader xxxxxxx-xxxxx xxxxxxx (xxxxxxxxxxxxx.xxxxxxx@xxx-xxxx.xx), последние две строки будут фактически заканчиваться на args от строки 

optlist, args = getopt.getopt(sys.argv[1:], '', need)

, поскольку они не связаны с --uploader

1 голос
/ 08 ноября 2011

Вы должны заключить аргумент в кавычки, если он содержит пробелы, как для всех инструментов командной строки: 

--uploader "xxxxxxx-xxxxx xxxxxxx (xxxxxxxxxxxxx.xxxxxxx@xxx-xxxx.xx)"
0 голосов
/ 08 ноября 2011

Вы можете также рассмотреть возможность использования gnu_getopt (), так как это позволит вам смешивать аргументы с опциями и без опций. Из документации

Функция getopt () останавливает обработку опций, как только встречается аргумент без опций

Если вы используете gnu_getopt, остальные параметры, а именно commit и pathset, все равно будут правильно проанализированы, даже если в аргументе загрузчика отсутствуют кавычки

...