Red Hat Bugzilla – Bug 819335
Symmetrica has too common global names and problems with headers inclusion order
Last modified: 2012-05-26 13:19:28 EDT
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 email@example.com, 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:
The solution should be to not define -DLOCALTRUE otherwise, programs
linking to symmetrica should define the stub functions local_anfang()
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.