unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Ken Brown <kbrown@cornell.edu>
To: Eli Zaretskii <eliz@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: [Emacs-diffs] emacs-25 33ec2ff: Add one more mod-test test
Date: Sun, 29 Nov 2015 10:21:39 -0500	[thread overview]
Message-ID: <565B1803.5090805@cornell.edu> (raw)
In-Reply-To: <83wpt1o4iw.fsf@gnu.org>

On 11/28/2015 10:42 PM, Eli Zaretskii wrote:
>> From: Ken Brown <kbrown@cornell.edu>
>> Date: Sat, 28 Nov 2015 21:12:03 -0500
>>
>> On 11/24/2015 2:14 PM, Eli Zaretskii wrote:
>>> +  (let ((descr (should-error (mod-test-sum 1 2 3))))
>>> +    (should (eq (car descr) 'wrong-number-of-arguments))
>>> +    (should (stringp (nth 1 descr)))
>>> +    (should (eq 0
>>> +                (string-match
>>> +                 (if (eq system-type 'windows-nt)
>>> +                     "#<module function at \\(0x\\)?[0-9a-fA-F]+ from .*>"
>>> +                   "#<module function Fmod_test_sum from .*>")
>>> +                 (nth 1 descr))))
>>
>> This fails on Cygwin because
>>
>>     (should-error (mod-test-sum 1 2 3))
>>
>> evaluates to
>>
>>     (wrong-number-of-arguments "#<module function at 0x440bd10dc>" 3)
>>
>> This is like the expected value on MS-Windows, but without the "from".
>> Does the missing "from" indicate a bug in the Cygwin build?
>
> It indicates that your dladdr implementation doesn't work (probably
> returns zero).
>
> Please ask on the Cygwin list, and if dladdr is indeed missing, the
> test should be updated to accommodate for this return value on Cygwin.

Thanks.  In fact, dladdr isn't implemented on Cygwin, so dynlib_addr 
always returns false.  I'll update the test.  Should the update be 
Cygwin-specific?  According to http://linux.die.net/man/3/dladdr, dladdr 
is a glibc extension to POSIX, so there could conceivably be other 
systems with the same issue.

Ken



  reply	other threads:[~2015-11-29 15:21 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20151124191433.3972.80810@vcs.savannah.gnu.org>
     [not found] ` <E1a1J2w-00013J-0o@vcs.savannah.gnu.org>
2015-11-29  2:12   ` [Emacs-diffs] emacs-25 33ec2ff: Add one more mod-test test Ken Brown
2015-11-29  3:42     ` Eli Zaretskii
2015-11-29 15:21       ` Ken Brown [this message]
2015-11-29 16:15         ` Eli Zaretskii
2015-11-29 16:29           ` Ken Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=565B1803.5090805@cornell.edu \
    --to=kbrown@cornell.edu \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).