ef35b39
@@ -28,7 +28,7 @@
neuron
neurord
octave
- paraview
+ # paraview
python3
python3-brian2
python-brian2-doc
Remove paraview from Comp-Neuro for the time being. Waiting on bug fixes from maintainer/upstream.
Since we are in f32 beta freeze now, you will need a freeze exception to get this in.
See: https://fedoraproject.org/wiki/QA:SOP_freeze_exception_bug_process
What bug fixes for what bugs? Without more context it's hard to judge.
Hi,
Since paraview wasn't installing on F32, it was making the comp-neuro image compose fail, so we'd decided to drop it for the time being.
https://bugzilla.redhat.com/show_bug.cgi?id=1799847
It has been since fixed, and there's an update in place. So this PR should not be needed any more (right?): https://bodhi.fedoraproject.org/updates/FEDORA-2020-3cb10f553b
However, and this is where I'm a little confused, even though this PR to the f32 branch has not been merged, the latest F32 comp-neuro image does not include paraview:
https://koji.fedoraproject.org/koji/buildinfo?buildID=1474248 https://kojipkgs.fedoraproject.org//packages/Fedora-Comp_Neuro-Live/32/20200305.n.0/data/logs/image/x86_64/root.log
It's there in the kickstart file, alright:
https://pagure.io/fedora-kickstarts/blob/f32/f/fedora-comp-neuro-common.ks#_31
So, is the F32 compose being made off the master branch at the moment? We already had a similar PR merged into master: https://pagure.io/fedora-kickstarts/pull-request/612
Sorry--we're still new to the spin process :)
Pull-Request has been closed by dan1mal
Indeed! We are mistakenly building f32 from the master/rawhide branch. ;(
This happened once before too... will get it fixed. Sorry for confusion.
Ah, no worries! Since paraview is OK now, this PR isn't needed so we'll leave it closed.
Maybe fate stepped in and got me to put in a poorly commented PR, zbyszek would respond for more info, ankursinha would remark on the strange error, and kevin would reveal a typo/misconfig and now, all is well in Fedora-land. 🌈☀️
Cheers fellas, its gonna be a nice weekend! 😄
It's a team effort! 👏
Remove paraview from Comp-Neuro for the time being. Waiting on bug fixes from maintainer/upstream.