Skip to content

Внесены исправления для корректной работы синхронизации списка репов #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

Merged
merged 5 commits into from
Apr 20, 2017

Conversation

theshadowco
Copy link
Contributor

Исправление бага #47

@theshadowco
Copy link
Contributor Author

Реверснул последний коммит. Неправильно понял ошибку

@theshadowco
Copy link
Contributor Author

@AlexStrizh Можешь сказать, как запускаешь, ибо ошибки 1 у меня нету?

@astrizhachuk
Copy link
Contributor

astrizhachuk commented Feb 21, 2017

На последнем релизе (после opm update -all) внес изменения в файлах из твоего коммита. Запускаю также как и при инициализации ошибки #47 gitsync all "путь к xml".

@theshadowco
Copy link
Contributor Author

O:\reps\fork\gitsync\src>oscript gitsync.os all ..\config-example.xml

GitSync v1.2.2
ИНФОРМАЦИЯ - Синхронизация началась
ИНФОРМАЦИЯ - Проверяю необходимость синхронизации: test
21.02.2017 14:59:49 Ошибка открытия файла базы (файл открыт другой программой?)

ОШИБКА - {Модуль o:\reps\libs\os-library\src\tool1cd\table-reader.os / Ошибка в строке: 46 / Tool_1CD вернул код: 2}
ИНФОРМАЦИЯ - Проверяю необходимость синхронизации: test2
21.02.2017 14:59:49 Ошибка открытия файла базы (файл открыт другой программой?)

ОШИБКА - {Модуль o:\reps\libs\os-library\src\tool1cd\table-reader.os / Ошибка в строке: 46 / Tool_1CD вернул код: 2}
ИНФОРМАЦИЯ - Синхронизация завершена

@theshadowco
Copy link
Contributor Author

мож также сделать?

@astrizhachuk
Copy link
Contributor

21.02.2017 15:12:37: ОШИБКА - {Модуль C:\Program Files (x86)\OneScript\lib\gitsync\src\multi-controller.os / Ошибка в строке: 64 / Метод объекта не обнаружен (ТребуетсяСинхронизироватьХранилище)}
21.02.2017 15:12:37: ИНФОРМАЦИЯ - Проверяю необходимость синхронизации: test2
21.02.2017 15:12:37: ОШИБКА - {Модуль C:\Program Files (x86)\OneScript\lib\gitsync\src\multi-controller.os / Ошибка в строке: 64 / Метод объекта не обнаружен (ТребуетсяСинхронизироватьХранилище)}
21.02.2017 15:12:37: ИНФОРМАЦИЯ - Синхронизация завершена

У тебя старая версия gitsync, у меня 2.1

@astrizhachuk
Copy link
Contributor

До обновления была ошибка как у тебя.

@theshadowco
Copy link
Contributor Author

@AlexStrizh Изменения из данного реквеста у тебя отуствуют

@theshadowco
Copy link
Contributor Author

@AlexStrizh сделал тестовый конфиг.
имею

O:\reps\fork\gitsync\src>oscript gitsync.os all ..\config-example.xml
GitSync v1.2.2
ИНФОРМАЦИЯ - Синхронизация началась
ИНФОРМАЦИЯ - Проверяю необходимость синхронизации: test
ИНФОРМАЦИЯ - Номер синхронизированной версии: 0
ИНФОРМАЦИЯ - Номер последней версии в хранилище: 395
ИНФОРМАЦИЯ - Синхронизация для 'test'
ИНФОРМАЦИЯ - Начинаю синхронизацию хранилища 1С и репозитария GIT
ИНФОРМАЦИЯ - Получение изменений с удаленного узла (pull)

^C

@astrizhachuk
Copy link
Contributor

@theshadowco ага, заново установлено было.

@astrizhachuk
Copy link
Contributor

ну а у меня
21.02.2017 15:16:43: ИНФОРМАЦИЯ - Проверяю необходимость синхронизации: test
21.02.2017 15:16:43: ОШИБКА - {Модуль C:\Program Files (x86)\OneScript\lib\gitsync\src\multi-controller.os / Ошибка в строке: 64 / Метод объекта не обнаружен (ТребуетсяСинхронизироватьХранилище)}

Это на чистом обновлении без правок.

@theshadowco
Copy link
Contributor Author

@AlexStrizh на чистом правильно, но с моими исправлениями должно работать.

@astrizhachuk
Copy link
Contributor

@theshadowco ну хз, после изменений вот так вот
21.02.2017 15:22:18: ОШИБКА - {Модуль C:\Program Files (x86)\OneScript\lib\tool1cd\table-reader.os / Ошибка в строке: 46 / Tool_1CD вернул код: 2}

@theshadowco
Copy link
Contributor Author

@AlexStrizh запусти с ключем -debug. перед ошибкой выдаст строку запуска tool_1cd.

@astrizhachuk
Copy link
Contributor

@theshadowco что конкретно с ключом -debug запустить то? Разве у gitsync есть такой ключ?

@theshadowco
Copy link
Contributor Author

gitsync.os all мойконфиг.xml -debug on

@astrizhachuk
Copy link
Contributor

astrizhachuk commented Feb 21, 2017

file: 'file:///c%3A/Program%20Files%20%28x86%29/OneScript/lib/gitsync/src/gitsync.os'
severity: 'Ошибка'
message: 'Внешнее исключение (System.NullReferenceException): Ссылка на объект не указывает на экземпляр объекта.'
at: '65,1'
source: ''

ИначеЕсли ЗначениеЗаполнено(ПараметрыСистемы.ИмяКомандыПоУмолчанию()) Тогда

@nixel2007
Copy link
Member

file: 'file:///c%3A/Program%20Files%20%28x86%29/OneScript/lib/gitsync/src/gitsync.os'
severity: 'Ошибка'
message: 'Внешнее исключение (System.NullReferenceException): Ссылка на объект не указывает на экземпляр объекта.'
at: '65,1'
source: ''

если я правильно понимаю, это исключение внутри VSC, и оно не относится к данной проблеме.

@theshadowco
Copy link
Contributor Author

@nixel2007 похоже, ибо я собрал тестовый конфиг и все отработало

@astrizhachuk
Copy link
Contributor

Какая у вас версия?

@theshadowco
Copy link
Contributor Author

VSC 1.9.1
1Script Execution Engine. Version 1.0.15.260

@khorevaa
Copy link
Member

Это не внутренне исключение.
Проблема в этой строчке
ИначеЕсли ЗначениеЗаполнено(ПараметрыСистемы.ИмяКомандыПоУмолчанию()) Тогда

Попробуйте удалить полнистью папку gitsync и установить заново.

@nixel2007
Copy link
Member

Ошибка работы линтера на старших версиях OneScript была зарегистрирована в самом репе OneScript.

@astrizhachuk
Copy link
Contributor

Те же версии. Я уже задавал вопрос, может быть связано с разрядностью машины? Вроде tool1cd зависит от разрядности (я этого точно не знаю)

@theshadowco
Copy link
Contributor Author

win 7 x64

@astrizhachuk
Copy link
Contributor

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

@astrizhachuk
Copy link
Contributor

Без изменений. Меня смущает, что у нас разные версии GitSync, у меня 2.1, у тебя 1.2.2

@theshadowco
Copy link
Contributor Author

O:\reps\fork\gitsync\src>oscript gitsync.os all ..\config-example.xml
GitSync v2.1
ИНФОРМАЦИЯ - Синхронизация началась
ИНФОРМАЦИЯ - Проверяю необходимость синхронизации: test
ИНФОРМАЦИЯ - Номер синхронизированной версии: 0
ИНФОРМАЦИЯ - Номер последней версии в хранилище: 395
ИНФОРМАЦИЯ - Синхронизация для 'test'
ИНФОРМАЦИЯ - Начинаю синхронизацию хранилища 1С и репозитария GIT
ИНФОРМАЦИЯ - Получение изменений с удаленного узла (pull)
ИНФОРМАЦИЯ - From <>

  • branch HEAD -> FETCH_HEAD
    Already up-to-date.

ИНФОРМАЦИЯ - Синхронизация изменений с хранилищем
ИНФОРМАЦИЯ - Начало синхронизации с git
ИНФОРМАЦИЯ - Номер синхронизированной версии: 0
ИНФОРМАЦИЯ - Номер последней версии в хранилище: 395
ИНФОРМАЦИЯ - Получаем исходники для версии 1
^C

@astrizhachuk
Copy link
Contributor

Взял твою сборку, ошибка с tool1cd. Однако без параметра all все работает.

@theshadowco
Copy link
Contributor Author

@artbear нид хелп

@theshadowco
Copy link
Contributor Author

@AlexStrizh Получилось с моим патчем запуститься?

@EvilBeaver
Copy link
Member

Парни, там такая длинная переписка, что я не в курсе дел. Что с этим PR делать?

@theshadowco
Copy link
Contributor Author

@EvilBeaver если кратко - этот реквест исправлет ошибку, возникающую при комманде ALL. Ошибку сгенерил я, когда переделал структуру файлов гитсинка (тест для команды ALL не очень качественный)

@theshadowco
Copy link
Contributor Author

Если командой ALL пользуются, то это исправление блокера

@EvilBeaver
Copy link
Member

@artbear у меня замечаний нет, подтверждаешь вливание?

@theshadowco
Copy link
Contributor Author

@artbear Удалось посмотреть?

@artbear
Copy link
Member

artbear commented Mar 15, 2017

@theshadowco я болел, не удалось посмотреть.
только сегодня выхожу на нормальный ритм

@EvilBeaver EvilBeaver merged commit 0ca5e6a into oscript-library:develop Apr 20, 2017
@artbear artbear modified the milestone: 2.2 Oct 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants