Bug 4279

Summary: Написать скрипт увеличивающий лимиты
Product: WINE@Etersoft Reporter: Vitaly Lipatov <lav>
Component: Интеграция в хост-системуAssignee: Andrey Vusik <night>
Status: CLOSED FIXED QA Contact: Денис Баранов <baraka>
Severity: major    
Priority: P2 CC: amorozov, kondratyuk, night, svzhu
Version: 1.0.12   
Target Milestone: ---   
Hardware: PC   
OS: All   
Whiteboard:
Заявки RT: Связано с:
Дата напоминания:
Bug Depends on:    
Bug Blocks: 5634, 6715    
Attachments: Готовый скрипт
Исправленный вариант
Скрипт

Description Vitaly Lipatov 2009-09-05 18:19:02 MSD
Поскольку самым частым вопросом является увеличение лимитов
на количество открытых файлов, предлагаю добавить скрипт, который
будет выполняться после первой установки и автоматически увеличивать лимит до
5 тысяч.
Остаётся вопрос - увеличивать для всех пользователей или для группы wine?
Я бы увеличивал для группы wine только, тем более что эта группа по умолчанию в setwineshare. Но это не даст желаемого результата (работы без вопросов).
С другой стороны, информацию о необходимости добавления пользователя в группу wine можно добавить в нашу программу, которая будет писать при превышении лимита.
Comment 1 Денис Баранов 2009-11-28 17:16:38 MSK
Нужно написать скрипт setwinelimits, который без указания параметров будет увеличивать количество открытых файлов для группы wine, при указании --group название (для определенной группы)
и при --all увеличение для всех.
Comment 2 Devaev Maxim 2009-11-28 18:03:45 MSK
@wine - nofile 5000

> и при --all увеличение для всех.
Для кого - всех?
Comment 3 Vitaly Lipatov 2009-11-29 03:18:58 MSK
(In reply to comment #2)
> @wine - nofile 5000
> 
> > и при --all увеличение для всех.
> Для кого - всех?
В файле limits.conf это задаётся звёздочкой.

Comment 4 Devaev Maxim 2009-11-30 18:52:50 MSK
Created attachment 1390 [details]
Готовый скрипт
Comment 5 Денис Баранов 2009-12-01 10:44:02 MSK
Максим, посмотри пожалуйста жизненный цикл баги: http://bugs.etersoft.ru/docs/html/lifecycle.html и следуй пожалуйста ему, чтобы они не терялись.
В git закомитил скрипт?
Comment 6 Денис Баранов 2009-12-01 12:40:31 MSK
1) Отсутствует help
2) При наличии limits.conf.winebak думаю следует создавать limits.conf.winebak1 или что то в этом роде.
3) При добавлении для определенной группы нужно проверять существует ли такая группа вообще, а то некорректно получается.
Comment 7 Devaev Maxim 2009-12-01 13:27:52 MSK
Created attachment 1395 [details]
Исправленный вариант

1 - сделал
2 - не думаю, что это нужно. Скрипт не дублирует свою работу, будучи запущенным на уже обработанном файле.
3 - сделал
Comment 8 Денис Баранов 2009-12-02 21:31:01 MSK
Гит закрытой части: git.office:/projects/wine-etersoft.git
Там есть скрипт tools/setwineshare
Сделай пожалуйста по образу и подобию.
Comment 9 Devaev Maxim 2009-12-03 14:53:16 MSK
Нужно добавить возможность определение уже установленного лимита, то есть, если уже стоит лимит больше 5000, то ничего не делать.

Привел скрипт в нормальное состояние, согласно рекомендациям.
Comment 10 Devaev Maxim 2009-12-03 15:17:54 MSK
Created attachment 1401 [details]
Скрипт

Подогнал скрипт к общему стилю, добавил копирайт.
Теперь скрипт не изменяет уже установленные параметры в меньшую сторону.
Comment 11 Денис Баранов 2010-03-14 19:35:43 MSK
Скрипт написан, но его так никто и не приложил в git.
Макс, сделай патчем пожалуйста.
Comment 12 Devaev Maxim 2010-03-15 18:03:30 MSK
Отправил в wine-patches
Comment 13 Денис Баранов 2010-04-15 13:40:52 MSD
Напиши пожалуйста еще документацию в git-docs
Comment 14 Devaev Maxim 2010-04-15 14:09:30 MSD
Откуда склонировать?
Comment 15 Денис Баранов 2010-04-15 15:11:37 MSD
(In reply to comment #14)
> Откуда склонировать?
> 
http://wiki.etersoft.ru/support/writedocs
Comment 16 Devaev Maxim 2010-04-23 17:54:02 MSD
Написал и выложил:
git.eter:/people/mdevaev/packages/wine-etersoft-docs.git
Comment 17 Devaev Maxim 2010-04-23 17:54:24 MSD
fixed
Comment 18 Денис Баранов 2010-10-15 11:51:38 MSD
Скрипт добавлен в сборку и работает.
Проверял на WINE@Etersoft 1.0.12
Comment 19 Денис Баранов 2011-01-14 20:11:31 MSK
Добавить возможность указывать количество устанавливаемых лимитов.
Comment 20 Devaev Maxim 2011-01-17 15:36:34 MSK
(В ответ на comment #19)
> Добавить возможность указывать количество устанавливаемых лимитов.

Как это - количество? Поясни.
Comment 21 Александр Морозов 2011-01-17 15:41:14 MSK
Подозреваю, что имеется ввиду величина лимита. Например, что бы можно было указать 10000 дескрипторов, или 20000.
Comment 22 Денис Баранов 2011-01-17 16:38:30 MSK
(В ответ на comment #21)
> Подозреваю, что имеется ввиду величина лимита. Например, что бы можно было
> указать 10000 дескрипторов, или 20000.

Абсолютно верно, не хватает этой функциональности.
Comment 23 Devaev Maxim 2011-01-17 20:42:36 MSK
Сделайте доступ, пожалуйста.


[mdevaev@builder projects]$ git clone git.office:/projects/wine-etersoft.git
Cloning into wine-etersoft...
remote: fatal: loose object ee3d3e2b0448fd9a4ef96455f491720d32d2ae34 (stored in ./objects/ee/3d3e2b0448fd9a4ef96455f491720d32d2ae34) is corrupted
error: git upload-pack: git-pack-objects died with error.
fatal: git upload-pack: aborting due to possible repository corruption on the remote side.
remote: aborting due to possible repository corruption on the remote side.
fatal: early EOF
fatal: index-pack failed
Comment 24 Devaev Maxim 2011-01-18 19:41:03 MSK
Переделал разбор опций, добавил возможность указания -l/--limit, отправил патч на wine-patches-test.
Comment 25 Devaev Maxim 2011-01-18 19:42:46 MSK
<fixed>
Comment 26 Денис Баранов 2011-01-20 19:22:18 MSK
(В ответ на comment #24)
> Переделал разбор опций, добавил возможность указания -l/--limit, отправил патч
> на wine-patches-test.

Опять где застряло. Проследи чтобы твои патчи попали в репозиторий.
Comment 27 Денис Баранов 2011-03-12 14:51:26 MSK
Добавил справку о параметре --limit
Comment 28 Svetlana Zhukova 2011-04-20 16:45:18 MSK
бутылка <wine@cellar bottle bugs/4279>

при  $ wine --limit        или        $ wine -l

wine: could not load L"C:\\windows\\system32\\-l.exe": Module not found
fixme:process:__wine_kernel_init 1: L"C:\\"(L"C:\\windows")
Comment 29 Andrey Vusik 2011-04-20 17:13:42 MSK
Нету опции..
Comment 31 Andrey Vusik 2011-04-20 18:12:30 MSK
Есть такое.