COVESA
Reapply "Use QCommandLineParser instead of custom cmd args parsing (#543)"
PR employs QCommandLineParser for parsing cmd args for dlt-viewer. As requested, in a followup PRs similar change can be applied to other cli tools developed in this repo.
The PR restores the previously reverted changes, that reported to cause some issues. Should remain unmerged until fix is found (but only after clear issue is reported) or we can merge it again and if issue is observed, request to create a bug ticket with clear description.
@alexmucde I would need your help to understand what kind of issues this causes and how to reproduce it. Unit tests can help to ensure workability of different cmd scenarious.
I have tried a first use case, which did not work: -c C:\traces\out.txt C:\traces\in.dlt -t The conversion to ASCII was not working.
from the path I conclude that it is on Windows. Thanks @alexmucde ,I'll investigate from here
I do not know what I do differently, but for me it just works on Windows too:
No it works also on my side. Do not know what i made wrong last time. I will test now further use cases.
The use cases i have tested work now. Should we give it another try now and merge it?
I would give it a try, yes. Quite confident thanks of unit tests, but of course platform-specific bugs are not excluded, so if there is an issue would be nice to have a nice bug report
@vifactor Tested it now also in my productive system in Ubuntu 24.04 and works fine.