v0.5 is broken on MacOs

[dmakhno]

https://docs.python.org/2/library/multiprocessing.html#multiprocessing.Queue.qsize

Note that this may raise NotImplementedError on Unix platforms like Mac OS X where sem_getvalue() is not implemented.

I got

  File "/Users/user/Projects/vermilion/venv/lib/python2.7/site-packages/nosetimer/plugin.py", line 96, in report
    for i in range(_results_queue.qsize()):
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/multiprocessing/queues.py", line 143, in qsize
    return self._maxsize - self._sem._semlock._get_value()

It used to work perfectly with multiprocessing and xunitmp with mac, centos and ubuntu, now it fails totally and have to roll back to 0.4.4

[dmakhno]

@skudriashev, I cannot contribute to fix, I don't quite understand the diff

[mahmoudimus]

@dmakhno I will take a look when I get a chance. Is this blocking you now?

[dmakhno]

@mahmoudimus, Thanks... I reverted back to 0.4.4, it works absolutely fine for me since I don't have windows at all, and as I understand support of windows in 0.5 broken this.

[skudriashev]

@dmakhno, this change was done to make multiprocessing nose plugin work with timer plugin. There is nothing about Windows.

[skudriashev]

@dmakhno, btw, please look the the comment: "# Windows and Python 2.7 multiprocessing don't marry well.", so we need to check that we are not on Windows. Perhaps we need to skip Mac OS too.

[skudriashev]

@dmakhno, do you run the nose-timer with multiprocessing plugin?

[dmakhno]

@skudriashev, yes, 0.4.4 works just as fine. Here is jenkins

These are example integration tests and pretty heavy with timing, but works as a charm. And again 0.4.4. Oh, need to mention ALL tests are written as xUnit (grouped by suites via TestCase)

[skudriashev]

@dmakhno, right, you use the multiprocessing plugin. So need to fix. Will do when possible.

[skudriashev]

Hi, @dmakhno! I've implemented fix for MacOs in the max-os-fix branch (https://github.com/mahmoudimus/nose-timer/tree/mac-os-fix). Could you, please, try it to check if this fixes your issue. Thanks.

[matclayton]

We're seeing the same issue, and this branch fixed it for us, any chance of releasing it to pipy?

[mahmoudimus]

@matclayton thanks for the feedback. we've made a pull request here #71 and we'll publish to pypi shortly.

[mahmoudimus]

@matclayton can you pull v 0.6.0 from pypi and confirm that it works for you?

[matclayton]

Have done, and it does :)

On 11 February 2016 at 19:23, Mahmoud Abdelkader notifications@github.com wrote:

@matclayton https://github.com/matclayton can you pull v 0.6.0 from pypi and confirm that it works for you?

— Reply to this email directly or view it on GitHub https://github.com/mahmoudimus/nose-timer/issues/68#issuecomment-183020415 .

Matthew Clayton | Co-Founder Mixcloud Limited

mixcloud http://www.mixcloud.com/mat twitter http://www.twitter.com/matclayton

email mat@mixcloud.com mobile +44 7872007851 skype matclayton

[mahmoudimus]

@matclayton :+1: thanks for reporting!

@dmakhno you may want to try this as well. let us know how it goes :)