core.logic

defrel cannot be called inside function

Details

  • Type: Defect Defect
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None

Activity

Hide
Martin Trojer added a comment -

Indeed, a make-rel function would be optimal, then you can def it (or store it somewhere else) at your leisure.

Show
Martin Trojer added a comment - Indeed, a make-rel function would be optimal, then you can def it (or store it somewhere else) at your leisure.
Hide
Jason Jackson added a comment -

I only implemented this so I could delete relation state. There's probably a better way to do this though.

Show
Jason Jackson added a comment - I only implemented this so I could delete relation state. There's probably a better way to do this though.
Hide
David Nolen added a comment - - edited

I've yet to hear a good rationale for this patch. defrel has the same semantics as def and should only exist at the top level. If someone puts more though into what defining a rel dynamically should look like I will consider it.

Show
David Nolen added a comment - - edited I've yet to hear a good rationale for this patch. defrel has the same semantics as def and should only exist at the top level. If someone puts more though into what defining a rel dynamically should look like I will consider it.
Hide
Martin Trojer added a comment - - edited

This patch solved the exact same problem for me aswell.

Cheers...

Show
Martin Trojer added a comment - - edited This patch solved the exact same problem for me aswell. Cheers...
Hide
Jason Jackson added a comment -

New patch with attribution info. Sorry about super long turn around time, in the future I should be much faster.

Show
Jason Jackson added a comment - New patch with attribution info. Sorry about super long turn around time, in the future I should be much faster.
Hide
David Nolen added a comment -

Can you reformat the patch w/ git so it has the proper attribution information? Thanks!

Show
David Nolen added a comment - Can you reformat the patch w/ git so it has the proper attribution information? Thanks!

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated: