[Bug 463229] Review Request: ACE Banners Module

2008-10-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bryan Kearney [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX




--- Comment #5 from Bryan Kearney [EMAIL PROTECTED]  2008-10-02 15:12:28 EDT 
---
Combining this package into https://bugzilla.redhat.com/show_bug.cgi?id=463226
as a subpackage.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 463229] Review Request: ACE Banners Module

2008-09-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Bryan Kearney [EMAIL PROTECTED]  2008-09-24 09:28:55 EDT 
---
Updated the packages per the review of the ace pakacge

Spec URL: http://bkearney.fedorapeople.org/ace-banners.spec
SRPM URL: http://bkearney.fedorapeople.org/ace-banners-0.0.3-3.fc9.src.rpm

rpmlint clean
koji build clean

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 463229] Review Request: ACE Banners Module

2008-09-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Bill Nottingham [EMAIL PROTECTED]  2008-09-23 16:05:18 
EDT ---
Is there a better way to package puppet recipes than this? It seems having
umpteen subpackages isn't the way to go.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 463229] Review Request: ACE Banners Module

2008-09-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Bryan Kearney [EMAIL PROTECTED]  2008-09-23 16:23:56 EDT 
---
I would be happy to discuss real time to find on. The goal is to have the
appliance ship with all the correct packages, and not have to call home at
initial start up. For that reason, puppet is not used to require packages.. the
spec file is. So each subpackage will have a puppet module, and a set of
requires.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 463229] Review Request: ACE Banners Module

2008-09-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Bill Nottingham [EMAIL PROTECTED]  2008-09-23 16:40:46 
EDT ---
Right, but...

- You've got these weird puppet modules that have initimate knowledge of the
configuration details of their apps (file locations, etc.). Why not package
them in some form with the app itself? How much will they need changed if the
apps themselves change?

- A separate source package for every single module you may want in a appliance
sounds like bloat. Are they really intended to be released independently?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review