search

openSUSE / python-rpm-macros

Multi-Python, Single-Spec macros generator
Other
22 stars 20 forks source link

%pyunittest{,_arch} discover doesn't work #57

Closed mcepl closed 4 years ago

mcepl commented 4 years ago

@s-t-e-v-e-n-k As the macro stands right now

%pyunittest discover

generates (abbreviated) $python -munittest -v discover, which is incorrect, because then unittest starts to search for -v module. The correct call is $python -munittest discover -v. Macro should make sure that the -v argument is always in the last position of the generated code.

Hmm, that actually doesn’t work either, because then %pyunittest test lead to $python -munittest test -v and that is broken as well (it should be $python -munittest -v test).

So, we have to probably actively search for discover keyword and modify our behaviour accordingly.

s-t-e-v-e-n-k commented 4 years ago

I think we should just drop -v for being awkward from the macro.