search

bkuner / opcUaUnifiedAutomation

EPICS opcUa device support with Unified Automation C++ based client sdk.
Other
0 stars 4 forks source link

Use standard locations for *.local overrides; .gitignore them (fixes #8) #19

Closed ralphlange closed 7 years ago

ralphlange commented 7 years ago

I'm proposing (again) to use the full set of locations for *.local overrides.

These changes are making the configuration files as close as possible to the files that are created using makeBaseApp.pl from the EPICS Base template.

It is perfectly fine to only mention one of the configuration options in the README and only use that option. However, please do not break a standard mechanism from Base for others that are using it.

(And please discuss a proposal before turning it down.)

bkuner commented 7 years ago

From which makeBaseApp.pl? You can put further includes to your .local file

ralphlange commented 7 years ago

The idea (and main feature) of the .local files is that you can configure a whole set of modules by putting definitions into a single .local file above the modules, without having to do changes in any of the modules. If you have to change the configuration inside all the modules to do that, you are losing most of that feature.

My suggestion is only changing the files to what you would get if you run makeBaseApp.pl on an empty directory. It is bringing the module closer to the standard.