Skip to content

Commit

Permalink
Command error moved to stderr, readme fix
Browse files Browse the repository at this point in the history
  • Loading branch information
popstas committed Oct 3, 2015
1 parent 785328c commit 07cdfa8
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 7 deletions.
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -91,9 +91,9 @@ Options:
`drall -m '-drs vget update_modified' -m visitkaplus` - получить все визитки+ не отключенные от апдейтов
`drall -v -m all_complete -m clean_all` - получить все готовые не измененные сайты
`drall --asroot -m snormal -m clean_css -v 'cp-chown /home/from/www/from.example.com $PWD sites/all/themes/theme/css/style.css'` - скопировать style.css из from.example.com в каждый неизмененный snormal
`drall 'request="/contacts"; page="http://$(basename $PWD)$request"; exists=$(curl -s --head "$page" | grep -c 200); if [ "$exists" = 1 ]; then echo $page; fi' -q | grep -v "^$"` - вывести все страницы /contacts
`export page="/contacts"; drall --asroot -q 'url="http://$(get_domain)$page"; curl -sI "$url" | grep "200 OK" > /dev/null && echo "$url"' 2>/dev/null | grep -v "^$"` - вывести все страницы /contacts


## Известные проблемы:
## Баги и фичи:
1. Команду нужно заключать в кавычки, фактически, команда должна быть одним параметром скрипта
2. Если в папке тестов match-tests будет несколько одноименных тестов, команда скорее всего упадет
3. Нужно различать одинарные и двойные кавычки, чаще всего команду надо заключать в одинарные кавычки. См. раздел "Команда"
12 changes: 8 additions & 4 deletions bin/drall
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@
# 23.03.2015 - --match-test, --name, removed required command
# 06.05.2015 - catch matchers exit codes
# 31.08.2015 - installer
# v 0.4.1
# see git log
# v 0.4.3

import optparse
from optparse import Option
Expand Down Expand Up @@ -169,19 +170,22 @@ class Drall:
sh.cd(site_root)

if self.options.asroot:
command_result = ''

# command_result = sh.Command('su -c "%s" %s' % (command_file_path, site_user)).run()

try:
command_result = sh.Command(command_file_path).run()
command_result = sh.Command(command_file_path).run()
except sh.ErrorReturnCode:
command_result = '# error executing script\n'
sys.stderr.write('# error while executing script\n')

# command_result = subprocess.call(command_file_path)

if not self.options.quiet:
print site_root
print command_result

if command_result != '':
print command_result
else:

# su
Expand Down

0 comments on commit 07cdfa8

Please sign in to comment.