Attention is currently required from: laforge, pespin. neels has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/pysim/+/39742?usp=email )
Change subject: personalization: refactor ConfigurableParameter, Iccid, Imsi ...................................................................... Patch Set 18: (1 comment) File pySim/esim/saip/personalization.py: https://gerrit.osmocom.org/c/pysim/+/39742/comment/7c69df23_a36ab212?usp=email : PS13, Line 50: r"""Base class representing a part of the eSIM profile that is configurable during the > > re the build failure, dear Vadim, on the contrary. […] about the stub idea: - if I have to add a fake non-abstract yet abstract stub to prevent ABC from doing what it was designed to do, it defies the point of ABC, right? =) Instead I can just remove the @abstractmethod decorator to achieve the same goal. - I can adjust these classes here, but i want to keep other peoples' subclasses that may be out there intact, too. I would like to add the new classmethod in an *optional* way, leaving the transition to the new features up to the implementing classes -- transitions are shown in the subsequent commits. As part of this discussion, I'd also like to mention from the future: In a later patch, we generate SdKey classes by using `type('Xxx',...)` instead of `class Xxx:` According to python docs, they are identical: https://docs.python.org/3/library/functions.html#type "The following two...create identical..." However, it turns out that, as soon as ConfigurableParameter inherits from ABC, the newly created `SdKeyXxx.__module__` member suddenly points to module `abc`, not to `pySim.esim.saip.personalization` where it was actually created, and I have to set the `__module__` member explicitly to make things work again. (I don't really like generating classes very much, we should be generating class *instances* instead, but the choice is again to stay backwards compatible with the existing API) So, in all honesty, my opinion is still that ... well ... that abc serves no purpose and doesn't work, as harsh as it may sound. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/39742?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I6522be4c463e34897ca9bff2309b3706a88b3ce8 Gerrit-Change-Number: 39742 Gerrit-PatchSet: 18 Gerrit-Owner: neels <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <[email protected]> Gerrit-Reviewer: laforge <[email protected]> Gerrit-Reviewer: neels <[email protected]> Gerrit-CC: pespin <[email protected]> Gerrit-Attention: laforge <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Comment-Date: Thu, 22 Jan 2026 23:53:23 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <[email protected]> Comment-In-Reply-To: laforge <[email protected]> Comment-In-Reply-To: fixeria <[email protected]> Comment-In-Reply-To: pespin <[email protected]>
