[Ovirt-devel] [PATCH] create-wui-appliance.sh: minor fixes

Perry N. Myers pmyers at redhat.com
Mon Aug 4 13:57:56 UTC 2008


Jim Meyering wrote:
> "Perry N. Myers" <pmyers at redhat.com> wrote:
>> Changes look fine so ACK.  One question though, why did you change
>> kickstart to lowercase and not IMGDIR?
> 
> Hi Perry,
> 
> Good point.
> I didn't notice that there was another non-const variable
> that was upper case.  I noticed/changed KICKSTART because I was
> already fixing a problem with it.
> Even so, I had qualms about mixing syntax-only changes
> with bug-fixing ones.  But this is nothing serious or deep,
> so it's no big deal to obscure the fixes slightly.
> 
> Also spotted an unused "ISO".
> 
> If no one objects, I'll push the following instead:

Looks good to me, go ahead and push.

Perry




More information about the ovirt-devel mailing list