Bug 759612

Summary: Cull dependencies list
Product: [Other] Topic Tool Reporter: Stephen Gordon <sgordon>
Component: cli-Topic_ToolAssignee: Stephen Gordon <sgordon>
Status: CLOSED WONTFIX QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 0.0.xCC: topic-tool-list
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-10-03 19:00:44 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Stephen Gordon 2011-12-02 20:44:47 UTC
Description of problem:

Topic tool, since it was initially prototyped, has included a number of apache-commons* dependencies which have made packaging it difficult. In particular the fact that the names of these dependencies vary between each support Fedora and RHEL version, and probably will for some time, leave us with a number of nasty hacks in the spec file. Additionally because some of them aren't available in RHEL we had to bundle them.

Given the rudimentary use we make of some of these libraries I think it would make sense to replace the code that uses them and remove the dependency.

Comment 1 Stephen Gordon 2012-09-24 18:39:06 UTC
The naming of these packages is now relatively stable:

F14:
====

Uses mix jakarta* and apache* prefix.

F15, F16, F17:
==============

Uses apache* prefix.

RHEL5, RHEL6:
=============

Uses jakarta* prefix.

I think the right change under this bug is to consolidate the macros for Fedora. 

- The F14 and F15 macros will be dropped entirely as they no longer pertain to supported releases.
- The F16 and F17 (and ultimately F18) macros will be combined as they effectively indicate the same condition (require apache*).
- The RHEL5 and RHEL6 macros will be left as is because they aren't going to change any time soon.

Comment 2 Stephen Gordon 2012-09-24 18:56:30 UTC
- Release specific Fedora checks are now replaced by %{fedora} check.
- Release specific RHEL checks remain for the foreseeable future.

Committed revision 85161 to trunk.