search

Vadoola / dxf2elmt

A CLI program to convert .dxf to .elmt
MIT License
1 stars 2 forks source link

Fix Command line flag order issue #11

Closed Vadoola closed 2 months ago

Vadoola commented 2 months ago

It would appear the day after the last binary was released from the original repo, the code was updated from manual command line parsing to using clap. There now appears to be an discrepancy with how the order of the flags can alter the output.

In QElectrotech the command is called as dxf2elmt -v and dumps the XML to stdout, but the master branch (and my forked version) using clap seem to change depending on where the -v flag is, or it may now require the -i flag as well. I'll need to take a closer look.

This can be see discussed here.

Vadoola commented 2 months ago

Resolved by removing the extra else that was preventing the verbose flag from being checked unless the info flag was also passed.