[Bug 751564] Review Request: The Aeolus Audrey Startup Agent

[bugzilla at redhat.com]

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=751564

--- Comment #19 from Joe VLcek <JVLcek at RedHat.com> 2011-11-09 10:36:17 EST ---
(In reply to comment #18)
> Created attachment 532490 [details]
> tweak spec file so "make rpms" succeeds
> 
> There were two problems.  I've attached a revised patch:
> 
>     tweak spec file so "make rpms" succeeds
> 
>     * audrey_startup/aeolus-audrey-agent.spec.in: Install into buildroot,
>     not directly into /usr/man.
>     (%files): List audrey.8*, not audrey.8, to get the .gz file, too.

Thank you Jim!

I had caught the "Install into buildroot not directly into /usr/man

But I was not sure of the correct way to handle the .gz file issue.

So again thank you.

I've pushed the latest changes here:
https://github.com/aeolusproject/audrey/tree/joev

I have a question: Instead of hard coding the version in the audrey_start.py
file as:
AUDREY_VER = '0.5.1'

I would like to track it with the RPM version number.

Do you have any suggestions on how best to approach that?

Thank you!
   Joe

-- 
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.