Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Command line 'arguments' parsing hyphens #17

Open
timegrinder opened this issue Jan 15, 2019 · 4 comments
Open

Command line 'arguments' parsing hyphens #17

timegrinder opened this issue Jan 15, 2019 · 4 comments
Assignees

Comments

@timegrinder
Copy link

Hey there,

Brilliant program, thanks for the work!

I've noticed that the command line --arguments "blah" doesn't play nice when the arguments contain a hyphen as the executable isn't escaping / ignoring any hyphens inside the quotation marks, so attempts to parse something like
--arguments "-tcp"
as a second -t time call such as -t cp and fails as a result (tested with -tcp and -nosplash), one provides the error -t/--timeout option violates format , the other is being treated as -n with no error and the usual list of command line options pops up.
Initially occurred with a shortcut made by HDM itself, so I assume just the way the executable reads them in.

As a secondary question (that isn't an issue) do you have a road map or list of 'possible features' somewhere? I'm interested to see where this is going and what is already planned so I don't end up posting feature requests you're already working on.

@falahati
Copy link
Owner

falahati commented Jan 16, 2019

Hey,

Yeah, I am aware of the issue with the command line. This is what .Net does and there is no clear away around that. I need to investigate this further.

In regard to what I have in mind to do with this project:

  • Make it possible to edit profiles
  • Make it possible to add a new profile directly from the application
  • Automatic update check and notifications
  • Create a read me file
  • Considering if we should save and apply monitor colour configuration (Vibrance reset #9)
  • Considering if we should save and apply taskbar settings (After switching taskbar end up on top #14) - The code behind this is already done actually
  • AMD Eyefinity support using ADL

@falahati falahati added the bug label Jan 16, 2019
@timegrinder
Copy link
Author

timegrinder commented Jan 16, 2019

Makes sense that .NET would have a silly problem like that.

Any possibility of keeping that list / considered features somewhere open like the Wiki page for the project? Including 'current complete / partially implemented features' so that people can get an idea without necessarily having to poke around the application?

Thanks again!

@falahati
Copy link
Owner

falahati commented Jan 16, 2019

That supposed to be the job of "Create a read me file" task. But at this point, I rather try to add the much-needed edit and add profile feature instead of writing a read me file. But in time, before even trying to implement AMD Eyefinity API I will make sure to fix the readme file. That would be the first release of this project as well (v1).

To this date, every release was a pre-release. Let's just say I am not happy with the state of this project as of now. I just need more time. I have a plan :)

@timegrinder
Copy link
Author

Ah right, I understood it was pre-release, my questions were more from the point of view of outside assistance, say someone else saw the project and wanted to see if they could help or had experience / existing code/information they could port to it, knowing where it's at and what is planned makes it easier to direct any such input.

@falahati falahati self-assigned this Jan 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants