dev-python/pillow seems to be the default (and in the latest one: only) provider for virtual/python-imaging. However, with reviewboard I'm getting errors about loading the backends: 2013-10-14 14:17:47,924 - ERROR - - Error loading authentication backend ldap: PIL Traceback (most recent call last): File "/usr/lib64/python2.7/site-packages/reviewboard/accounts/backends.py", line 515, in get_registered_auth_backends yield entry.name, entry.load() File "/usr/lib64/python2.7/site-packages/pkg_resources.py", line 2259, in load if require: self.require(env, installer) File "/usr/lib64/python2.7/site-packages/pkg_resources.py", line 2272, in require working_set.resolve(self.dist.requires(self.extras),env,installer))) File "/usr/lib64/python2.7/site-packages/pkg_resources.py", line 626, in resolve raise DistributionNotFound(req) DistributionNotFound: PIL 2013-10-14 14:22:48,906 - ERROR - - Error loading authentication backend x509: PIL Traceback (most recent call last): File "/usr/lib64/python2.7/site-packages/reviewboard/accounts/backends.py", line 515, in get_registered_auth_backends yield entry.name, entry.load() File "/usr/lib64/python2.7/site-packages/pkg_resources.py", line 2259, in load if require: self.require(env, installer) File "/usr/lib64/python2.7/site-packages/pkg_resources.py", line 2272, in require working_set.resolve(self.dist.requires(self.extras),env,installer))) File "/usr/lib64/python2.7/site-packages/pkg_resources.py", line 626, in resolve raise DistributionNotFound(req) DistributionNotFound: PIL Removing dev-python/pillow and installing dev-python/imaging makes things work again.
I have had the same issue since Pillow is not supported by reviewboard. Below an upstream discussion just for reference: https://groups.google.com/forum/#!searchin/reviewboard-dev/pillow/reviewboard-dev/pTRQOVPsC08/FchpEkyYqlwJ
(In reply to Julien Sanchez from comment #1) > I have had the same issue since Pillow is not supported by reviewboard. > > Below an upstream discussion just for reference: > > https://groups.google.com/forum/#!searchin/reviewboard-dev/pillow/ > reviewboard-dev/pTRQOVPsC08/FchpEkyYqlwJ reviewboard-1.7.7.1 was the very first version I added to portage last June. On re-visiting it its deps are problematic and problem ridden. To bump it to 1.7.24 requires a now old version of django-pipeline (>=1.2.24,<1.3) which has never been in portage. The lack of any bumping of Djblets in past months has further degraded the viablilty and value of this package. There are some rc versions of a major version revision (2.0). Frankly I think this warrants masking @ minimum but it's not me who uses it. Requires more input
(In reply to Ian Delaney from comment #2) > reviewboard-1.7.7.1 was the very first version I added to portage last June. > On re-visiting it its deps are problematic and problem ridden. To bump it > to 1.7.24 requires a now old version of django-pipeline (>=1.2.24,<1.3) > which has never been in portage. As previously explained, upstream restricts the deps like this in order to keep python compatibility with old python versions. Since we don't have to worry about those old versions, we can just remove the restriction. > > Frankly I think this warrants masking @ minimum but it's not me who uses it. > Requires more input Then why add it to the tree in the first place?
(In reply to Michael Palimaka (kensington) from comment #3) > (In reply to Ian Delaney from comment #2) > > reviewboard-1.7.7.1 was the very first version I added to portage last June. > > On re-visiting it its deps are problematic and problem ridden. To bump it > > to 1.7.24 requires a now old version of django-pipeline (>=1.2.24,<1.3) > > which has never been in portage. > As previously explained, upstream restricts the deps like this in order to > keep python compatibility with old python versions. Since we don't have to > worry about those old versions, we can just remove the restriction. > > > > > Frankly I think this warrants masking @ minimum but it's not me who uses it. > > Requires more input > Then why add it to the tree in the first place? That wasn't the question. I think it was because you or someone asked for it, in fact it think it was my former mentor. It's added and if you indicate you don't actually use it then it possibly warrants rm or masking, and I didn't look for us to be made maintainers.
We're using this at work and I'd hate to see it go. Unfortunately I can't offer to maintain it :-(
There's no reason for the package to be masked or removed.
(In reply to Michael Palimaka (kensington) from comment #6) > There's no reason for the package to be masked or removed. how about it doesn't build?
# Ian Delaney <idella4@gentoo.org> (01 Aug 2013) # Mask while awaiting bump to dep Djblets-0.7.16 =dev-util/reviewboard-1.7.12 how does that make sense anymore?
(In reply to Julian Ospald (hasufell) from comment #7) > (In reply to Michael Palimaka (kensington) from comment #6) > > There's no reason for the package to be masked or removed. > > how about it doesn't build? That sqlite issue was not previously encountered, thanks for filing the bug about it. (In reply to Julian Ospald (hasufell) from comment #8) > # Ian Delaney <idella4@gentoo.org> (01 Aug 2013) > # Mask while awaiting bump to dep Djblets-0.7.16 > =dev-util/reviewboard-1.7.12 > > how does that make sense anymore? It never made sense, who knows why it was bumped with dependencies that cannot be satisfied in the first place.
(In reply to Julian Ospald (hasufell) from comment #8) > # Ian Delaney <idella4@gentoo.org> (01 Aug 2013) > # Mask while awaiting bump to dep Djblets-0.7.16 > =dev-util/reviewboard-1.7.12 > > how does that make sense anymore? It never made sense, who knows why it was bumped with dependencies that cannot be satisfied in the first place. I really don't wish to have to explain any more. *reviewboard-1.7.28 (21 Sep 2014) 21 Sep 2014; Ian Delaney <idella4@gentoo.org> +files/1.7.28-docs.patch, +reviewboard-1.7.28.ebuild, -files/docs.patch, -reviewboard-1.7.7.1-r1.ebuild, -reviewboard-1.7.7.1.ebuild: bump; ebuild based on graaf's verion from his overlay with additions to the doc builds, thx graaf. Remove old ebuilds and patch, see Bug 522472
