On Wed, May 26, 2021 at 7:44 PM Bharath Rupireddy <bharath.rupireddyforpostg...@gmail.com> wrote: > > On Wed, May 26, 2021 at 7:18 PM vignesh C <vignes...@gmail.com> wrote: > > +1 for fixing this issue, we have handled this error in other places. > > The patch does not apply on head, could you rebase the patch on head > > and post a new patch. > > Thanks. I thought of rebasing once the other patch (which reorganizes > "...specified more than once" error) gets committed. Anyways, I've > rebased for now on the latest master. Please review v2 patch.
The test changes look good to me, I liked the idea of rebasing the source changes once "specified more than once" patch gets committed. I will review the code changes after that. Regards, Vignesh