Attention is currently required from: jolly, laforge.
View Change
1 comment:
Patchset:
Patch Set #5:
Actually I don't see why we should use sync()/fsync() at all? As you mentioned i a call, modern file […]
I'm fine with whatever means not adding a new thread to libosmocore just for this specific case used seldomly.
One can also use the vty cmd to write the config file to another file, and then over ssh copy over the new file to the old file in whatever sync/safe manner they want.
To view, visit change 37798. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-MessageType: comment
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I3cb2ee68b2e4c730f96522208c4abf00d0f49a44
Gerrit-Change-Number: 37798
Gerrit-PatchSet: 5
Gerrit-Owner: jolly <andreas@eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-CC: Hoernchen <ewild@sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy@sysmocom.de>
Gerrit-CC: laforge <laforge@osmocom.org>
Gerrit-Attention: jolly <andreas@eversberg.eu>
Gerrit-Attention: laforge <laforge@osmocom.org>
Gerrit-Comment-Date: Fri, 23 Aug 2024 11:16:13 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: jolly <andreas@eversberg.eu>
Comment-In-Reply-To: pespin <pespin@sysmocom.de>