[Bug 226387] Merge Review: samba

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: samba


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226387





------- Additional Comments From dmitry at butskoy.name  2007-03-30 08:17 EST -------
Whether it could be better to put migration stuff into system init script(s) ?
In general, consider that you are upstream, therefore you provide only tarball
(not rpm or deb package), and you have to do some migration things for new
version...

Also a good idea could be to play with trigger scripts, i.e.:

%triggerpostun -- samba <= latest_version_with_old_layout
some_code_here...

or

%triggerpostun common -- samba-common <= latest_...

Such a technique is already used by some packages (alsa-lib, openssh etc.) where
something should be cleaned up after old version deinstalled.
May be another type of trigger scripts are suitable here (%triggerin etc.)


Since /etc/samba/smbpasswd is a text file very similar to system /etc/passwd,
maybe better to save it under /etc/samba/ dir, not move too dip under
/var/lib/samba/private. (in other words, why /etc/passwd is not moved under,
say, /var/lib/security/secrets/ ? ).


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.