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

wand.image import failed on imagemagick 6.9.6.2 #310

Open
changwoo opened this Issue Oct 22, 2016 · 1 comment

Comments

2 participants
@changwoo

changwoo commented Oct 22, 2016

From https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=841548

  Traceback (most recent call last):
    File "<string>", line 1, in <module>
    File "/usr/lib/python3/dist-packages/wand/image.py", line 21, in <module>
      from .color import Color
    File "/usr/lib/python3/dist-packages/wand/color.py", line 12, in <module>
      from .version import QUANTUM_DEPTH
    File "/usr/lib/python3/dist-packages/wand/version.py", line 95, in <module>
      *map(int, MAGICK_RELEASE_DATE_STRING.split('-')))
  TypeError: Required argument 'month' (pos 2) not found

The format of the release date seems to be changed.

@changwoo

This comment has been minimized.

changwoo commented Oct 22, 2016

tests/misc_test.py:test_version() should be fixed too.

_________________________________ test_version _________________________________

    def test_version():
        """Test version strings."""
        match = re.match('^ImageMagick\s+\d+\.\d+\.\d+(?:-\d+)?', MAGICK_VERSION)
        assert match
        assert isinstance(MAGICK_VERSION_INFO, tuple)
        assert (len(MAGICK_VERSION_INFO) ==
                match.group(0).count('.') + match.group(0).count('-') + 1)
        assert all(isinstance(v, int) for v in MAGICK_VERSION_INFO)
        assert isinstance(MAGICK_VERSION_NUMBER, numbers.Integral)
        assert isinstance(MAGICK_RELEASE_DATE, datetime.date)
>       assert (MAGICK_RELEASE_DATE_STRING ==
                MAGICK_RELEASE_DATE.strftime('%Y-%m-%d'))
E       assert '20161010' == '2016-10-10'
E         - 20161010
E         + 2016-10-10
E         ?     +  +

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment