Created attachment 938811 [details] Patch Build fails due to type incompatibilities in swig generated c++ code. Issue is cased by forcing size_t to be int on 32-bit/31-bit platforms, but on s390 size_t is long. Removing -DSIZE_T_32 swig flag fixes build on s390, but breaks build on other 32-bit platforms(similar ways as is now on s390), so patch is platform dependant. In attachment is patch with up mentioned changes. Upstream report: http://gnuradio.org/redmine/issues/728 Failing build: http://s390.koji.fedoraproject.org/koji/buildinfo?buildID=235359 Successful scratch builds: http://s390.koji.fedoraproject.org/koji/taskinfo?taskID=1540031 http://koji.fedoraproject.org/koji/taskinfo?taskID=7611375
Created attachment 938813 [details] Improved patch
Thanks, but could you provide s390-build-fix.patch?
Created attachment 938818 [details] Requested patch Here it is.
Jaroslav, if possible please apply the fix to F-20 and up. Don't think anybody will use gnuradio on s390, but if possible we should get bugs fixed.
(In reply to Dan Horák from comment #4) > Jaroslav, if possible please apply the fix to F-20 and up. Don't think > anybody will use gnuradio on s390, but if possible we should get bugs fixed. OK, NP. BTW gnuradio can be used for various data processing not only RF and it's even used by NASA, so who knows :)
Created attachment 939065 [details] Proposed patch sent upstream
Waiting for upstream.
Upstream reply: > This is more of a "wontfix for now." We might revisit this issue again in the > future when we update our swig dependencies. The issue is correctly diagnosed, > but the current implementation took a while to get just right that it would > build and work on all of the known systems we support. We'd need to verify > that the updated patch supports SWIG back to 1.3.31 (the earliest version we > require) on a variety of the various installs we support. > > For now, I would suggest you keep this patch around locally and apply it as > needed. The code you're patching is unlikely to change. When we make a > decision on the new minimum SWIG version in 3.8, we should revisit this issue. So they are not against the distro fix and may later apply it upstream. Currently, I cannot see anything wrong with the fix, so let's try it and see what happen :)
gnuradio-3.7.5-3.el7 has been submitted as an update for Fedora EPEL 7. https://admin.fedoraproject.org/updates/gnuradio-3.7.5-3.el7
gnuradio-3.7.5-3.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/gnuradio-3.7.5-3.fc21
Package gnuradio-3.7.5-3.el7: * should fix your issue, * was pushed to the Fedora EPEL 7 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=epel-testing gnuradio-3.7.5-3.el7' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-EPEL-2014-3164/gnuradio-3.7.5-3.el7 then log in and leave karma (feedback).
gnuradio-3.7.5-3.fc21 has been pushed to the Fedora 21 stable repository. If problems still persist, please make note of it in this bug report.
gnuradio-3.7.5.1-1.el7 has been submitted as an update for Fedora EPEL 7. https://admin.fedoraproject.org/updates/gnuradio-3.7.5.1-1.el7
gnuradio-3.7.5.1-1.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report.