[Bug 613001] Review Request: Heimdal - Alternative Kerberos implementation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



--- Comment #62 from Ken Dreyer <ktdreyer@xxxxxxxxxxxx> ---
Here's the new version. This should address the issues in comment #58 (along
with other fixes).

* Thu May 22 2014 Ken Dreyer <ktdreyer@xxxxxxxxxxxx> -
1.6.0-0.4.20140522git229d8c7
- Update git snapshot to latest tip of heimdal-1-6-branch
- Drop upstreamed text-fx patch
- Install Texinfo files (RHBZ #613001)
- Add Provides: heimdal-static to -devel subpackage (RHBZ #613001)
- Drop %%defattr (RHBZ #613001)
- Add text content to kadmind.acl to help users (and remove a zero-length file)
- Install profile.d scripts with non-executable permissions
- Remove .la files
- Patch to remove AC_PROG_LIBTOOL macro
- Reload xinetd when using systemd
- Require logrotate and setup, since we drop config files into directories that
  these packages own.
- Add unowned Heimdal directories in %%files
- Replace "heimdal" with %%{name} in %%files
- Do not BR libcap-ng-devel on EL5

Exact changes in Git:
https://gitorious.org/ktdreyer/heimdal-packaging/commit/720db3876630945b2841b354d798827c2ace43df

Spec URL: http://ktdreyer.fedorapeople.org/reviews/heimdal.spec
SRPM URL:
http://ktdreyer.fedorapeople.org/reviews/heimdal-1.6.0-0.4.20140522git229d8c7.fc21.src.rpm

F21 scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6894313

A couple of comments on the remaining rpmlint issues:
- rpmlint found a dangling-symlink, but I don't see how this could be the case,
  since it Requires: pam, which Provides the symlink destination,
  /etc/security/access.conf.
- invalid-locale-man-dir - this is a result of putting the man pages in a
  "heimdal" subdirectory so they don't conflict with existing man pages in
  other packages.
- Some files have non-standard-dir-perm or non-readable settings. These files
  should only be readable by root, so it's ok to ignore rpmlint warnings about
  these.
- The /etc/heimdal-slaves file is zero-length because there's no content to add
  to it by default. (Deleting the file from the package would leave a dangling
  symlink in /var/heimdal.)
- The devel-file-in-non-devel-package warnings are because Heimdal has plugins
  that look like shared libraries.
- The explicit-lib-dependency error with heimdal-path is a false positive:
  heimdal-path requires end-user binaries, not shared libaries that could be
  autorequired. The cleanest solution is to just Require: heimdal-libs.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]