Bug 1290239 - Update krb5 spec file with changes made in fedora
Summary: Update krb5 spec file with changes made in fedora
Alias: None
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: krb5
Version: 7.3
Hardware: All
OS: Unspecified
Target Milestone: rc
: ---
Assignee: Robbie Harwood
QA Contact: Patrik Kis
Depends On:
TreeView+ depends on / blocked
Reported: 2015-12-09 22:26 UTC by Robbie Harwood
Modified: 2016-11-03 20:23 UTC (History)
2 users (show)

Fixed In Version: krb5-1.14.1-2.el7
Doc Type: No Doc Update
Doc Text:
Clone Of:
Last Closed: 2016-11-03 20:23:28 UTC
Target Upstream Version:

Attachments (Terms of Use)

System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2016:2591 0 normal SHIPPED_LIVE Low: krb5 security, bug fix, and enhancement update 2016-11-03 12:10:29 UTC

Description Robbie Harwood 2015-12-09 22:26:33 UTC
Mostly general housekeeping, but a bunch of cruft has been removed and the result is easier to work with and maintain.

Comment 1 Patrik Kis 2015-12-10 09:41:12 UTC
Would it be possible to post here some details about the planned changes.
In general, if the changes were good for Fedora, I believe they will be good for RHEL too. But it would be nice to see in advance, what changes are planned.

Comment 2 Robbie Harwood 2015-12-10 23:49:57 UTC
Sure thing.  At the moment, the biggest thing is dropping support for old versions of rhel/fedora that are kicking around.  I've also gotten a number of requests for dependency slimming - things I especially want to remove include dependencies on pax and the various sysV support packages.  Mostly https://github.com/frozencemetery/krb5_fedora/commit/a328acab1bac263ca71b0399342fd4c79f294f87 and https://github.com/frozencemetery/krb5_fedora/commit/5ec8cb89e024ae7794e38d71dd6373f638bdd705

Comment 6 Patrik Kis 2016-06-27 10:43:12 UTC
All changes were reviewed and make sense.

The following piece probably can be cleaned as well as it is not going to be executed after the definition of 'compiled_default_ccache_name' was removed from spec file. I just note it here for possible future cleaning plans.

--- krb5-1.14.1-2.el7.src/krb5.spec	2016-03-31 19:10:40.000000000 +0200
+++ krb5.spec	2016-06-27 12:36:58.955101857 +0200
@@ -309,9 +309,6 @@ sed -i -e s,7778,`expr "$PORT" + 1`,g $c
 source %{_libdir}/tclConfig.sh
 pushd src
 # Keep the old default if the package is built against older releases.
-%if 0%{?compile_default_ccache_name}
-DEFCCNAME=%{compiled_default_ccache_name}; export DEFCCNAME
 # Set this so that configure will have a value even if the current version of
 # autoconf doesn't set one.
 runstatedir=%{_localstatedir}/run; export runstatedir

Comment 9 errata-xmlrpc 2016-11-03 20:23:28 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.