[Bug 226407] Merge Review: sendmail

[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: sendmail


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





------- Additional Comments From paul at city-fan.org  2007-04-12 11:14 EST -------
(In reply to comment #17)
> > E: sendmail hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/libmilter.a
> > => false positive, the %install script is removing the library incorrectly
> > => installed there by the upstream build script
> This is wrong, because /usr/lib/libmilter.a gets removed for a 64 bit build,
> because /usr/lib/libmilter.a has to be 32 bit here, but sendmail installs the 64
> bit build.

OK, but that's what I meant; the upstream installer puts it in /usr/lib and the
spec puts it in /usr/lib64, so the upstream-installed one needs to be removed on
x86_64.

> symlinks is not usable, because it would create realtive links outside of the
> buildroot.

Are you sure? I tried the patch from Comment #8 before posting it and it looked
to be doing the right thing to me.

> %{_localstatedir} ist not /var it is %{_prefix}/var, ergo /usr/var.

$ rpm --eval '%{_localstatedir}'
/var

> /etc/aliases is not part of the sendmail package.

It should be if the package is supposed to work with %{old_setup} = yes
(obviously not for current Fedora versions but for people trying to rebuild for
older distros). I don't think %{old_setup} should be supported half-heartedly -
either let it work properly or remove it altogether.

> ===> Please have a look at sendmail-8.14.1-1.

Where can I find that? It's not in my just-updated cvs checkout.

-- 
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.