-
Notifications
You must be signed in to change notification settings - Fork 93
Внесены исправления для корректной работы синхронизации списка репов #48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The head ref may contain hidden characters: "feature/\u0418\u0441\u043F\u0440\u0430\u0432\u043B\u0435\u043D\u0438\u0435\u041E\u0448\u0438\u0431\u043A\u0438All"
Внесены исправления для корректной работы синхронизации списка репов #48
Conversation
Реверснул последний коммит. Неправильно понял ошибку |
@AlexStrizh Можешь сказать, как запускаешь, ибо ошибки 1 у меня нету? |
На последнем релизе (после opm update -all) внес изменения в файлах из твоего коммита. Запускаю также как и при инициализации ошибки #47 gitsync all "путь к xml". |
O:\reps\fork\gitsync\src>oscript gitsync.os all ..\config-example.xml GitSync v1.2.2 ОШИБКА - {Модуль o:\reps\libs\os-library\src\tool1cd\table-reader.os / Ошибка в строке: 46 / Tool_1CD вернул код: 2} ОШИБКА - {Модуль o:\reps\libs\os-library\src\tool1cd\table-reader.os / Ошибка в строке: 46 / Tool_1CD вернул код: 2} |
мож также сделать? |
21.02.2017 15:12:37: ОШИБКА - {Модуль C:\Program Files (x86)\OneScript\lib\gitsync\src\multi-controller.os / Ошибка в строке: 64 / Метод объекта не обнаружен (ТребуетсяСинхронизироватьХранилище)} У тебя старая версия gitsync, у меня 2.1 |
До обновления была ошибка как у тебя. |
@AlexStrizh Изменения из данного реквеста у тебя отуствуют |
@AlexStrizh сделал тестовый конфиг. O:\reps\fork\gitsync\src>oscript gitsync.os all ..\config-example.xml ^C |
@theshadowco ага, заново установлено было. |
ну а у меня Это на чистом обновлении без правок. |
@AlexStrizh на чистом правильно, но с моими исправлениями должно работать. |
@theshadowco ну хз, после изменений вот так вот |
@AlexStrizh запусти с ключем -debug. перед ошибкой выдаст строку запуска tool_1cd. |
@theshadowco что конкретно с ключом -debug запустить то? Разве у gitsync есть такой ключ? |
gitsync.os all мойконфиг.xml -debug on |
file: 'file:///c%3A/Program%20Files%20%28x86%29/OneScript/lib/gitsync/src/gitsync.os' ИначеЕсли ЗначениеЗаполнено(ПараметрыСистемы.ИмяКомандыПоУмолчанию()) Тогда |
если я правильно понимаю, это исключение внутри VSC, и оно не относится к данной проблеме. |
@nixel2007 похоже, ибо я собрал тестовый конфиг и все отработало |
Какая у вас версия? |
VSC 1.9.1 |
Это не внутренне исключение. Попробуйте удалить полнистью папку gitsync и установить заново. |
Ошибка работы линтера на старших версиях OneScript была зарегистрирована в самом репе OneScript. |
Те же версии. Я уже задавал вопрос, может быть связано с разрядностью машины? Вроде tool1cd зависит от разрядности (я этого точно не знаю) |
win 7 x64 |
Полностью идентичны, попробую заново все поставить. |
Без изменений. Меня смущает, что у нас разные версии GitSync, у меня 2.1, у тебя 1.2.2 |
O:\reps\fork\gitsync\src>oscript gitsync.os all ..\config-example.xml
ИНФОРМАЦИЯ - Синхронизация изменений с хранилищем |
Взял твою сборку, ошибка с tool1cd. Однако без параметра all все работает. |
@artbear нид хелп |
@AlexStrizh Получилось с моим патчем запуститься? |
Парни, там такая длинная переписка, что я не в курсе дел. Что с этим PR делать? |
@EvilBeaver если кратко - этот реквест исправлет ошибку, возникающую при комманде ALL. Ошибку сгенерил я, когда переделал структуру файлов гитсинка (тест для команды ALL не очень качественный) |
Если командой ALL пользуются, то это исправление блокера |
@artbear у меня замечаний нет, подтверждаешь вливание? |
@artbear Удалось посмотреть? |
@theshadowco я болел, не удалось посмотреть. |
Исправление бага #47