On 02/28/2012 05:33 PM, Matt Wagner wrote:
This is a resend of my earlier patch, "BZ 795743 - Detect
architecture for imported images" in response to Jan's feedback.
Namely, he pointed out that, since we're storing this architecture
attribute for imported images, we might as well *display* it. I added
that in this patch.
I got sick of writing "image.os.arch || image.architecture", so I
added an "architecture" method to images in aeolus-image-rubygem to
do that for us. This patch is already introducing changes to
aeolus-image-rubygem so this seemed tolerable, though this change
does mean that you'll get exceptions if you upgrade Conductor and not
aeolus-image-rubygem. (So I'll send out an [ANNOUNCE] message noting
this after this patch is ACKed but before pushing, as someone
proposed earlier today.)
-- Matt
ACK to the patchset. Nice.
Jan