Ignore extension suffixes if the interpreter is Python 2

[ian-hurst]

TODO:

• Write new tests or update the old ones to cover new functionality.
• Update doc-strings where appropriate.
• Update or write new documentation in packit/packit.dev.

Fixes Issue 17

RELEASE NOTES BEGIN

Ignore extension suffixes if the interpreter is Python 2

RELEASE NOTES END

[softwarefactory-project-zuul]

Build succeeded.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/8e917fdaeb02424b88d5275ed6880a0c

✔️ pre-commit SUCCESS in 1m 26s

[nforro]

Thanks! I have a few remarks.

[softwarefactory-project-zuul]

Build succeeded.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/631df27a192642b5b5a54db70368b210

✔️ pre-commit SUCCESS in 1m 28s

[nforro]

Great! Just two nitpicks. The commit message should describe the changes, although it can also include a reference to a GitHub issue, such info is less than useful to someone reading the commit message in their offline local clone.

[softwarefactory-project-zuul]

Build succeeded.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/b68bfe5e09594f58a4c6020195557118

✔️ pre-commit SUCCESS in 1m 29s

[ian-hurst]

Great! Just two nitpicks. The commit message should describe the changes, although it can also include a reference to a GitHub issue, such info is less than useful to someone reading the commit message in their offline local clone.

Ok, should be set now. Thanks for the review again, and let me know if I should do anything else.

[nforro]

Thanks!

Fix get_system_sitepackages_and_suffixes() on systems where /usr/libexec/platform-python is python 2.7 (issue 17)

Commit message title shouldn't exceed 72 characters, and even though we don't enforce that limit, this is a bit too long. How about:

Fix `get_system_sitepackages_and_suffixes()` on systems where `platform-python` is Python 2

or maybe even:

Ignore extension suffixes if the interpreter is Python 2

I would skip the issue reference (it's in the PR description where it is actually relevant), but if you really want to include it, place it in the body, not title.

[ian-hurst]

I would skip the issue reference (it's in the PR description where it is actually relevant), but if you really want to include it, place it in the body, not title.

Sure, updated

[softwarefactory-project-zuul]

Build succeeded.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/8333fdd1de424e3b990532d152cdb499

✔️ pre-commit SUCCESS in 1m 33s

[softwarefactory-project-zuul]

Build succeeded (gate pipeline).
https://softwarefactory-project.io/zuul/t/packit-service/buildset/358c8f28cebb4d7cb95647364a6ca264

✔️ pre-commit SUCCESS in 1m 34s