swig207-setools.patch, apparently added for #822539 (but with no description of the rationale/details in either the bug or the patch) renames generated member functions, e.g. for > struct libvk_volume_property {}; > %extend libvk_volume_property { > %typemap(newfree) char * "g_free ($1);"; > %immutable label; > %newobject label; > char *label; swig used to use the name libvk_volume_property_label_get() for the getter function; with the patch, it now uses struct_libvk_volume_property_label_get(). This is in conflict with http://www.swig.org/Doc2.0/SWIGDocumentation.html#SWIG_adding_member_functions > Note that %extend can be used to access externally written functions provided they follow the naming convention used in this example … > A little known feature of the %extend directive is that it can also be used to add synthesized attributes or to modify the behavior of existing data attributes. … > The name used for %extend should be the name of the struct and not the name of any typedef to the struct. Version-Release number of selected component (if applicable): swig-2.0.11-1.fc20.x86_64 (up to and including current master branch) Steps to Reproduce: Try to check out and build volume_key from http://git.fedorahosted.org/cgit/volume_key.git . Observe > python/volume_key_wrap.c: In function '_wrap_libvk_volume_property_label_get': > python/volume_key_wrap.c:3767:3: warning: implicit declaration of function 'struct_libvk_volume_property_label_get' [-Wimplicit-function-declaration] > result = (char *)struct_libvk_volume_property_label_get(arg1); and many similar warnings. Additional info: Preferably, please revert the patch. If that were not possible and I had to rename the functions in the swig input, please advise how to write swig input that builds correctly against both the Fedora-patched swig and upstream unmodified one.
Hello Miloslav, does this happen with SWIG-2.0.X on F20, only? Can you reproduce that with SWIG-3.0.X on >= fc21? I just did some quick investigation on that patch and it's purpose. To me that looks like a kludge applied to SWIG for fixing a build- / functionality-issue in SETools, which is based on faulty or semantically incorrect swig-wrapper-code in there. I will dig a bit deeper into this during the next days and see, if we can find a way to solve this in a better way. Cheers, Björn
Yes, reproduced with swig-3.0.5-2.fc21. To do so, in more detail: > # install packages that provide: cryptsetup-luks-devel, gettext-devel, glib2-devel, /usr/bin/gpg, gpgme-devel, libblkid-devel, nss-devel, python-devel > git clone http://git.fedorahosted.org/cgit/volume_key.git > cd volume_key > autoreconf -fis -Wall > ./configure --prefix=$(pwd)/p > make and observe the warnings
This bug appears to have been reported against 'rawhide' during the Fedora 22 development cycle. Changing version to '22'. More information and reason for this action is here: https://fedoraproject.org/wiki/Fedora_Program_Management/HouseKeeping/Fedora22
The patch can be removed. The issue was solved by upstream at 2.0.8. For more information see: https://github.com/swig/swig/issues/515
swig-3.0.7-6.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-16287
swig-3.0.7-3.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-16289
swig-3.0.7-3.fc21 has been submitted as an update to Fedora 21. https://bodhi.fedoraproject.org/updates/FEDORA-2015-16291
swig-3.0.7-3.fc21 has been pushed to the Fedora 21 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with $ su -c 'dnf --enablerepo=updates-testing update swig' You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-16291
swig-3.0.7-3.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with $ su -c 'dnf --enablerepo=updates-testing update swig' You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-16289
swig-3.0.7-6.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with $ su -c 'dnf --enablerepo=updates-testing update swig' You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-16287
swig-3.0.7-6.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.
swig-3.0.7-3.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report.
swig-3.0.7-3.fc21 has been pushed to the Fedora 21 stable repository. If problems still persist, please make note of it in this bug report.
Confirmed, thanks!