For #35869: do version token replacement in the args setting

[kporangehat]

Apply our version token replacement to the *_args setting like we currently do for the path and icon settings. This enables clients who use a wrapper to launch their DCC, take advantage of the version tokens in the *_args setting. And allows them to support launching multiple versions of their DCC using the versions setting.

[kporangehat]

Hmmm. I see #25 now and think that's maybe a nicer implementation. thoughts?

[robblau]

First thought is that in #25 there is logic about falling back to the first version configured if the arg is passed in as None. Doing that only for the args as opposed to the other places where version is used seems inconsistent.

An approach like this makes more sense to me with the current implementation and we should talk through adding in smarter default behavior. I'm not sure what workflow ends up with version being passed in as None if versions have been configured.

Lets go forward with this kind of implementation and I'll ask in the other PR.

[robblau]

Ok... looking deeper, I'm seeing the same code for _get_app_path (using versions[0]).
Les use #25 instead, although a refactor to centralize that logic would be awesome.

[kporangehat]

Okay I commented on #25 with the refactor suggestion and another minor comment. Will await movement on that one.