Re: Replacing pytest -n auto with pytest -n %{_smp_build_ncpus}

2022-10-24 Thread Miro Hrončok
On 12. 07. 22 11:50, Miro Hrončok wrote: On 12. 07. 22 11:25, Petr Viktorin wrote: Or pytest-xdist could be taught to check an environment variable for `auto`, making this seamless for packagers? This could work, except that sometimes pytest-xdist is installed and we don't want to run

Re: Replacing pytest -n auto with pytest -n %{_smp_build_ncpus}

2022-07-12 Thread Miro Hrončok
On 12. 07. 22 11:25, Petr Viktorin wrote: Or pytest-xdist could be taught to check an environment variable for `auto`, making this seamless for packagers? This could work, except that sometimes pytest-xdist is installed and we don't want to run tests in parallel because they are not

Re: Replacing pytest -n auto with pytest -n %{_smp_build_ncpus}

2022-07-12 Thread Petr Viktorin
On 07. 07. 22 14:44, Miro Hrončok wrote: On 07. 07. 22 14:00, Petr Viktorin wrote: On 07. 07. 22 11:38, Miro Hrončok wrote: Hello Pythonistats, packagers, A handful of Fedora Python packages uses pytest-xdist to run tests in parallel like this:    %pytest -n auto -n auto means pytest

Re: Replacing pytest -n auto with pytest -n %{_smp_build_ncpus}

2022-07-07 Thread Ben Beasley
On 7/7/22 05:38, Miro Hrončok wrote: In the spirit of other packaging guidelines, I believe we should use this instead:   %pytest -n %{_smp_build_ncpus} I agree that this is more strictly correct. Should I do this in a mass change? Not so many packages use pytest -n auto in the spec: I

Re: Replacing pytest -n auto with pytest -n %{_smp_build_ncpus}

2022-07-07 Thread Major Hayden
On 7/7/22 04:38, Miro Hrončok wrote: > Should I do this in a mass change? Not so many packages use pytest -n > auto in the spec: > >   $ rg -l -- '(-n|--numprocesses)(\s*|=)auto(\s|$)' >   azure-cli.spec As the azure-cli maintainer, I'm okay with that change. > (Other packages have that in

Re: Replacing pytest -n auto with pytest -n %{_smp_build_ncpus}

2022-07-07 Thread Miro Hrončok
On 07. 07. 22 14:00, Petr Viktorin wrote: On 07. 07. 22 11:38, Miro Hrončok wrote: Hello Pythonistats, packagers, A handful of Fedora Python packages uses pytest-xdist to run tests in parallel like this:    %pytest -n auto -n auto means pytest will spawn a number of workers processes equal

Re: Replacing pytest -n auto with pytest -n %{_smp_build_ncpus}

2022-07-07 Thread Petr Viktorin
On 07. 07. 22 11:38, Miro Hrončok wrote: Hello Pythonistats, packagers, A handful of Fedora Python packages uses pytest-xdist to run tests in parallel like this:   %pytest -n auto -n auto means pytest will spawn a number of workers processes equal to the number of available CPUs. In the

Replacing pytest -n auto with pytest -n %{_smp_build_ncpus}

2022-07-07 Thread Miro Hrončok
Hello Pythonistats, packagers, A handful of Fedora Python packages uses pytest-xdist to run tests in parallel like this: %pytest -n auto -n auto means pytest will spawn a number of workers processes equal to the number of available CPUs. In the spirit of other packaging guidelines, I