Max has posted comments on this change. ( https://gerrit.osmocom.org/12319 )
Change subject: Use absolute path for default config ...................................................................... Patch Set 1: > Patch Set 1: Code-Review-1 > > As far as I can tell, in all repos we have default cfg_file as $CWD/*.cfg, so > I'd rather keep same behavior for all osmocom projects. > > Another discussion is whether we want to move all osmocom projects to have > /etc/osmocom/*.cfg by default. It'd then make sense to by default check first > $CWD and if not found look for /etc/osmocom, to maintain it > backwards-compatible. The plan is to use this as login shell and I'm not sure what's $CWD in this case so it's better to use absolute path. -- To view, visit https://gerrit.osmocom.org/12319 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-sysmon Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I37c559af99872de9290335699e2118924ae2156a Gerrit-Change-Number: 12319 Gerrit-PatchSet: 1 Gerrit-Owner: Max <msur...@sysmocom.de> Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Reviewer: Max <msur...@sysmocom.de> Gerrit-Reviewer: Pau Espin Pedrol <pes...@sysmocom.de> Gerrit-Comment-Date: Fri, 14 Dec 2018 15:58:42 +0000 Gerrit-HasComments: No Gerrit-HasLabels: No
