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@xxxxxxxxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review