Refinement of the run function in app.py #148
Conversation
| @@ -73,6 +73,9 @@ def run(self, command, gdb=False, old_binary=False, return_id=True, env=None, | |||
| command = [command[0], *use_backup_dir.pb_args, *command[1:]] | |||
| elif use_backup_dir: | |||
| command = [command[0], *self.backup_dir.pb_args, *command[1:]] | |||
| else: | |||
| use_backup_dir = self.backup_dir.pb_args[2:] | |||
| command = [command[0], *use_backup_dir, *command[1:]] | |||
There was a problem hiding this comment.
Тут не
command = [command[0], *self.backup_dir.pb_args, *command[1:]] ?
There was a problem hiding this comment.
@demonolock,
Если у нас use_backup_dir=true, то строчка действительно так будет выглядеть.
Но если use_backup_dir=false, то значит, что мы сами передаем путь к backup_dir (флаг и путь будет у нас находиться в command). А первые 2 аргумента в self.backup_dir.pb_args это -В и путь. В итоге, будет выдаваться ошибка, что мы дважды указали -В. Поэтому первые 2 значения я пропускаю и добавляю только доп флаги, если они есть.
There was a problem hiding this comment.
Скорее, command = [command[0], *self.backup_dir.pb_args[2:], *command[1:]]
There was a problem hiding this comment.
@MetalDream666 ,
Да, действительно. Так лучше. Поправил.
MakSl
force-pushed
the
refinement_run_function
branch
from
9366283 to
2d2a156
Compare
No description provided.