Bug 15749

Summary:	Split "to be changed" and "to be used as is" options
Product:	Sisyphus	Reporter:	Mikhail Gusarov <dottedmag>
Component:	sisyphus-mirror	Assignee:	Nobody's working on this, feel free to take it <nobody>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	enhancement
Priority:	P2
Version:	unstable
Hardware:	all
OS:	Linux

Description Mikhail Gusarov 2008-05-22 10:45:55 MSD

В конфиге sisyphus-mirror соседствуют как меняемые каждым админом (DESTROOT), 
так и чрезвычайно редко трогаемые (TMOUT1) опции.

Предлагаю чрезвычайно редко трогаемые опции унести в /usr/share/sisyphus-mirror/
default-config, и source-ить его перед /etc/sisyphus-mirror/config, дабы 
расхламить последний, но оставить возможность переопределить их при желании.

Меняемые каждым админом опции: SRCROOT (с дефолтом на rsync.altlinux.org в /usr/
share), DESTROOT, LIST, SPEED, MAINTAINER

Меняемые гораздо реже опции: RSHOME, MAXATTEMPTS, TMOUT1, TMOUT2, RSYNC_PROXY, 
EXCLUDE_FILE, PARTIAL, ARGS, INTERACTIVE

Comment 1 Vladimir V. Kamarzin 2008-05-27 14:33:53 MSD

Не вижу смысла плодить конфиги. Думаю, что лучше будет просто редко меняемые
опции поместить в конец конфига, а основные - в начало.

Comment 2 Mikhail Gusarov 2008-05-27 14:40:43 MSD

Не плодить, а делать короче :) в /usr/share не конфиг, а часть программы.

Впрочем, просто пересортировать опции - тоже решение.

Comment 3 Vladimir V. Kamarzin 2008-05-27 16:37:45 MSD

Отсортировал по степени важности, как я это вижу:

http://git.altlinux.org/people/vvk/packages/sisyphus-mirror.git?p=sisyphus-mirror.git;a=blob_plain;f=sisyphus-mirror/sisyphus-mirror.conf;hb=9e8aa812042ce3178d617b55a4aa7c2281c73871

Comment 4 Mikhail Gusarov 2008-05-27 18:23:00 MSD

ok, вполне

Comment 5 Vladimir V. Kamarzin 2008-05-28 08:56:06 MSD

Fixed in 0.7-alt1