Opened 12 years ago

Closed 12 years ago

#33357 closed defect (fixed)

pyopencl-py27 missing dependency to pytools

Reported by: martin.b.eriksen@… Owned by: larryv (Lawrence Velázquez)
Priority: Normal Milestone:
Component: ports Version: 2.0.3
Keywords: Cc: michelle.lynn.gill@…, drkp (Dan Ports)
Port: py-pyopencl

Description (last modified by mf2k (Frank Schima))

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 (1)

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

Download all attachments as: .zip

Change History (9)

comment:1 Changed 12 years ago by mf2k (Frank Schima)

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

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

Changed 12 years ago by michelle.lynn.gill@…

Attachment: Portfile.diff added

comment:2 Changed 12 years 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 12 years ago by michelle.lynn.gill@…

Cc: michelle.lynn.gill@… added

Cc Me!

comment:4 Changed 12 years ago by larryv (Lawrence Velázquez)

The patch looks good.

comment:5 in reply to:  4 ; Changed 12 years 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 12 years ago by larryv (Lawrence Velázquez)

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 12 years ago by drkp (Dan Ports)

Cc: dports@… added

Cc Me!

comment:8 Changed 12 years ago by drkp (Dan Ports)

Resolution: fixed
Status: newclosed

Thanks, committed in r91711.

Note: See TracTickets for help on using tickets.