New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redundant warning on compilation when installing swiper #324

Closed
asampal opened this Issue Dec 18, 2015 · 4 comments

Comments

Projects
None yet
2 participants
@asampal

asampal commented Dec 18, 2015

I'm using Windows 10 64 bit, Emacs 24.5

When installing swiper, Compile-Log shows the following warning. I suppose the functions referred to used to be needed only in swiper.el, but are now used elsewhere. Maybe the names could be changed to reflect their current use. Not sure if that would eliminate the warning.

Compiling file d:/dev/msys64/home/adi/.emacs.d/elpa/swiper-20151217.235/ivy.el at Thu Dec 17 19:31:07 2015

In end of data:
ivy.el:2462:1:Warning: the following functions are not known to be defined:
    swiper--cleanup, swiper--add-overlays,
    pulse-momentary-highlight-one-line

@abo-abo abo-abo closed this in 8306be1 Dec 18, 2015

@abo-abo

This comment has been minimized.

Show comment
Hide comment
@abo-abo

abo-abo Dec 18, 2015

Owner

Thanks.

Owner

abo-abo commented Dec 18, 2015

Thanks.

@asampal

This comment has been minimized.

Show comment
Hide comment
@asampal

asampal Dec 18, 2015

Looks good for the first two, but there is still a warning about pulse-momentary-highlight-one-line.

asampal commented Dec 18, 2015

Looks good for the first two, but there is still a warning about pulse-momentary-highlight-one-line.

@asampal

This comment has been minimized.

Show comment
Hide comment
@asampal

asampal Dec 18, 2015

I guess in a stable version you'd have probably cleaned these kinds of things up. Do you want me to stop reporting similar things if I'm loading non-stable?

asampal commented Dec 18, 2015

I guess in a stable version you'd have probably cleaned these kinds of things up. Do you want me to stop reporting similar things if I'm loading non-stable?

@abo-abo

This comment has been minimized.

Show comment
Hide comment
@abo-abo

abo-abo Dec 19, 2015

Owner

Please report the warnings if you catch them. I didn't see this one because I had a bug in my Makefile, so make compile was producing no warnings.

Owner

abo-abo commented Dec 19, 2015

Please report the warnings if you catch them. I didn't see this one because I had a bug in my Makefile, so make compile was producing no warnings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment