Rare project developers

[Alexander Gall <>]

Hi

On Wed, 13 Apr 2022 10:00:13 +0200, mc36 <> said:

> On 4/13/22 09:50, Alexander Gall wrote:
>>> please re-add sticky-onlychanged, but forget sticky-persistent, but
>>> add this new knob sticky-extpersist /etc/freertr/p4-profile...
>>> imho it'll do the trick...
>> AFAICT, this behaves exactly like what I described in my previous mail
>> with the exception that in case 2, i.e. the most common case, the boot
>> log shows (as opposed to no message)
>> 
>> 2022-04-13 07:07:06 info cfgInit.executeSWcommands:cfgInit.java:750 line 
>> 123: "alias exec tna-set-profile sticky-param CPE" : % already selected
>> 2022-04-13 09:07:06 error cfgInit.doInit:cfgInit.java:956 1 errors found
>> 
>> I'm not sure if this is an improvement?
>> 
> :))) it was a bug, but it's one is gone with this change:
> https://github.com/mc36/freeRouter/commit/38ab07dd558e5a624f27a8333146cb1522504d4f

>> OTOH, it avoids the unnecessary process restart when setting the
>> profile to what it already is, but I find that not very important.
>> 
>> I tend to favor the variant from my last mail (i.e. no "onlychanged"
>> and neither "persistent" nor "extpersist"). Or am I overlooking
>> something?
> it also fixes the first case (unsaved change & reload)...
> also it eliminates the need for a preseed profile from the installer...
> (it's enough to have the sticky parameter in the -sw.txt)

That was also the case for the other method :) Now the only difference
is that setting the profile to the current value is a true NOP. That's
an improvement, so I'd like to keep that as the final version for the
first release.

I'll commit these changes and create another test release (called
"1theta").  When everything works as expected and there are no
objections I will create production release #1 from it.

-- 
Alex