Re: setup.py issue - some files are included as intended, but one is not

[Dan Stromberg]

On Sat, Jan 11, 2014 at 2:04 PM, Dan Stromberg <drsa...@gmail.com> wrote:
> Hi folks.
>
> I have a setup.py problem that's driving me nuts.

Anyone? I've received 0 responses.

> I have a treap.py file that tries to "import * from pyx_treap.so", and
> failing that, it'll "import * from py_treap.py" (sans extensions of
> course). Naturally, all 3 of these should be included - although in
> the case of pyx_treap.so, it probably should be a .c that's included.
>
> It is also intended to include nest.py, which has a simple form.
>
> Well, treap.py, nest.py and pyx_treap.c are included fine, but I can't
> seem to get py_treap.py to be included for some reason.
>
> I get no errors during "python setup.py sdist upload", but upon
> "python $(which pip) install treap", I get:
> $ sudo /usr/bin/python $(which pip) install treap
> Downloading/unpacking treap
> Running setup.py egg_info for package treap
>
> file py_treap.py (for module py_treap) not found
> Installing collected packages: treap
> Running setup.py install for treap
> file py_treap.py (for module py_treap) not found
> file py_treap.py (for module py_treap) not found
>
> file py_treap.py (for module py_treap) not found
> file py_treap.py (for module py_treap) not found
> Successfully installed treap
> Cleaning up...
>
> And it's not successfully installed - py_treap.py is missing. The pyx
> code does its job, so the problem is masked, other than the messages
> above, and the absence of py_treap.py from
> /usr/local/lib/python2.7/dist-packages
>
> I can clearly see py_treap.py in ./dist/treap-1.35.tar.gz - it's at
> least getting packaged up that much. It's not listed in
> /usr/local/lib/python2.7/dist-packages/treap-1.31.egg-info/SOURCES.txt
> , but I can see it in
> /usr/local/lib/python2.7/dist-packages/treap-1.31.egg-info/top_level.txt
> .
>
> My setup.py is at:
> http://stromberg.dnsalias.org/svn/treap/trunk/setup.py
>
> I've tried that setup.py and several variations on that theme, but
> none seem to include py_treap.py .
>
> Please make some suggestions? How can I get py_treap.py included in
> the pip install?
>
> Thanks!

[Ethan Furman]

On 01/14/2014 01:26 PM, Dan Stromberg wrote:
> On Sat, Jan 11, 2014 at 2:04 PM, Dan Stromberg <drsa...@gmail.com> wrote:
>> Hi folks.
>>
>> I have a setup.py problem that's driving me nuts.
>
> Anyone? I've received 0 responses.

I have no answer, but forwarding to Distutils (hopefully it's an appropriate topic ;)

[Piet van Oostrum]

Dan Stromberg <drsa...@gmail.com> writes:

> On Sat, Jan 11, 2014 at 2:04 PM, Dan Stromberg <drsa...@gmail.com> wrote:
>> Hi folks.
>>
>> I have a setup.py problem that's driving me nuts.
>
> Anyone? I've received 0 responses.

I can't even install your code because there's a bug in it.

m4_treap.m4 contains this instruction twice:

ifdef(/*pyx*/,cp)if current is None:
ifdef(/*pyx*/,cp)raise KeyError

Which when generating pyx_treap.pyx (with *pyx* defined) expands to the syntactically incorrect

cpif current is None:
cpraise KeyError

--
Piet van Oostrum <pi...@vanoostrum.org>
WWW: http://pietvanoostrum.com/
PGP key: [8DAE142BE17999C4]

[Dan Stromberg]

On Wed, Jan 15, 2014 at 7:18 AM, Piet van Oostrum <pi...@vanoostrum.org> wrote:
> Dan Stromberg <drsa...@gmail.com> writes:
>
>> On Sat, Jan 11, 2014 at 2:04 PM, Dan Stromberg <drsa...@gmail.com> wrote:
>>> Hi folks.
>>>
>>> I have a setup.py problem that's driving me nuts.
>>
>> Anyone? I've received 0 responses.
>
> I can't even install your code because there's a bug in it.
>
> m4_treap.m4 contains this instruction twice:
>
> ifdef(/*pyx*/,cp)if current is None:
> ifdef(/*pyx*/,cp)raise KeyError
>
> Which when generating pyx_treap.pyx (with *pyx* defined) expands to the syntactically incorrect
>
> cpif current is None:
> cpraise KeyError

Apologies. I'd fixed that in my source tree, but not checked it in.

It's checked in now.