[Starlingx-discuss] [multi-os] replace "/etc" in config files by a variable "sysconfdir"

[Penney, Don]

There are a set of defined RPM macros already available:
https://fedoraproject.org/wiki/Packaging:RPMMacros

For example, /etc should be replaced with %{_sysconfdir} in the spec files.


From: Hu, Yong [mailto:yong.hu at intel.com]
Sent: Friday, September 28, 2018 8:24 PM
To: starlingx-discuss at lists.starlingx.io
Subject: [Starlingx-discuss] [multi-os] replace "/etc" in config files by a variable "sysconfdir"

As a part of initiatives, I would like to suggest to replace “/etc”, which is hardcoded in some StarlingX config files,
by a variable something like “sysconfdir”, which would be “instantiated” a bit later upon the OS type (CentOS, ClearLinux or Ubuntu).

Taking “stx-nfv/nfv/nfv-vim/nfv_vim/config.ini” as an example,
“/etc” is coded in this “config.ini” directly to refer other config files:
config_file=/etc/nfv/nfv_plugins/alarm_handlers/config.ini
config_file=/etc/nfv/nfv_plugins/nfvi_plugins/config.ini

My recommendation is to use a variable “sysconfdir” as instead, and to “define” this variable on later stage,
like in “nfv.spec” for CentOS, in which “sysconfdir” is “/etc”.

The benefit I see here is in other OSs, such as Clear Linux in which we can set different value,
for example, “/usr/local/etc” for this variable “sysconfdir”.

Should you have any questions or feedback, please let me know.
If I don’t see objections with this kind of changes, I will make some patches accordingly afterward.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.starlingx.io/pipermail/starlingx-discuss/attachments/20181001/84dfad18/attachment-0001.html>