Skip to content
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

Compilation warning #28

Closed
WojciechKarpiel opened this issue Jul 1, 2017 · 4 comments
Closed

Compilation warning #28

WojciechKarpiel opened this issue Jul 1, 2017 · 4 comments

Comments

@WojciechKarpiel
Copy link

WojciechKarpiel commented Jul 1, 2017

Hello!
(disclamer: I'm new to both HoTT and Agda)

I use current (git) version of Agda (Agda version 2.6.0-dc58db0) and HoTT (commit 831dcfc), both pulled 1st July

When I try to compile the library itself it gives warning:

/home/wojciech/Kod/HoTT-Agda/core/lib/Base.agda:70,1-25
Builtin REFL does no longer exist. It is now bound by BUILTIN
EQUALITY
when checking the pragma BUILTIN REFL idp

It's following piece of code:

infix 30 _==_
data _==_ {i} {A : Type i} (a : A) : A → Type i where
  idp : a == a

Path = _==_

{-# BUILTIN EQUALITY _==_ #-}
{-# BUILTIN REFL idp #-}

Simply commenting out the BUILTIN REFL line fixes the problem.

Last changes to these lines are from 2013. Has anyone had similar issue and knows how to fix it?

@favonia
Copy link
Contributor

favonia commented Jul 1, 2017

@WojciechKarpiel Hello! Thanks for reporting the issue. I was aware of this issue, but it is required by the current stable version (2.5.2) of Agda and its next (stable) version has not been released yet. :-( Due to this I am a little bit reluctant to remove it even though I am also using the unstable (git) version. You can track issue #27 for future updates on this library, or downgrade your Agda back to 2.5.2 for the moment. Does that sound reasonable to you?

@favonia
Copy link
Contributor

favonia commented Jul 1, 2017

By the way, technically speaking this is a warning not an error, and your development should still work.

@WojciechKarpiel
Copy link
Author

WojciechKarpiel commented Jul 1, 2017

Does that sound reasonable to you?

Sure, thanks for clarification!

Also: I changed 'Error' to 'Warning' in issue title

@WojciechKarpiel WojciechKarpiel changed the title Compilation error Compilation warning Jul 1, 2017
@favonia
Copy link
Contributor

favonia commented Jul 1, 2017

@WojciechKarpiel Thanks. However, let me close this because it is already on the to-do list #27.

@favonia favonia closed this as completed Jul 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants