New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #33357 (closed defect: fixed)

Opened 15 months ago

Last modified 14 months ago

pyopencl-py27 missing dependency to pytools

Reported by: martin.b.eriksen@… Owned by: larry.velazquez@…
Priority: Normal Milestone:
Component: ports Version: 2.0.3
Keywords: Cc: michelle.lynn.gill@…, dports@…
Port: py-pyopencl

Description (last modified by macsforever2000@…) (diff)

Trying to implement the example at http://documen.tician.de/pyopencl/, there is a problem importing a module named "pytools". Installing "sudo easy_install-2.7 pytools" solves the problem. Is there missing dependencies in the macports package?

[1]
Traceback (most recent call last):
  File "./a.py", line 25, in <module>
    """).build()
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pyopencl/__init__.py", line 113, in build
    from pyopencl.cache import create_built_program_from_source_cached
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pyopencl/cache.py", line 11, in <module>
    from pytools import Record

Attachments

Portfile.diff (708 bytes) - added by michelle.lynn.gill@… 14 months ago.

Change History

comment:1 Changed 15 months ago by macsforever2000@…

  • Owner changed from macports-tickets@… to larry.velazquez@…
  • Port set to py-pyopencl
  • Description modified (diff)

Are you saying the port builds fine but errors out at runtime?

Changed 14 months ago by michelle.lynn.gill@…

comment:2 Changed 14 months ago by michelle.lynn.gill@…

I can confirm that this is a runtime requirement. I get the same error upon running the example linked above. Installing pytools fixes the issue. I have created a ticket 33937 with a Portfile for pytools. I have also attached a diff of proposed (trivial) changes the Portfile for py-pyopencl, which adds py-pytools as a runtime dependency.

comment:3 Changed 14 months ago by michelle.lynn.gill@…

  • Cc michelle.lynn.gill@… added

Cc Me!

comment:4 follow-up: ↓ 5 Changed 14 months ago by larry.velazquez@…

The patch looks good.

comment:5 in reply to: ↑ 4 ; follow-up: ↓ 6 Changed 14 months ago by michelle.lynn.gill@…

Replying to larry.velazquez@…:

The patch looks good.

Thanks, I guess we need to wait for pytools to be committed as I don't have commit rights. I'll post here when I see this has been completed.

comment:6 in reply to: ↑ 5 Changed 14 months ago by larry.velazquez@…

Replying to michelle.lynn.gill@…:

Thanks, I guess we need to wait for pytools to be committed as I don't have commit rights.

Me neither. Makes being a maintainer a little difficult :)

comment:7 Changed 14 months ago by dports@…

  • Cc dports@… added

Cc Me!

comment:8 Changed 14 months ago by dports@…

  • Status changed from new to closed
  • Resolution set to fixed

Thanks, committed in r91711.

Note: See TracTickets for help on using tickets.