Aeolus on Fedora 17 / Ruby 1.9.3 (rev 6)

Mo Morsi mmorsi at redhat.com
Fri May 4 06:28:23 UTC 2012


On 05/03/2012 05:08 PM, Martyn Taylor wrote:
> On 05/02/2012 02:48 PM, Mo Morsi wrote:
>> This is the last patchset[1] rebased against the current
>> HEAD and updated to incorporate mtaylor's feedback [2]
>>
>> Vit submitted the backports [3] gem to Fedora for review. I will
>> look at taking this up (if noone gets to this first) but until
>> it's in, am still relying on the manual path workaround in our codebase.
>>
>>    -Mo
>>
>> [1]
>> https://fedorahosted.org/pipermail/aeolus-devel/2012-April/009996.html
>> [2]
>> https://fedorahosted.org/pipermail/aeolus-devel/2012-April/010051.html
>> [3] https://bugzilla.redhat.com/show_bug.cgi?id=816991
> Condition ack with the following changes:
>
> * remove the fastercsv dep from the spec.in
>   This is causing the build to fail as it cant find this rpm for f17
>
> * add require 'will_paginate/array' to applicaiton controller
>   in will_paginate 3.0.2 array is no longer loaded as default as such
> needs to be explicitly required
>
> Notes going forward.
>
> We should do configure first then the other projects.  Since this took
> quite a lot of setting up and fiddling.  Would be much easier with
> cnofigure already working.
>
> Thanks alot
>
> Martyn

Pushed patchset. Sorry forgot about your comments before I pushed so
just pushed a small follow on patch with these changes (already had
integrated Jan's feedback though).

Thanks for the review, will begin the process of pushing update rpms to
Fedora soon.

  -Mo



More information about the aeolus-devel mailing list