[PATCH] base-files: sysupgrade: store status of system-services
Stijn Segers
stijn at volatilesystems.org
Mon Jan 11 08:38:41 EST 2021
Hi Alberto,
Alberto Bursi <bobafetthotmail at gmail.com> schreef op 11 januari 2021 03:56:23 CET:
>
>
>On 10/01/21 22:50, Stijn Segers wrote:
>> Hi Sven,
>>
>> Op zondag 10 januari 2021 om 22u28 schreef Sven Roederer
>> <devel-sven at geroedel.de>:
>>> Am Samstag, 9. Januar 2021, 12:28:31 CET schrieb Stijn Segers:
>>>> > @@ -228,6 +229,7 @@ do_save_conffiles() {
>>>> >
>>>> > if [ "$SAVE_INSTALLED_PKGS" -eq 1 ]; then
>>>> > echo "${INSTALLED_PACKAGES}" >> "$CONFFILES"
>>>> > + echo "${SERVICE_STATUS}" >> "$CONFFILES"
>>>> > mkdir -p "$ETCBACKUP_DIR"
>>>>
>>>> Am I reading this correctly and is this only keeping track of service
>>>> status if you tell sysupgrade to save packages? What's the rationale
>>>> behind that?
>>>>
>>>> I have a personal build with all packages preinstalled, so I don't need
>>>> that. Would like to keep track of service status though. Can those two
>>>> things be entangled?
>>>>
>>>
>>> Stijn,
>>>
>>> my intention was to not change the current behavior by default, so an
>>> extra
>>> switch or extending an existing switch looked like the way. I've
>>> choosen the
>>> lazy one, based on "when the user is storing the packages-list, he is
>>> for sure
>>> interested in the services".
>>> But I'm happy to add a separate switch to sysupgrade. Any preference
>>> of the
>>> letter? What about using "-s"?
>>>
>>> Sven
>>
>> Yes, that's still free and the most intuitive I think.
>>
>> Thanks!
>>
>> Stijn
>>
>
>Since we (me, Andre Heider and Paul Spooren) are discussing/asking about
>enabling this by default, do you have any opinion on that?
>
>-Alberto
That would be great, as Adrian pointed out it's something you'd expect would be saved when you tell sysupgrade to keep settings.
So +1 from me on making it default. All my AP/testing devices now have scripting to disable e.g. DHCP and DNS again after an upgrade.
Stijn
>
>_______________________________________________
>openwrt-devel mailing list
>openwrt-devel at lists.openwrt.org
>https://lists.openwrt.org/mailman/listinfo/openwrt-devel
--
Verstuurd vanaf mijn Android apparaat met K-9 Mail. Excuseer mijn beknoptheid.
More information about the openwrt-devel
mailing list