https://bugzilla.redhat.com/show_bug.cgi?id=1010000 --- Comment #7 from Sandro Mani <manisandro@xxxxxxxxx> --- Debian only uses /usr/lib (that is, before multiarch[1]), so the reason /usr/lib is used is probably because the package targets debian. On Fedora OTOH we have /usr/lib64, so I'd say that is where the shared library should go. As far as debchange.pl is concerned, /usr/lib appears only in a comment, so hardly worth patching it. Sed vs patch + %if: I guess when a patch should be preferred over a shell command in the spec is a matter of taste for very small changes. In this case I feel that a sed command is simpler and easier to maintain (i.e. much like adjusting line endings is also done with shell commands). The postinst and postrm scripts: - postinst upgrades /etc/devscripts.conf with new options added with new releases of the package. I think in Fedora we usually just ship the newest default config file, specify the config files as %config(noreplace), which then causes rpm to install the files with the .rpmnew suffix, and it is up to the user to adapt the old config file. So postinst in unnecessary. - postrm: this removes the config file when --purge is passed to apt-get. rpm does not have purge, but automatically saves the config files to .rpmsave. So this scriplet also is unnecessary. [1] https://wiki.debian.org/Multiarch -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=zhtScu6Z6Y&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review