cli merge requestshttps://dev.funkwhale.audio/funkwhale/cli/-/merge_requests2020-04-09T13:37:34Zhttps://dev.funkwhale.audio/funkwhale/cli/-/merge_requests/8Fix hack incompatibility with cryptfile keyring2020-04-09T13:37:34ZAdrian PetrescuFix hack incompatibility with cryptfile keyringI'm not exactly sure what this hack was ever supposed to accomplish, but it's clearly unnecessary for dummy keyrings like `keyrings.alt` or `keyrings.cryptfile`. When I try to login, the CLI just crashes with an `AttributeError` on `keyr...I'm not exactly sure what this hack was ever supposed to accomplish, but it's clearly unnecessary for dummy keyrings like `keyrings.alt` or `keyrings.cryptfile`. When I try to login, the CLI just crashes with an `AttributeError` on `keyring.backend.get_all_keyring.__wrapped__.always_returns`, which is what the hack is trying to remove anyway.
So this patch just skips trying to `del` the attribute if it already doesn't exist.backlog