The __main__.py module breaks under Windows #177

Activity

1. slott56

   added

   enhancementNew feature or requestNew feature or request

   good first issueGood for newcomersGood for newcomers

   on May 21, 2026
2. Quantum0uasar commented on May 21, 2026

   

   Quantum0uasar

   on May 21, 2026

   Collaborator

   More actions

   Hi! I'd be happy to work on this fix.

   The issue is that os.getegid() (and os.getuid() / os.getgid() if used similarly) is a Unix-only call and raises AttributeError on Windows.

   My plan:

   1. Wrap the group_access and user_access calls in a platform check: if hasattr(os, 'getegid') (or if sys.platform != 'win32')
   2. On Windows, gracefully skip or return a sensible default for those attributes
   3. Update the test in tests/test_main.py to handle both platforms

   This should be a minimal, targeted fix. I'll open a draft PR shortly — let me know if you have a preference on how the Windows fallback should behave!
3. Rohan5commit

   added a commit that references this issue on May 21, 2026

   fix: make stat() Windows-compatible by guarding os.getegid/geteuid

   ...

   This issue will close once commit 26bec6d is merged into the 'main' branch.

   26bec6d
4. slott56 commented on May 21, 2026

   

   slott56

   on May 21, 2026

   CollaboratorAuthor

   More actions

   Sorry for not seeing this before the PR was submitted.

   An if using sys.platform is not the best approach. The use of hasattr() or a try/except block is going to work out more consistently across more OS's. See the https://docs.python.org/3/library/intro.html#notes-on-availability which is rather vague.
5. Quantum0uasar commented on May 21, 2026

   

   Quantum0uasar

   on May 21, 2026

   Last edited by Quantum0uasar

   Collaborator

   More actions

   Thanks for the feedback @slott56! You're right — using sys.platform (or platform.system()) is fragile.

   @Rohan5commit please update PR #178 to use a try/except AttributeError block instead, which is more robust and idiomatic:

   try:
       data["group_access"] = celtypes.BoolType(status.st_gid == os.getegid())  # type: ignore[attr-defined]
       data["user_access"] = celtypes.BoolType(status.st_uid == os.geteuid())  # type: ignore[attr-defined]
   except AttributeError:
       pass  # os.getegid() and os.geteuid() are not available on all platforms (e.g. Windows)

   Also remove import platform and the _IS_WINDOWS flag entirely — they are no longer needed. This approach handles any platform where those attributes are missing, not just Windows.
6. slott56

   assigned

   Quantum0uasar

   on May 22, 2026
7. Quantum0uasar

   added a commit that references this issue on May 24, 2026

   fix: use EAFP try/except for os.getegid/geteuid in stat() (issue clou…

   ...

   29c6a92
8. slott56

   closed this as completedin #188on Jul 2, 2026