Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dev-util/vint: verbump to 0.3.14 and related changes #5708

Closed
wants to merge 2 commits into from
Closed

dev-util/vint: verbump to 0.3.14 and related changes #5708

wants to merge 2 commits into from

Conversation

Coacher
Copy link
Contributor

@Coacher Coacher commented Sep 15, 2017

Add virtual/python-typing similarly to how we have virtual/python-enum34, etc. packages.
vint-0.3.14 requires the typing Python module and virtual/python-typing makes dependencies easy.

@gentoo-repo-qa-bot gentoo-repo-qa-bot added new package The PR is adding a new package. assigned PR successfully assigned to the package maintainer(s). labels Sep 15, 2017
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull Request assignment

Areas affected: ebuilds
Packages affected: dev-python/typing, dev-util/vint, virtual/python-typing

dev-python/typing: @alunduil
dev-util/vint: @Coacher, @gentoo/proxy-maint
virtual/python-typing: @gentoo/proxy-maint (new package)

@Coacher
Copy link
Contributor Author

Coacher commented Sep 21, 2017

Rebased and updated PR to resolve merge conflicts.

This module is shipped with >=dev-lang/python-3.5.
dev-python/typing provides a backport for Pythons that don't have it.
KEYWORDS are intersection of dev-python/typing and dev-lang/python.

Closes: https://bugs.gentoo.org/628378
Package-Manager: Portage-2.3.8, Repoman-2.3.3
Closes: https://bugs.gentoo.org/628380
Package-Manager: Portage-2.3.8, Repoman-2.3.3
@gentoo-repo-qa-bot
Copy link
Collaborator

👍 All QA issues have been fixed!


EAPI=6

PYTHON_COMPAT=( python{2_7,3_4,3_5,3_6} pypy{,3} )
Copy link
Member

@monsieurp monsieurp Sep 23, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct me if I'm wrong but since the typing module is shipped with Python 3.5 and 3.6, these versions shouldn't be listed here, should they?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They should. This allows a package, which supports e.g. python2.7 and python3.5, depend on virtual-typing. Otherwise it would be necessary to create a conditional dep for pythons < 3.5, which defies the purpose of this virtual. Check out virtual/python-enum34.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@monsieurp, ping. Please re-review.

Copy link
Member

@monsieurp monsieurp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok. Thanks for clearing up my doubts. Merging underway.

@Coacher
Copy link
Contributor Author

Coacher commented Sep 29, 2017

Thank you, @monsieurp.

@Coacher Coacher deleted the vint-0.3.14 branch September 29, 2017 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). new package The PR is adding a new package.
Projects
None yet
3 participants