Bug 14519

Summary: Ошибки импорта/экспорта Sketch в Inkscape
Product: ПСПО Reporter: Andrey Cherepanov <cas>
Component: bugsAssignee: Alexey Rusakov <ktirf>
Status: REOPENED --- QA Contact: qa-sisyphus
Severity: normal    
Priority: P2 CC: aen, alexandre.prokoudine, cas, ktirf, lav, ruslandh, vvk
Version: 20080311   
Hardware: all   
OS: Linux   
Bug Depends on: 15404    
Bug Blocks: 15075    
Attachments:
Description Flags
Патч для skencil none

Description Andrey Cherepanov 2008-02-18 19:09:25 MSK
В Inkscape требует для поддержки импорта из Sketch пакет skencil, вылетает с 
ошибкой при импорте из sketch и не поддерживает экспорт в sketch.
Comment 1 ruslandh 2008-03-19 19:17:47 MSK
Посмотрю
Comment 2 Andrey Cherepanov 2008-04-10 17:24:45 MSD
Перенос на конкретный пакет
Comment 3 Michael Shigorin 2008-04-11 15:53:38 MSD
Бага как бага, critical -- это если б он на старте ложился.
Comment 4 AEN 2008-04-12 12:45:55 MSD
(In reply to comment #3)
> Бага как бага, critical -- это если б он на старте ложился.

Это critical, так как не соответстует условиям КД.
Comment 5 Michael Shigorin 2008-04-12 16:59:09 MSD
(In reply to comment #4)
> Это critical, так как не соответстует условиям КД.
(посмотрев в Product) к Sisyphus это каким боком?  Задача для core dev/QA ООО
или контрактная, а не майнтейнерская.  Разве если packaging bug, передать в
апстрим или майнтейнер участвует в разработке, но тогда нет слова "срок", что
вряд ли приемлемо.
Comment 6 Andrey Rahmatullin 2008-04-12 17:01:04 MSD
+1
Comment 7 Mikhail Gusarov 2008-04-12 19:23:40 MSD
-> cas@
Comment 8 AEN 2008-04-12 19:29:09 MSD
Перевесил на Школьный и на ktirf@, critical относится к ним.
Comment 9 Alexey Rusakov 2008-04-12 19:51:04 MSD
Господа, не мутите воду, багами на Школьный комплект занимается Руслан. Я могу
быть только в CC, а данный конкретный баг вообще может исправить только sbolshakov@.
Comment 10 AEN 2008-04-12 20:33:30 MSD
(In reply to comment #9)
> Господа, не мутите воду, багами на Школьный комплект занимается Руслан. Я могу
> быть только в CC, а данный конкретный баг вообще может исправить только
sbolshakov@.

Ok. Вы начальник, Вам виднее на кого вешать.
Comment 11 Sergey Bolshakov 2008-04-12 21:27:32 MSD
пакет skencil включается в школьный комплект с 20080311, если не раньше.
если баг заключался в его отсутствии в более ранних версиях, то fixed.
Comment 12 Andrey Cherepanov 2008-04-14 16:42:56 MSD
Не поддерживается импорт (ошибка конвертации в скрипте) и экспорт (вообще нет 
в списке форматов) Sketch.
Comment 13 Sergey Bolshakov 2008-04-17 19:19:57 MSD
удалось выяснить, в каком пакете ошибка ?
Comment 14 Alexandre Prokoudine 2008-04-17 20:56:24 MSD
Если формата нет в списке, значит skencil не установлен, или установлен, но не
найден.
Comment 15 ruslandh 2008-04-17 23:37:03 MSD
Не залезая в код Inskape, а только ориетируясь пока на Google для импорта 
файлов *.sk и *.sk1 нужен uniconvertor 
http://wiki.inkscape.org/wiki/index.php/Tools#uniconvertor. 

Sketch создаёт эти файлы, но не участвует в импорте Inkscape. 

Сам uniconvertor может конвертировать различные файлы и сам, но из командной 
строки. 

ps смотрю дальше
Comment 16 ruslandh 2008-04-18 00:03:35 MSD
Извиняюсь - у нас участвует через sckonvert, входящий в пакет skencil
Comment 17 Andrey Cherepanov 2008-04-18 15:34:35 MSD
(In reply to comment #15)
> Не залезая в код Inskape, а только ориетируясь пока на Google для импорта 
> файлов *.sk и *.sk1 нужен uniconvertor 
> http://wiki.inkscape.org/wiki/index.php/Tools#uniconvertor. 
Прокудин тоже говорил, что нужно его использовать. 
Осталось сбэкпортировать его на school-4.0 (или в branch-4.0) и обеспечить 
интеграцию с Inkscape.
Comment 18 Michael Shigorin 2008-04-18 15:58:06 MSD
Там этот *&^&*^&* питон, будь он не ладен.  "Последний".
Comment 19 Alexandre Prokoudine 2008-04-18 18:58:59 MSD
Мишенька, UC в обозримом будущем будет must have для импорта чужих файлов
вообще, нравится тебе это или нет. Причём, он будет нужен не только Inskcape, но
и Scribus. И, кстати, для импорта не столько SK, сколько CDR и прочей
проприетарной фигни. Так что нравится это тебе или нет, но тебе придётся с этим
жить.

Да, текущая версия UC лажает с импортом SK. Это уже исправлено в транке sK1 и
будет перетащено в UC при первой возможности.
Comment 20 Michael Shigorin 2008-04-18 19:22:57 MSD
(In reply to comment #19)
> Мишенька, UC в обозримом будущем будет must have для импорта чужих файлов
> вообще, нравится тебе это или нет.
Слушай, я почти пошёл его в сизиф собирать в своё время благодаря frob :)

Одна маленькая проблема, которая быстро вылезла (сразу после необходимости
обновления/перелопачивания ряда модулей) -- python-2.5, на который автор успел
спрыгнуть, пока разбирались с первой частью.

Для мастхэвов противопоказано гоняться за последними пе... питонами.  Увы, факт.
Comment 21 AEN 2008-04-18 20:34:53 MSD
(In reply to comment #19)
> Мишенька, UC в обозримом будущем будет must have для импорта чужих файлов
> вообще, нравится тебе это или нет. 

Нам это нравится, но школьный комплект живет в обозримом прошлом, мы не можем
выпустить его на свежем бранче.
Comment 22 ruslandh 2008-04-19 18:27:39 MSD
> удалось выяснить, в каком пакете ошибка ?
Ошибка в скрипте skconvert.py, входящем в skencil.

error in line 19
Traceback (most recent call last):
  File "/usr/lib/skencil-0.6.17/Plugins/Filters/skloader.py", line 469, in Load
    funcname, args, kwargs = parse(line)
SyntaxError: unexpected character
Traceback (most recent call last):
  File "/usr/bin/skconvert", line 55, in ?
    main()
  File "/usr/bin/skconvert", line 52, in main
    convert(sys.argv[1], sys.argv[2])
  File "/usr/bin/skconvert", line 35, in convert
    doc = load.load_drawing(infile)
  File "/usr/lib/skencil-0.6.17/Sketch/Base/load.py", line 368, in load_drawing
    return load_drawing_from_file(file, filename)
  File "/usr/lib/skencil-0.6.17/Sketch/Base/load.py", line 343, in 
load_drawing_
from_file
    doc = loader.Load()
  File "/usr/lib/skencil-0.6.17/Plugins/Filters/skloader.py", line 506, in Load
    raise SketchLoadError('%d:%s' % (num, value))
Sketch.skexceptions.SketchLoadError: 19:unexpected character
Comment 23 ruslandh 2008-04-19 21:30:16 MSD
На сегодня можно предложить хак для Inkscape

- в скрипте /usr/share/inkscape/extensions/sk2svg.sh 
строчку 
skconvert "$1" "$TMPSVG" > /dev/null 2>&1 || rc=1
заменить на 
LC_ALL=C skconvert "$1" "$TMPSVG" > /dev/null 2>&1 || rc=1
Comment 24 ruslandh 2008-04-19 22:09:18 MSD
А вот и подтверждение :)
http://wald.intevation.org/tracker/index.php?func=detail&aid=345&group_id=5&atid=101
Comment 25 AEN 2008-04-19 22:12:29 MSD
(In reply to comment #23)
> На сегодня можно предложить хак для Inkscape
> 
> - в скрипте /usr/share/inkscape/extensions/sk2svg.sh 
> строчку 
> skconvert "$1" "$TMPSVG" > /dev/null 2>&1 || rc=1
> заменить на 
> LC_ALL=C skconvert "$1" "$TMPSVG" > /dev/null 2>&1 || rc=1
>

Почему же это хак? Наверное, будет корректнее LC_NUMERIC=C
 
Comment 26 ruslandh 2008-04-19 23:40:42 MSD
Created attachment 2579 [details]
Патч для skencil

Аот это уже в меньшей мере хак :)
Comment 27 ruslandh 2008-04-19 23:45:23 MSD
>Почему же это хак? Наверное, будет корректнее LC_NUMERIC=C
 
Потому, что менять надо в скриптах sketcil, а не в вызывающих скриптах :)

В приложенном выше хаке, на всякий случай изменил LC_NUMERIC в аналогичных 
скриптах :

sk2ps.py и sk2ppm.py, но я из не проверил.
Comment 28 ruslandh 2008-06-13 14:27:37 MSD
Исправленно
Comment 29 Andrey Cherepanov 2008-06-16 12:33:05 MSD
Да? И сохранение в Sketch работает?
Comment 30 Alexandre Prokoudine 2008-06-16 14:58:25 MSD
Выдать расширение, добавляющее экспорт в скетч при наличии юниконвертора в системе?
Comment 31 Michael Shigorin 2008-06-21 15:52:41 MSD
,----------------------------------------------------------------------------.
| Выдать расширение, добавляющее экспорт в скетч при наличии юниконвертора в |
| системе?                                                                   |
|                                                                            |
|                           [ Зажать ]  | Выдать |                           |
|                                                                            |
`----------------------------------------------------------------------------'
Comment 32 Alexandre Prokoudine 2008-06-21 18:43:58 MSD
Ну раз ты, Майк, такой шутник, собирай юниконвертор :)
Comment 33 Vitaly Lipatov 2008-06-21 22:06:45 MSD
Вы тут не о
http://sisyphus.ru/srpm/Sisyphus/uniconvertor
?
Comment 34 ruslandh 2008-06-22 10:10:43 MSD
> Вы тут не о
>http://sisyphus.ru/srpm/Sisyphus/uniconvertor
>?
Именно о нём, вернее о его бэкпортировании в школьный комплект.
Comment 35 Michael Shigorin 2008-06-22 19:40:28 MSD
(In reply to comment #32)
> Ну раз ты, Майк, такой шутник, собирай юниконвертор :)
Вообще-то меня frob и пинал это сделать, но основной разработчик очень вовремя тогда спрыгнул на python-2.5, а в сизифе был ещё 2.4...

(In reply to comment #34)
> >http://sisyphus.ru/srpm/Sisyphus/uniconvertor?
> Именно о нём, вернее о его бэкпортировании в школьный комплект.
Смутно припоминается, что опять же Валёк уточнял насчёт 2.4/2.5, но лучше переспросить апстрима.

2 cas: возможно, сейчас это лучше проигнорировать, а решить к M41 в рабочем порядке.  Если только не "любой ценой" (ТМ).

PS: хорошая иллюстрация к тому, почему питон для production противопоказан: постоянные вилки (вилы) с версиями-модулями-всеми_необходимыми_софтинками.
Comment 36 Vitaly Lipatov 2008-06-23 00:45:33 MSD
(In reply to comment #35)
...
> > Именно о нём, вернее о его бэкпортировании в школьный комплект.
> Смутно припоминается, что опять же Валёк уточнял насчёт 2.4/2.5, но лучше
> переспросить апстрима.
Да, вроде бы с 2.4 не собирается.


> 2 cas: возможно, сейчас это лучше проигнорировать, а решить к M41 в рабочем
> порядке.  Если только не "любой ценой" (ТМ).
> 
> PS: хорошая иллюстрация к тому, почему питон для production противопоказан:
> постоянные вилки (вилы) с версиями-модулями-всеми_необходимыми_софтинками.
Мне кажется уже стоит прекратить рассказывать какой плохой Питон и почему его нельзя использовать. Слишком уже похоже на "сосед напел", или "да 30 лет назад я программистом работал". просто вот взять и прекратить выступать об этом на каждом встречном пеньке.
И иллюстрация к тому - плохая. Если бы разработчики не использовали новые конструкции, появившиеся в 2.5, проблемы бы не было.


Comment 37 Michael Shigorin 2008-06-24 16:40:01 MSD
(In reply to comment #36)
> > PS: хорошая иллюстрация к тому, почему питон для production противопоказан:
> > постоянные вилки (вилы) с версиями-модулями-всеми_необходимыми_софтинками.
> Мне кажется уже стоит прекратить рассказывать какой плохой Питон и почему
> его нельзя использовать. Слишком уже похоже на "сосед напел"
Да нет, сам споткнулся.

> просто вот взять и прекратить выступать об этом на каждом встречном пеньке.
> И иллюстрация к тому - плохая. Если бы разработчики не использовали новые
> конструкции, появившиеся в 2.5, проблемы бы не было.
(пожимая плечами) Ты майнтейнер, тебе видней.

Да только помимо новых конструкций -- на новые версии питона разработчиков толкает и то, что старый код на них не работает.  Потому как с совместимостью проблемы в обе стороны.

Всё, слез с пенька.