[rfc][icedtea-web] PolicyEditor uses OptionParser

Andrew Azores aazores at redhat.com
Thu Jan 15 18:20:14 UTC 2015


On 01/06/2015 11:14 AM, Jie Kang wrote:
> Hello,
>
> Patch looks fine to me. In terms of aazore's changes, I'd leave them out of this patch. Whichever one goes in second will deal with the conflicts appropriately and it won't be too much work.
>
>
> Regards,
>
> ----- Original Message -----
>> Hello,
>>
>> This patch changes PolicyEditor to use OptionParser to parse its arguments.
>> I have noticed that Andrew's PolicyEditor gains a real parser patch removes
>> the codebase tag as an option. This patch does not at the moment, but I can
>> follow if that is what is determined to be wanted.
>>
>> Thoughts ?
>>
>> Thank you,
>> Lukasz Dracz
>>
>

Agreed, leave it to the second one to go in (probably my parser stuff) 
to remove the option, don't do it now.

Looks good to me.

Thanks,
-- 
Andrew Azores


More information about the distro-pkg-dev mailing list