pmatilai left a comment (rpm-software-management/rpm#3750)
Read-only home dir is not an issue at all: the migration-in-progress flag file
creation fails, and thus the migration never starts at all. Actually the flag
file probably simplifies the cleanup code because we need to guess less about
the circumstances on cleanup.
It also occurred to me like at some point over the weekend (but no I didn't
think about this all weekend :laughing: ): we should have that migration flag
file to protect against parallel execution, which could otherwise go sideways
in various interesting ways if an unsuspecting user runs multiple rpm instances
simultaneously. Chances for that might not be high but we've been seeing these
kind of issues filed increasingly in recent years.
The bigger question is whether to do it automatically or not, and that's up to
discussion I think. The motivation for wanting fully automatic migration was to
be able to rely on the XDG directory presence for storing auto-signing keys (so
#2678 and #3522) in an rpm-specific location. Since this wasn't done at that
point, I needed to do it differently, and so (one of) the main motivation for
auto-migration is no longer there.
I'm now in this endless loop of:
1. telling each and every rpm user to run "rpm --migraterc" when we could just
do it seems mindnumbingly silly
2. just doing it automatically will inevitably run into a case we didn't think
of and handle -> keeping it manual is simple and safe
3. jump back to 1.
Thoughts welcome :smile:
We'll keep the migration code in C++ anyway so you can go ahead with the flag
file thing regardless of the manual/auto decision though.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3750#issuecomment-2890022227
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/3750/[email protected]>
_______________________________________________
Rpm-maint mailing list
[email protected]
https://lists.rpm.org/mailman/listinfo/rpm-maint
