dheeraj12347 commented on PR #229: URL: https://github.com/apache/cloudstack-cloudmonkey/pull/229#issuecomment-6109903224
> > I've pushed a follow-up commit, `c79972f`, that moves map-field matching before the argument-prefix check > > @dheeraj12347 tags and tags[0].k work now, but tags= still breaks in the real shell. pressing tab there gives tags=[0]. because the shell adds the text after the = instead of replacing it. can the test check the full line after tab, not just the suggestions? @Damans227 , I traced the `tags=` issue to readline's completion behavior. `AutoCompleter.Do()` returns candidate suffixes, and readline appends them to the existing buffer, so the trailing `=` cannot be removed by the CloudMonkey completer alone. A proper fix appears to require changing readline's replacement behavior. Would you be comfortable with a narrowly scoped change to the vendored readline code, or would you prefer an approach that avoids modifying the dependency? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
