[rfc][icedtea-web] PolicyEditor uses OptionParser

Jiri Vanek jvanek at redhat.com
Wed Jan 21 10:03:52 UTC 2015


On 01/15/2015 07:20 PM, Andrew Azores wrote:
> 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.

Yes pelase, push.

Btw - how does it will works without codebase argument (with realaprser in)?
>
> Thanks,



More information about the distro-pkg-dev mailing list