OberonCore

Библиотека  Wiki  Форум  BlackBox  Компоненты  Проекты
Текущее время: Вторник, 23 Октябрь, 2018 12:17

Часовой пояс: UTC + 3 часа




Начать новую тему Ответить на тему  [ Сообщений: 36 ]  На страницу Пред.  1, 2
Автор Сообщение
СообщениеДобавлено: Воскресенье, 18 Май, 2014 20:26 
Аватара пользователя

Зарегистрирован: Пятница, 25 Сентябрь, 2009 13:10
Сообщения: 1162
Откуда: Tel-Aviv
ilovb писал(а):
Самое забавное, что ошибка найдена благодаря подсветке питона в ST3. Кто там говорил, что подсветка зло? :lol:
Тс-с-с! Чтоб не накликать...


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Воскресенье, 18 Май, 2014 20:40 

Зарегистрирован: Вторник, 30 Июнь, 2009 14:58
Сообщения: 1438
Роман М. писал(а):
Не понял зачем нужен 5-й пункт.


Это потому что make мы делаем в папке репа, а не питона. А первый раз копировали чтобы нашло все модули


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Воскресенье, 18 Май, 2014 20:41 

Зарегистрирован: Вторник, 30 Июнь, 2009 14:58
Сообщения: 1438
Ну теперь нужно протестить на как можно большем количестве исходников. И проверить что мы не поломали плюсы


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Воскресенье, 18 Май, 2014 21:20 
Аватара пользователя

Зарегистрирован: Пятница, 25 Сентябрь, 2009 13:10
Сообщения: 1162
Откуда: Tel-Aviv
"make test" падает, причём без правок КП.
Код:
Pygments 2.0pre test suite running (Python 2.7.3)...
......................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................F.............................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................
======================================================================
FAIL: test_basic_api.test_lexer_options(<class 'pygments.lexers.functional.LiterateCryptolLexer'>,)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/romiras/Projects/pygments/tests/test_basic_api.py", line 106, in verify
    ensure(inst.get_tokens('a\nb'), 'a\nb')
  File "/home/romiras/Projects/pygments/tests/test_basic_api.py", line 90, in ensure
    '%s: %r != %r' % (lexer, concatenated, output)
AssertionError: <class 'pygments.lexers.functional.LiterateCryptolLexer'>: u'a\n' != 'a\nb'

----------------------------------------------------------------------
Ran 1108 tests in 23.070s

FAILED (failures=1)
make: *** [test] Error 1


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Воскресенье, 18 Май, 2014 21:37 

Зарегистрирован: Вторник, 30 Июнь, 2009 14:58
Сообщения: 1438
Ну так это ж не релиз, а текущее состояние хранилища. Ничего удивительного.

Нам нужно только своё тестить.


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Воскресенье, 18 Май, 2014 23:19 
Аватара пользователя

Зарегистрирован: Пятница, 25 Сентябрь, 2009 13:10
Сообщения: 1162
Откуда: Tel-Aviv
Роман М. писал(а):
"make test" падает, причём без правок КП.


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Воскресенье, 18 Май, 2014 23:25 

Зарегистрирован: Вторник, 30 Июнь, 2009 14:58
Сообщения: 1438
А зачем тебе make test?

Нам надо натравить эту штуку в цикле на исходники ББ и смотреть на результат. Все.


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Вторник, 20 Май, 2014 14:23 
Аватара пользователя

Зарегистрирован: Пятница, 25 Сентябрь, 2009 13:10
Сообщения: 1162
Откуда: Tel-Aviv
Если смотреть на правку для КП в отрыве от других языков программирования, то автор Pygments не примет нашу правку и тогда поддержки КП в GitHub не видать как своих ушей.

"make test" нужен для того, чтобы убедиться в работе всех лексеров.

Цитата:
Once the example renders as expected, you should run the complete test suite:
Код:
$ make test

Adding and testing a new lexer


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Вторник, 20 Май, 2014 16:21 

Зарегистрирован: Вторник, 30 Июнь, 2009 14:58
Сообщения: 1438
Так ты же сам сказал что и без нашего лексера тест не проходит.
Надо значит взять стабильный релиз и на нем тестить сначала.


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Вторник, 20 Май, 2014 17:28 
Аватара пользователя

Зарегистрирован: Пятница, 25 Сентябрь, 2009 13:10
Сообщения: 1162
Откуда: Tel-Aviv
Да, можно. Попробуй откатить на более старую версию и проверить заново. У меня пока руки не дошли.


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Среда, 21 Май, 2014 12:09 
Аватара пользователя

Зарегистрирован: Пятница, 25 Сентябрь, 2009 13:10
Сообщения: 1162
Откуда: Tel-Aviv
О темпе внедрения новых лексеров можно судить по количеству открытых запросов в birkenfeld/pygments-main.
Судя по-всему, подсветку КП можно увидеть не скоро.

Уточнение: в авторской версии тег 1.6 тесты с правкой КП проходят полностью.


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Среда, 21 Май, 2014 12:40 

Зарегистрирован: Вторник, 30 Июнь, 2009 14:58
Сообщения: 1438
Роман М. писал(а):
в авторской версии тег 1.6 тесты с правкой КП проходят полностью.


Отлично. :) А то я вчера так и не нашел времени на тестинг.


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Воскресенье, 13 Сентябрь, 2015 19:34 
Аватара пользователя

Зарегистрирован: Пятница, 25 Сентябрь, 2009 13:10
Сообщения: 1162
Откуда: Tel-Aviv
Роман М. писал(а):
О темпе внедрения новых лексеров можно судить по количеству открытых запросов в birkenfeld/pygments-main.
Судя по-всему, подсветку КП можно увидеть не скоро.

Уточнение: в авторской версии тег 1.6 тесты с правкой КП проходят полностью.

Минул год, а воз и ныне там...
Мой PR с моими минорными изменениями от версии Бориса так и остался незамеченным. С тех пор изменилась структура лексеров в pygments - стала модульной.
Переработал исходники и залил другую версию для PR: diff
Жду чуда.


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Четверг, 15 Октябрь, 2015 00:46 
Аватара пользователя

Зарегистрирован: Пятница, 25 Сентябрь, 2009 13:10
Сообщения: 1162
Откуда: Tel-Aviv
Роман М. писал(а):
Роман М. писал(а):
О темпе внедрения новых лексеров можно судить по количеству открытых запросов в birkenfeld/pygments-main.
Судя по-всему, подсветку КП можно увидеть не скоро.

Уточнение: в авторской версии тег 1.6 тесты с правкой КП проходят полностью.

Минул год, а воз и ныне там...
Мой PR с моими минорными изменениями от версии Бориса так и остался незамеченным. С тех пор изменилась структура лексеров в pygments - стала модульной.
Переработал исходники и залил другую версию для PR: diff
Жду чуда.

Чудо свершилось!
Pull Request "Add Component Pascal lexer of Oberon family" был влит в основную ветку pygments-main.


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Четверг, 15 Октябрь, 2015 11:22 

Зарегистрирован: Вторник, 30 Июнь, 2009 14:58
Сообщения: 1438
Роман, большое спасибо за усилия.


Вернуться к началу
 Профиль  
 
СообщениеДобавлено: Четверг, 15 Октябрь, 2015 16:37 
Аватара пользователя

Зарегистрирован: Пятница, 25 Сентябрь, 2009 13:10
Сообщения: 1162
Откуда: Tel-Aviv
ilovb писал(а):
Роман, большое спасибо за усилия.

Нужно признать, основные усилия были таки всё-таки твоими. Я же убрал лишнее, скомпоновав по примеру других лексеров. Ну и, первый блин был комом.

Со второй попытки же, спустя год, получилось.
Теперь лексер находится в pygments-main / pygments / lexers / oberon.py
куда потенциально можно добавить поддержку других языков семейства Оберона.


Вернуться к началу
 Профиль  
 
Показать сообщения за:  Поле сортировки  
Начать новую тему Ответить на тему  [ Сообщений: 36 ]  На страницу Пред.  1, 2

Часовой пояс: UTC + 3 часа


Кто сейчас на конференции

Сейчас этот форум просматривают: нет зарегистрированных пользователей и гости: 2


Вы не можете начинать темы
Вы не можете отвечать на сообщения
Вы не можете редактировать свои сообщения
Вы не можете удалять свои сообщения
Вы не можете добавлять вложения

Найти:
Вся информация, размещаемая участниками на конференции (тексты сообщений, вложения и пр.) © 2005-2018, участники конференции «OberonCore», если специально не оговорено иное.
Администрация не несет ответственности за мнения, стиль и достоверность высказываний участников, равно как и за безопасность материалов, предоставляемых участниками во вложениях.
Без разрешения участников и ссылки на конференцию «OberonCore» любое воспроизведение и/или копирование высказываний полностью и/или по частям запрещено.
Powered by phpBB® Forum Software © phpBB Group
Русская поддержка phpBB