Overview

Request 852722 accepted

New package

Loading...


Atri Bhattacharya's avatar

Thanks for the sr. Here are my suggestions:

  • Pkg should be preferably called python-bCNC as it installs a python module.
  • Please run the tests as part of a %check section.
  • BuildFailures for Leap look straightforward to fix, e.g. using %suse_update_desktop_file.
  • Would be nice to have a full URL for Source1.

Stefan Brüns's avatar
author source maintainer target maintainer

Its implemented in python, that does not make it a python module - firewall-applet, hplip are all implemented in python as well.

There are no tests run.

No full URL, as this uses a git snaphot/_service.


Atri Bhattacharya's avatar

Its implemented in python, that does not make it a python module - firewall-applet, hplip are all implemented in python as well.

Yes, makes sense.

There are no tests run.

There is a tests directory. Don't tests in there work?

No full URL, as this uses a git snaphot/_service.

For SOURCE1, the svg file?


Stefan Brüns's avatar
author source maintainer target maintainer

Since when have running tests become a requirement?

The svg file is a vectorized version of the included icon, it has no repository.


Atri Bhattacharya's avatar

Since when have running tests become a requirement?

It has always been recommended. If possible, tests should be run where they are shipped, unless there is a missing pkg that is needed by the tests or the tests are broken as shipped by upstream. Tests ensure that the build works; I can't understand why this is a problem.


Stefan Brüns's avatar
author source maintainer target maintainer

So, tell me, how should these tests be run?

Feel free to enhance the package.


Atri Bhattacharya's avatar

If you would have just said that these tests don't run for x.y.z reason, that would have been great. Instead you choose to confront my suggestions, made in good faith, with 'since when have tests been required', etc.

Look, your pkg, your project, feel free to check in whatever. I was just giving suggestions potentially to help, but they weren't in any way binding.

Done with this, cheers.

Request History
Stefan Brüns's avatar

StefanBruens created request

New package


Stefan Brüns's avatar

StefanBruens accepted request

openSUSE Build Service is sponsored by