[Toaster] Review request for Bug 5947 - Replace fstypes lookup on main build page

[Reyna, David]

Hi Alex,

I can and will redo the commit. Did you revert the original patch?

I did a pull at 11:00 pm 2014/3/20 and I still see the patch. If you allow me to revert it explicitly with the updated patch, then you do not need to revert and I do not need to rebase yet again and lose a day, nor would anyone else preparing commits with changes to “projecttags.py”.

Also, we are seeing image file entries like “rootfs.manifest”. How does the algorithm know that “rootfs.tar.bz2” and “rootfs.ext3” are image files,  but ‘rootfs.manifest” is not? Or is the issue that entries like “rootfs.manifest” should not be in the image file list in the first place?

- David

From: toaster-bounces at yoctoproject.org [mailto:toaster-bounces at yoctoproject.org] On Behalf Of Damian, Alexandru
Sent: Thursday, March 20, 2014 4:45 AM
To: Wymore, Farrell
Cc: toaster at yoctoproject.org
Subject: Re: [Toaster] Review request for Bug 5947 - Replace fstypes lookup on main build page

Hi,

I made a sloppy job reviewing this patch, I'm sorry for this.

The filters should have no knowledge of the object system. All object manipulation should happen only in the views.py page, and relevant values passed through the context.

I'm reverting this patch for the 2014/03/20 submission. Can you please rework this patch with the relevant data queries inside views.py ?

Thank you,
Alex

On Fri, Mar 14, 2014 at 5:37 PM, Damian, Alexandru <alexandru.damian at intel.com<mailto:alexandru.damian at intel.com>> wrote:
Taken in.

On Tue, Mar 11, 2014 at 11:00 PM, Wymore, Farrell <Farrell.Wymore at windriver.com<mailto:Farrell.Wymore at windriver.com>> wrote:
Hi Belen,

I  have pushed the fix for this defect.
    fwymore/fstypes

Thanks in advance - fw

--
_______________________________________________
toaster mailing list
toaster at yoctoproject.org<mailto:toaster at yoctoproject.org>
https://lists.yoctoproject.org/listinfo/toaster



--
Alex Damian
Yocto Project
SSG / OTC



--
Alex Damian
Yocto Project
SSG / OTC
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/toaster/attachments/20140321/b45016b1/attachment-0001.html>