[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