Hi Adrian,

On 08.11.2019 13:05, Adrian Schmutzler wrote:
This provides a uci system config setting that will be set only
during initial setup. This can be used by uci-defaults script to
determine whether they are run during initial setup or after a
sysupgrade.

Since the setting is removed again after uci-defaults have been
processed, it won't be recognized by the user on the running device,
but can be exploited also for downstream setup tasks.

This looks for me like a misuse of uci configuration and some kind of workaround for a missing feature, maybe in procd/ubus?

NAK on this one from me.

--
Cheers,
Piotr


Signed-off-by: Adrian Schmutzler <freif...@adrianschmutzler.de>
---
  package/base-files/files/bin/config_generate                  | 1 +
  .../base-files/files/etc/uci-defaults/90_end-initial-setup    | 4 ++++
  2 files changed, 5 insertions(+)
  create mode 100644 
package/base-files/files/etc/uci-defaults/90_end-initial-setup

diff --git a/package/base-files/files/bin/config_generate 
b/package/base-files/files/bin/config_generate
index b473eba9e9..273561229a 100755
--- a/package/base-files/files/bin/config_generate
+++ b/package/base-files/files/bin/config_generate
@@ -243,6 +243,7 @@ generate_static_system() {
                set system.@system[-1].ttylogin='0'
                set system.@system[-1].log_size='64'
                set system.@system[-1].urandom_seed='0'
+               set system.@system[-1].initial_setup='1'
delete system.ntp
                set system.ntp='timeserver'
diff --git a/package/base-files/files/etc/uci-defaults/90_end-initial-setup 
b/package/base-files/files/etc/uci-defaults/90_end-initial-setup
new file mode 100644
index 0000000000..779d858d5f
--- /dev/null
+++ b/package/base-files/files/etc/uci-defaults/90_end-initial-setup
@@ -0,0 +1,4 @@
+uci -q delete system.@system[0].initial_setup
+uci commit system
+
+exit 0



_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to