Overview

Request 627637 accepted

- Switch to multibuild to run tests
- Drop doc subpkg, they have compiled docu on web, much better
* Drops patch for_sphinx.patch

- Version update to 4.1.1:
* Fixes on python 3.x

Loading...

Dominique Leuenberger's avatar

+Requires: this-is-only-for-build-envs

why? The -test should not contain the python code - at best you'd publish a test log in the package there; special tricks to make this not-installable seems weird here


Tomáš Chvátal's avatar

It is a shortest way that makes the tests being run while not having to rewrite the whole %files section.


Dominique Leuenberger's avatar

Lazyness is no excuse for crap, sorry


Tomáš Chvátal's avatar

That is not laziness the package will not install any files, so it is better to just block it than to create empty files with license.


Dominique Leuenberger's avatar

The test log in the files section would be value - as it could be inspected


Dominique Leuenberger's avatar

(the problem with this is that the bot won't find an installable solution - which in turn means it needs be white listed; all the 'only-for-build-env' whitelists are for the sake of bootstrap/cycles, and not for sake of publishing unusable stuff into the repo)


Tomáš Chvátal's avatar

You can have empty package like python-canonicaljson.spec Or have it much more readable with just one conflict, i will go for readable over empty package any day.


Dominique Leuenberger's avatar

I frankly don't see this as a proper solution to the problem at hand; readable maybe... but still crap

The test doesn't even run against the 'exact' files we deliver to the user; and with TW not doing full rebuilds of the entire distro all the time, there is not even a relation establishable between rebuilds of the delivered package vs what the test ran against. This test is a pure proforma thing for somebodys conscious - but the value is questionable


Dominique Leuenberger's avatar

(we had a very similar issue in meson/meson-testsuite - where the testsuite package intentionally was done to use meson (the package we deliver) to use for the test suite, intentionally removing all sources that could possibly interfere... we want to test what we deliver after all)

Request History
Tomáš Chvátal's avatar

scarabeus_iv created request

- Switch to multibuild to run tests
- Drop doc subpkg, they have compiled docu on web, much better
* Drops patch for_sphinx.patch

- Version update to 4.1.1:
* Fixes on python 3.x


Saul Goodman's avatar

licensedigger accepted review

ok


Factory Auto's avatar

factory-auto added opensuse-review-team as a reviewer

Please review sources


Factory Auto's avatar

factory-auto added repo-checker as a reviewer

Please review build success


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Staging Bot's avatar

staging-bot added openSUSE:Factory:Staging:D as a reviewer

Being evaluated by staging project "openSUSE:Factory:Staging:D"


Staging Bot's avatar

staging-bot accepted review

Picked openSUSE:Factory:Staging:D


Ismail Dönmez's avatar

namtrac accepted review


Repo Checker's avatar

repo-checker accepted review

cycle and install check passed


Dominique Leuenberger's avatar

dimstar_suse accepted review

ready to accept


Dominique Leuenberger's avatar

dimstar_suse approved review

ready to accept


Dominique Leuenberger's avatar

dimstar_suse accepted request

Accept to openSUSE:Factory

openSUSE Build Service is sponsored by