[Bug 207781] Review Request: libmatheval - parse and evaluate symbolic expressions
bugzilla at redhat.com
bugzilla at redhat.com
Sun Sep 24 02:24:27 UTC 2006
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: libmatheval - parse and evaluate symbolic expressions
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=207781
------- Additional Comments From ed at eh3.com 2006-09-23 22:24 EST -------
Hi Jason, thatanks for taking a look. I spent some time today digging
into the "make check" errors and it turns out that there are some
deprecated guile bits which are:
SCM_STRINGP is deprecated. Use scm_is_string instead.
SCM_LENGTH is deprecated. Use scm_c_string_length instead,
for example, or see the manual.
SCM_STRING_LENGTH is deprecated. Use scm_c_string_length instead.
SCM_CHARS is deprecated. See the manual for alternatives.
SCM_STRING_CHARS is deprecated. See the manual for alternatives.
The interesting thing is that these deprecated bits *ONLY* appear
in the the file "libmatheval-1.1.3/tests/matheval.c" and nowhere else
in the actual libmatheval code.
So it appears that there are some deprecated bits only in the testing
code that then triggers some warnings that then causes one of the tests
(test number 5) to "fail" (although the test actually works, it just
emits a bunch of these warnings so that the test script *thinks* that
it has failed).
In any case, I'll report the deprecated bits in the testing code to
the upstream folks. But I see no reason to do anything else since
there appear to be no other deprecated (or otherwise broken) bits in
any of the actual library code.
Here's an update that includes the BR: texinfo
http://mitgcm.org/eh3/fedora_misc/libmatheval-1.1.3-4.src.rpm
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.
More information about the Fedora-package-review
mailing list