Davisp python-spidermonkey

Comments and changes to this ticket

• 

  Paul J. Davis April 4th, 2009 @ 10:04 PM

  • State changed from “new” to “open”

  (from [e5ae70c015ff0d6b3142018e2f90af788e08d9da]) Patch for ticket #6.

  setup.py was failing to parse nspr-config output. This should fix for at least the error that was reported.

  [#6 state:open] http://github.com/davisp/python-...

• 

  Paul J. Davis April 4th, 2009 @ 10:05 PM

  Committed a fix, if it still breaks can you send me the output of:

  $ nspr-config --cflags --libs
  
  

  Thanks for the report, Paul Davis

• You flagged this item as spam.
  

  Eric Hanson April 4th, 2009 @ 10:09 PM

  eric@zephyr ~/projects/gharvest2/python-spidermonkey $ sudo python setup.py build Traceback (most recent call last): File "setup.py", line 111, in

  **platform_config()
  
  

  File "setup.py", line 54, in platform_config

  config = nspr_config()
  
  

  File "setup.py", line 47, in nspr_config

  ret[name].append(b[trim:])
  
  

  KeyError: 'extra_compile_args' eric@zephyr ~/projects/gharvest2/python-spidermonkey $ nspr-config --cflags --libs -I/usr/include/nspr -Wl,-R/usr/lib64/nspr -L/usr/lib64/nspr -lplds4 -lplc4 -lnspr4 -lpthread -ldl eric@zephyr ~/projects/gharvest2/python-spidermonkey $

• 

  Paul J. Davis April 4th, 2009 @ 10:16 PM

  (from [1c9d0cc16055cbf4d27bd5cf12676450f2e3064b]) I am blind.

  Forgot to add a member for extra_compile_args to the nspr config return.

  [#6] http://github.com/davisp/python-...

• 

  Paul J. Davis April 4th, 2009 @ 10:18 PM

  (from [83f81c48b6e842e06082faa035fd53e272e05ae0]) Updates #6

  I'm on commit three so far. Hopefully this finishes it.

  [#6] http://github.com/davisp/python-...

• 

  Paul J. Davis April 4th, 2009 @ 10:20 PM

  Oh, I should totally have checked the gcc manual before changing that, ignore the last two commits before trying again.

• 

  Paul J. Davis April 4th, 2009 @ 10:26 PM

  (from [0f1b176591c36c63e132ba052f187b6d442ca918]) Hopefully the real fix for #6

  In the future I should not assume that a bug is a single commit. And also I should read relevant manuals instead of assuming what options do.

  -Wl is a linker option pass through, not a compile warning. Added info the relevant config updates.

  [#6] http://github.com/davisp/python-...

• 

  Paul J. Davis April 4th, 2009 @ 10:27 PM

  Now can you try again? I should've checked the man page and not have assumed that -Wanything was a warning option.

• You flagged this item as spam.
  

  Eric Hanson April 5th, 2009 @ 03:57 AM

  Hey Paul, I'm getting a similar error after deleting the repository and checkout out again:

  
  eric@zephyr ~/projects/gharvest2 $ git clone git://github.com/davisp/python-spidermonkey.git
  Initialized empty Git repository in /home/eric/projects/gharvest2/python-spidermonkey/.git/
  remote: Counting objects: 674, done.
  remote: Compressing objects: 100% (645/645), done.
  remote: Total 674 (delta 361), reused 110 (delta 22)
  Receiving objects: 100% (674/674), 1.03 MiB | 282 KiB/s, done.
  Resolving deltas: 100% (361/361), done.
  eric@zephyr ~/projects/gharvest2 $ cd python-spidermonkey/
  eric@zephyr ~/projects/gharvest2/python-spidermonkey $ 
  eric@zephyr ~/projects/gharvest2/python-spidermonkey $ sudo python setup.py build
  Traceback (most recent call last):
    File "setup.py", line 119, in <module>
      **platform_config()
    File "setup.py", line 62, in platform_config
      config = nspr_config()
    File "setup.py", line 54, in nspr_config
      name, trim = prfx[b[:2]]
  KeyError: '-W'
  

• 

  Paul J. Davis April 5th, 2009 @ 11:57 AM

  I'm generally not this blind, but there's a new commit for you to try.

• You flagged this item as spam.
  

  Eric Hanson April 5th, 2009 @ 04:26 PM

  That worked!

  On a side note, it complained on

  
  python setup.py test
  

  because I didn't have nose installed, might want to add that to requirements.

  Thanks! Eric

• 

  Paul J. Davis April 5th, 2009 @ 04:30 PM

  • State changed from “open” to “resolved”