Bug 210775

Comment 1 VK 2016-07-08 19:34:26 UTC

Thanks for the patch. Please confirm the patch passes QA (poudriere, portlint).

It certainly passes portlint. poudriere I'd have to install and configure first, I could check tomorrow.

Comment 3 Jan Beich 2016-07-09 12:06:02 UTC

This breaks www/py-flexget beyond bug 204729.

$ flexget execute
2016-07-09 13:03 CRITICAL plugin                        Plugin `flexget.plugins.parsers.parser_guessit` failed to import dependencies
2016-07-09 13:03 ERROR    plugin                        No module named containers
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/flexget/plugin.py", line 386, in _load_plugins_from_dirs
    __import__(module_name)
  File "/usr/local/lib/python2.7/site-packages/flexget/plugins/parsers/parser_guessit.py", line 9, in <module>
    from guessit.containers import PropertiesContainer, NoValidator
ImportError: No module named containers

Comment 4 Jan Beich 2016-07-09 12:14:20 UTC

Comment on attachment 172058 [details]
py-guessit update to 2.0.5

Like devel/py-rebulk you may need to drop setup_requires from setup.py.

=======================<phase: configure      >============================
===>   py27-guessit-2.0.5 depends on package: py27-setuptools27>0 - found
===>   py27-guessit-2.0.5 depends on file: /usr/local/bin/python2.7 - found
===>  Configuring for py27-guessit-2.0.5
Download error on https://pypi.python.org/simple/pytest-runner/: [Errno 49] Can't assign requested address -- Some packages may not be found!
Couldn't find index page for 'pytest-runner' (maybe misspelled?)
Download error on https://pypi.python.org/simple/: [Errno 49] Can't assign requested address -- Some packages may not be found!
No local packages or download links found for pytest-runner
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "setup.py", line 80, in <module>
    setup(**args)
  File "/usr/local/lib/python2.7/distutils/core.py", line 111, in setup
    _setup_distribution = dist = klass(attrs)
  File "/usr/local/lib/python2.7/site-packages/setuptools/dist.py", line 269, in __init__
    self.fetch_build_eggs(attrs['setup_requires'])
  File "/usr/local/lib/python2.7/site-packages/setuptools/dist.py", line 313, in fetch_build_eggs
    replace_conflicting=True,
  File "/usr/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 826, in resolve
    dist = best[req.key] = env.best_match(req, ws, installer)
  File "/usr/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 1092, in best_match
    return self.obtain(req, installer)
  File "/usr/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 1104, in obtain
    return installer(requirement)
  File "/usr/local/lib/python2.7/site-packages/setuptools/dist.py", line 380, in fetch_build_egg
    return cmd.easy_install(req)
  File "/usr/local/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 658, in easy_install
    raise DistutilsError(msg)
distutils.errors.DistutilsError: Could not find suitable distribution for Requirement.parse('pytest-runner')
*** [do-configure] Error code 1

Created attachment 172318 [details]
dropped setup_requires from setup.py

Dropped setup_requires from setup.py as per Jan Beich's suggestion.

Created attachment 172319 [details]
poudriere log

Created attachment 172320 [details]
poudriere-portlint log

Should be ok now. As for breaking flexget - it(flexget) could really use an update, they moved on to guessit >2.0 in the new versions, but there are also 
some dependencies there aren't ports for, mostly for the webui and also configuration changes in the new versions.

Comment 9 commit-hook 2016-07-18 19:07:22 UTC

A commit references this bug:

Author: jbeich
Date: Mon Jul 18 19:06:41 UTC 2016
New revision: 418733
URL: https://svnweb.freebsd.org/changeset/ports/418733

Log:
  multimedia/py-guessit: update to 2.0.5

  Changes:	https://pypi.python.org/pypi/guessit#history
  PR:		210775
  Submitted by:	Andrej Ebert <andrej@ebert.su>
  Approved by:	maintainer timeout (2 weeks)

Changes:
  head/multimedia/py-guessit/Makefile
  head/multimedia/py-guessit/distinfo

Comment 10 Jan Beich 2016-07-18 19:16:36 UTC

Thanks. Landed.