[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Feature] Add --self-compare option in fedabipkgdiff



Patch with typo fix has been pushed to master
https://sourceware.org/git/?p=libabigail.git;a=commit;h=847d96b1d109480b776854c49c40bef42026ab98

On Wed, May 31, 2017 at 4:15 PM, Sinny Kumari <ksinny@gmail.com> wrote:
> On Wed, May 31, 2017 at 4:13 PM, Dodji Seketeli <dodji@seketeli.org> wrote:
>> Chenxiong Qi <cqi@redhat.com> a écrit:
>>
>>> On Sun, May 21, 2017 at 6:29 PM, Sinny Kumari <ksinny@gmail.com> wrote:
>>>> Hi,
>>>>
>>>> This patch (available in email attachment)  adds --self-compare option
>>>> in fedabipkgdiff tool.
>>>>
>>>> fedabipkgdiff tool can communicate with Fedora koji and has capability
>>>> to download and perform ABI comparison between specified NVRs.
>>>>
>>>> With addition of --self-compare option, it will be possible to perform
>>>> ABI comparison on same package. One of the important usecase of this
>>>> option is to run automated ABI checks on packages with known expected
>>>> result i.e. no ABI change. This usecase will be useful to ensure that
>>>> libabigail functionality doesn't break with new commits made.
>>>> This option can be invoked as:
>>>>   fedabipkgdiff -a --self-compare -fc24 pacakge_name
>>>
>>> Typo here.
>>>
>>> LGTM.
>>
>> For what it's worth, I agree fully with Chenxiong here.  It looks good
>> to me with the typo above fixed.
>
> Thanks Dodji! I will push it then with typo fixed



-- 
http://sinny.io/