iceoryx
iceoryx copied to clipboard
eclipse-iceoryx
getopt_long is not implemented in windows
Brief feature description
getopt_long is not implemented in windows
Detailed information
My project needs large buffers, and I followed the configuration document, fill the size it needed, but when I was trying to start iox-roudi.exe with parameter "-c <configuration.location.toml>", it says "getopt_long is not implemented in windows". I have 2 solutions:
- Hardcode the size of memory pool in roudi_main_static_config.cpp
- Find an open sourced implementation of getopt_long and put it into the project to build a version which can make -c work.
Since I'd like to give my project more flexibilities, I chose solution 2, and it works well. I'd like to make a contribution to the project, but getopt_long is not written by myself, and I don't know the relevant policy of the project, so I have to ask this feature by posting this request.
Thank you.
@wukongiii We are aware of this and this is why we implemented a modern c++ abstraction for command line parsing, see: #1102
We will merge it later this day and based on that I will refactor the command line parsing for the iox-roudi so you should get nice and shiny new command line parsing capabilities on windows.
I will ping you when it is available!
Sadly we cannot go for the getopt_long open source implementation since it is licensed under GPL which is not compatible with Apache 2.0.
That's really an exciting news, thank you @elfenpiff ! You guys are awsome!
@elfenpiff given the merge, is this something that will be implemented soon ?
@niclar You are right! This is a tiny change now and I will look into it.
@niclar I'm not aware of anybody currently working on this. Patches are welcome :)
@elBoberido we have merged the command line parser and I thought we already using it in the roudi app. If not we could integrate it there quickly. This would remove a lot of code, and provide an awesome cli interface and we have windows covered with it.
@elfenpiff AFAIK we merged the CLI abstraction but did not port RouDi to it. Not sure how simple this would be since it has a layered design with optionally having an option for the path to a TOML config file.
