Bug 15749 - Split "to be changed" and "to be used as is" options
Summary: Split "to be changed" and "to be used as is" options
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: sisyphus-mirror (show other bugs)
Version: unstable
Hardware: all Linux
: P2 enhancement
Assignee: Nobody's working on this, feel free to take it
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-05-22 10:45 MSD by Mikhail Gusarov
Modified: 2008-06-13 18:23 MSD (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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