Definition of host parameter

[MarcusZuber]

We have multiple ways to define the host-parameter: the param-default value, the class-paramters, passed to the plugin-manager and the environment variable.

In the current implementation, the parmeters from the plugin-manager are completly ignored (which is not good. currently I'm fiddeling around with environment variables in concert, since I can not define it in the constructor).
Currently https://github.com/ufo-kit/uca-net/blob/master/uca-net-camera.c#L618 reads the environment variable and depending on its exitence sets the host property to its value or to "localhost".

Now the question: Which configuration should be stronger?
I would suggest that a set paramter via the plugin-manager should not be overwritten from an environment-variable. If it isnot set via the manager we check the environment variable and if both are not set we set it to localhost.

Why should we set it to localhost as default? The only use-case is for running local tests.

[MarcusZuber]

When you run the example from the readme

from concert.devices.cameras.uca import Camera
camera = Camera('net', {'host': 'foo.bar:1234'})

you will get an "can not connect to 'localhost' error.

[matze]

Of course I do not have any final say into that but to me, environment variables usually override any build time configuration otherwise they are pretty useless. But not taking the property value is indeed stupid.

[tfarago]

So as I understand Marcus will take care of this?

[MarcusZuber]

Sure. I will come up with a PR today or tomorrow.