Bug 796805 - Merge aeolus-all into aeolus-conductor
Summary: Merge aeolus-all into aeolus-conductor
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-all
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: rc
Assignee: John Eckersberg
QA Contact: wes hayutin
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-23 17:13 UTC by John Eckersberg
Modified: 2012-08-30 17:16 UTC (History)
1 user (show)

Fixed In Version: v0.8.0-36
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-30 17:16:11 UTC


Attachments (Terms of Use)

Description John Eckersberg 2012-02-23 17:13:02 UTC
Description of problem:

Presently within the conductor source, we have two spec files, one for aeolus-all and one for aeolus-conductor.  Both are always the same version, and both always contain the same changelog.  When cutting a new release the process is as follows:

1. Update spec for conductor
2. Copy conductor spec updates into specfile for aeolus-all
3. Run `make rpms`, which generates two srpms, one for aeolus-conductor and one for aeolus-all
4. Use rhpkg to import aeolus-conductor into dist-git, commit, push, build
5. Use rhpkg to import aeolus-all into dist-git, commit, push, build

It makes much more sense to have aeolus-all be a subpackage of aeolus-conductor instead.  This way aeolus-all will inherit the version and changelog from aeolus-conductor automatically.  `make rpms` would only output one srpm, and we would only have to import and build a single srpm.

Comment 1 John Eckersberg 2012-02-23 17:15:14 UTC
I forgot to mention, jgreguske specifically pointed out that standalone meta-packages (which aeolus-all presently is) are frowned upon and the proposed solution is more standard for them.

Comment 2 John Eckersberg 2012-02-23 17:30:14 UTC
Patch on the list:

https://fedorahosted.org/pipermail/aeolus-devel/2012-January/008433.html

Comment 3 John Eckersberg 2012-02-23 17:38:01 UTC
40ced33 on github/master

Comment 4 Steve Linabery 2012-02-24 04:16:14 UTC
f36474d

Comment 5 wes hayutin 2012-02-27 21:19:27 UTC
verified


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