I am just starting to contribute to fedora, so, do not know all the procedures to get patches in other packages, but please consider and review the packages at: Spec URL: http://kenobi.mandriva.com/~pcpa/symmetrica.spec SRPM URL: http://kenobi.mandriva.com/~pcpa/symmetrica-2.0-7.fc16.src.rpm It adds a patch to rename the global "sum" and "sort" symbols, as well as allow including macro.h before def.h (otherwise there is kind of a cyclic loop there), and also a patch to not print a banner the first time a function from symmetrica is called (what can be annoying or cause several sagemath doctests to fail due to the banner in output).
Go here: https://admin.fedoraproject.org/pkgdb/acls/name/symmetrica and apply for watchbugzilla / watchcommits / commit / approveacls — the first two will be approved automatically, and I'll approve the second two. Then you can just apply the changes yourself =).
Thanks. I selected aproveacls, but I do not think I need it. Also, Jerry already corrected the package, so, I think I will not look at it again for some time, but something that should be useful is to rename more symbols, because it is very easy to get hard to debug errors if those symbols end up being used by another library.
I commented only in scitech.org, and that I would prefer to not change it right now because Jerry did work on an update and other patches, but there is still a problem in symmetrica. First there was the problem reported here, and now there is a pending update for symmetrica, but there is another one I found after managing to actually build and run sagemath in the buildroot environment, to generate its documentation on fedora 16, commented at: http://lists.fedoraproject.org/pipermail/scitech/2012-May/000119.html The solution should be to not define -DLOCALTRUE otherwise, programs linking to symmetrica should define the stub functions local_anfang() and local_ende()
I will use this case to get more confidence with fedpkg and correct the problem with compiling it with -DLOCALTRUE, what generates calls to undefined functions. Blame me if I make anything wrong :-)
Corrected in rawhide, and package should be only used by the sagemath package when all dependencies are resolved, otherwise, for custom usages it is easy to workaround the problem.