FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Use Kyiv instead of Kiev in test_email · Issue #109017 · python/cpython · GitHub

Repository navigation

Use Kyiv instead of Kiev in test_email #109017

Closed

Description

Bug report

Bug description:

As proposed in #108533 the time zone was renamed but we can't simply rename it in the test cause it fails on old(?) MacOS systems. I would propose to prepend it with:

@unittest.skipUnless(os.path.exists('/usr/share/zoneinfo/Europe/Kyiv') or
                         os.path.exists('/usr/lib/zoneinfo/Europe/Kyiv'),
                         "Can't find the Olson's TZ database")

CPython versions tested on:

3.11

Operating systems tested on:

Linux

Linked PRs

Activity

  1. corona10 commented on Sep 10, 2023

    Member

    @serhiy-storchaka Would you like to take a look since you are the Ukranian?

  2. added 2 commits that reference this issue on Sep 11, 2023
    9026723
    43a38cf
  3. serhiy-storchaka commented on Sep 11, 2023

    Member

    I do not like relying on the particular location of the database. Although it was already here in the existing code, the comment notes that it is not a robust way. Now we have zoneinfo in the stdlib, so perhaps it could be used.

  4. added a commit that references this issue on Sep 11, 2023
  5. jspricke commented on Sep 11, 2023

    ContributorAuthor

    @serhiy-storchaka thanks for the proposal, I've updated my PR.

  6. added a commit that references this issue on Sep 11, 2023
  7. added a commit that references this issue on Oct 24, 2023
  8. added 2 commits that reference this issue on Oct 24, 2023
  9. added 2 commits that reference this issue on Oct 24, 2023
  10. added a commit that references this issue on Feb 11, 2024
  11. added a commit that references this issue on Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

type-bugAn unexpected behavior, bug, or error

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions


    Back | FazBrowse Home | New Git URL