Bug 6417

Summary: Wine-patches: отправка файлов с логами по почте
Product: [Свободные проекты (Open source projects)] WINE-tests Reporter: Виталий Перов <vitperov>
Component: ОбщееAssignee: Виталий Перов <vitperov>
Status: CLOSED FIXED QA Contact:
Severity: minor    
Priority: P4 CC: baraka, lav
Version: не указана   
Target Milestone: ---   
Hardware: PC   
OS: All   
Whiteboard:
Заявки RT: Связано с:
Дата напоминания:
Bug Depends on: 6280, 6291    
Bug Blocks: 6264    

Description Виталий Перов 2010-11-10 19:03:04 MSK
При возникновении ошибки хотелось бы отправлять файл с логом сборки wine автору патча по почте.
Также при "проваливании" тестов надо отправлять файлы *.before и *.after
Comment 1 Виталий Перов 2010-11-11 16:34:35 MSK
Добавил функцию attach_textfile().
Алгоритм отправки письма слегка изменился.
На всякий случай проверил. Возникла пролема  - теперь приходит пустое письмо.
Comment 2 Виталий Перов 2010-11-11 16:52:56 MSK
> Возникла пролема  - теперь приходит пустое
> письмо.
Исправил 

Comment 3 Виталий Перов 2010-11-11 17:32:51 MSK
Ошибка при сборке и ошибка при приложении возникают в классе wine_repository. Этот класс не наследуется от класса send_mail. И не может сам ничего посылать по почте.

Можно вместо булевого значения возвращать строку с именем полученного файла, а затем классе wine_patch её обрабатывать. Но думаю это неправильное решение, продумать наследование от send_mail (бага #6291)
Comment 4 Виталий Перов 2010-11-15 17:24:38 MSK
Добавил приаттачивание лога при ошибке сборки.
Исправил метод присоединения. Если файл присоединять как текст, то он почему-то отображается в тексте письма (что очень не удобно). Теперь присоединяю как application/octet-stream. Так работает.
Comment 5 Виталий Перов 2010-11-15 17:35:37 MSK
Добавил отправку логов если есть изменение в тестах.
Comment 6 Виталий Перов 2010-11-15 18:10:13 MSK
Сделал. Проверил. Работает.