Доработан функционал смены ветки #49
Conversation
| ИмяТекущейВетки = Сред(СтрокаФайла, 4, ПозицияРазделителя - 4); | ||
| Если ИмяТекущейВетки <> ИмяВетки Тогда | ||
|
|
||
| Лог.Отладка("Текущая ветка репозитория '" + ИмяТекущейВетки + "', а требуется '" + ИмяВетки + "'"); |
There was a problem hiding this comment.
Предлагаю использовать формат СтрШаблон,
в методы вывода лога я давно добавил возможность использования этой фичи.
например, Лог.Отладка("строка %1", НужнаяСтрока);
очень удобно
| Функция НовыйКомандныйФайлGit(ЛокальныйРепозиторий) | ||
|
|
||
| КомандныйФайл = Новый КомандныйФайл; | ||
| ИмяФайлаЛога = ВременныеФайлы.СоздатьФайл("log"); |
There was a problem hiding this comment.
1commands уже научился получать непосредственный вывод от команд.
Поэтому промежуточный файл лога, с использованием СуффиксПеренаправленияВывода, теперь не нужен.
Пример соответствующего кода есть в методе, выполняющем коммит
Предлагаю доработать и здесь, раз уж начали доработку :)
There was a problem hiding this comment.
ИМХО в этом случае не нужно управление файлом лога и не нужно доп.функции обертки НовыйКомандныйФайлGit, ЗакрытьКомандныйФайлGit
ИсполнитьКомандныйФайлGit вполне можно оставить.
| ИмяФайлаЛога = ВременныеФайлы.СоздатьФайл("log"); | ||
| Если ЭтоWindows Тогда | ||
|
|
||
| КомандныйФайл.ДобавитьКоманду(СтроковыеФункции.ПодставитьПараметрыВСтроку("cd /d ""%1""", ЛокальныйРепозиторий)); |
There was a problem hiding this comment.
Вместо
СтроковыеФункции.ПодставитьПараметрыВСтроку("cd /d ""%1""", ЛокальныйРепозиторий)
предлагаю сразу юзать СтрШаблон, он уже давно поддерживается.
СтрШаблон("cd /d ""%1""", ЛокальныйРепозиторий)
|
|
||
| Иначе | ||
|
|
||
| КомандныйФайл.ДобавитьКоманду(СтроковыеФункции.ПодставитьПараметрыВСтроку("cd ""%1""", ЛокальныйРепозиторий)); |
|
Тесты прошли успешно, но все также где-то в серединете теста git-sync-test происходит смена кодировки и выводятся кракозябры в консоль |
|
@theshadowco Создай ишуз на этот баг, разберемся. |
|
|
||
| // вывод содержит много мусора, поэтому обработаем построчно | ||
| // нужная нам строка начинается с * | ||
| СтрокиФайла = СтрРазделить(КомандныйФайлСтатуса.ПолучитьВывод(), Символы.ПС, ЛОЖЬ); |
There was a problem hiding this comment.
А почему бы регулярки не использовать вместо построчного перебора?
Формат-то все равно един * пробел имяВетки
There was a problem hiding this comment.
Вообще это не самый простой способ получения ветки. В гитраннере есть готовая реализация. Наверное, надо потихоньку переводить на него :)
There was a problem hiding this comment.
There was a problem hiding this comment.
Стоит менять? или в рамках задачи перевода на гитраннер все сделаем?
| ПараметрыКоманды.Добавить("git branch"); | ||
|
|
||
| Результат = ИсполнитьКомандныйФайлGit(КомандныйФайлСтатуса, ПараметрыКоманды); | ||
| Если Результат = 0 Тогда |
There was a problem hiding this comment.
Не обрабатывается неуспешный результат git branch.
Вдруг локальный репо это не репо гит?
| Если ЗначениеЗаполнено(ИмяВетки) Тогда | ||
|
|
||
| // надо проверить текущую ветку, совпдает ли она с требуемой | ||
| КомандныйФайлСтатуса = НовыйКомандныйФайлGit(ЛокальныйРепозиторий); |
There was a problem hiding this comment.
Название КомандныйФайлСтатуса уже не отвечает своему назначения, т.к. вместо гит статус юзаем гит бранч
|
Совсем чуть-чуть осталось :) |
|
В понедельник гляну
24 февр. 2017 г. 17:13 пользователь "Nikita Gryzlov" <
[email protected]> написал:
… ***@***.**** commented on this pull request.
------------------------------
In src/core/Классы/МенеджерСинхронизации.os
<#49 (comment)>
:
> - Иначе
- КомандныйФайл.ДобавитьКоманду(СтроковыеФункции.ПодставитьПараметрыВСтроку("cd ""%1""", ЛокальныйРепозиторий));
+ Если ЗначениеЗаполнено(ИмяВетки) Тогда
+
+ // надо проверить текущую ветку, совпдает ли она с требуемой
+ КомандныйФайлСтатуса = НовыйКомандныйФайлGit(ЛокальныйРепозиторий);
+
+ ПараметрыКоманды = Новый Массив;
+ ПараметрыКоманды.Добавить("git branch");
+
+ Результат = ИсполнитьКомандныйФайлGit(КомандныйФайлСтатуса, ПараметрыКоманды);
+ Если Результат = 0 Тогда
+
+ // вывод содержит много мусора, поэтому обработаем построчно
+ // нужная нам строка начинается с *
+ СтрокиФайла = СтрРазделить(КомандныйФайлСтатуса.ПолучитьВывод(), Символы.ПС, ЛОЖЬ);
Вообще это не самый простой способ получения ветки. В гитраннере есть
готовая реализация. Наверное, надо потихоньку переводить на него :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#49 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AJJbs2l1dk4iSUCzWTFIqw4Gz5wAN4dMks5rfuWbgaJpZM4MInbx>
.
|
… выполнения команды
|
@artbear up |
|
Пока некогда было. |
|
@artbear Удалось посмотреть? |
|
up. Надо бы слить PR |
|
@artbear не останавливайся. PR ждут тебя =) |
|
@theshadowco Еще актуально? :( |
|
@theshadowco В общем, ждем тебя, твой PR остался последний из незакрытых. |
|
@artbear ПР висит с первого марта. Ждал тебя. Потом протух. И тут внезапно @theshadowco должен что то поправлять... Плохая политика. |
|
Ну извините, опенсорс, понимаешь :) |
|
Посмотрю на днях
10 окт. 2017 г. 10:00 PM пользователь "Artur Ayukhanov" <
[email protected]> написал:
… @theshadowco <https://github.com/theshadowco> В общем, ждем тебя, твой PR
остался последний из незакрытых.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#49 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AJJbsxy9TAjPUNdZxBUDrMK4SlUzKVnKks5sq77XgaJpZM4MInbx>
.
|
|
@theshadowco У тебя получится посмотреть/доработать этот ПР ? |
Доработан функционал смены ветки в ВыполнитьGitPull (сообщение #26)