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 rebased against the current
> HEAD and updated to incorporate mtaylor's feedback 
> Vit submitted the backports  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.
>  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.
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