There should be tests for this; please do not add commented out code.

Also, please rebase instead of merging if possible.

@QuLogic Please reconsider the question of tests. This is fundamentally a simple one-line fix to a bug in resetup.py which probably goes way back, and was propagated rather than fixed in the cleanup b32a06a. It is just putting in the right set of strings. For something this simple, I think that adding tests is just adding code to be maintained, not making anything safer.

This appears to be fixing such a simple and obvious bug that we should be sure to get it into 3.3, if at all possible.

There already are tests in this PR; they're just commented out. They either need to work, or not be added at all.

Exactly, and I'm suggesting that they should not be added at all. All we need is the first commit in the series.

Apologies for not responding here, was caught up in some other commitments. I will remove the test cases and rebase within the two days.

Perhaps I was unclear; I did not mean that there should be tests, but that there should be or should not be tests, not some code in limbo.

Thanks @MuhammadFarooq1234!

I think this is your first PR for us, correct? Congratulations, and thank you!

Thank you both @efiring @QuLogic for your help. I look forward to contributing further.