Issues with loading macros #128

shaunsingh · 2022-08-07T17:50:11Z

Hi, I'm trying out aniseed again on my config, which you can see here: https://github.com/shaunsingh/nyoom.nvim/tree/aniseed-support

The macros in question are here: https://github.com/shaunsingh/nyoom.nvim/tree/aniseed-support/fnl/macros

Essentially, aniseed can recognize command-macros event-macros and all of the top-level macros just fine. However, those depend on other macros as well: these which aniseed can't find. How should I proceed with this, is this something aniseed should solve or rather is there a certain way I should name/redo those macros?

Here is the resulting error:

Error detected while processing /Users/shauryasingh/.local/share/nvim/site/pack/packer/start/aniseed/plugin/aniseed.vim:
line    3:
Compile error in /Users/shauryasingh/.config/nvim/fnl/core/commands.fnl:2
  .../shauryasingh/.config/nvim/fnl/macros/command-macros.fnl:1: module 'macros.lib.types' not found:
^Ino field package.preload['macros.lib.types']
^Ino file './share/lua/5.1/macros/lib/types.lua'
^Ino file './macros/lib/types.lua'
^Ino file './macros/lib/types/init.lua'
^Ino file './lib/lua/5.1/macros/lib/types.so'
^Ino file './macros/lib/types.so'
^Ino file './lib/lua/5.1/loadall.so'
^Ino file './lib/lua/5.1/macros.so'
^Ino file './macros.so'
^Ino file './lib/lua/5.1/loadall.so'
stack traceback:
^I[C]: in function 'require'
^I.../shauryasingh/.config/nvim/fnl/macros/command-macros.fnl:1: in function 'loader'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:1996: in function 'require-macros'
^Isrc/fennel/macros.fnl:322: in function <src/fennel/macros.fnl:317>
^I[C]: in function 'xpcall'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:2634: in function 'macroexpand_2a'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:2820: in function 'compile1'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:784: in function 'special'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:2658: in function 'compile1'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:3054: in function 'compileString'
^I...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:50: in function <...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:49>
^I[C]: in function 'str'
^I...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:57: in function 'file'
^I...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:75: in function 'glob'
^I.../nvim/site/pack/packer/start/aniseed/lua/aniseed/env.lua:57: in function 'init'
^I[string ":lua"]:1: in main chunk
                                      ^
runtime error: module 'macros.lib.seq' not found:
^Ino field package.preload['macros.lib.seq']
^Ino file './share/lua/5.1/macros/lib/seq.lua'
^Ino file './macros/lib/seq.lua'
^Ino file './macros/lib/seq/init.lua'
^Ino file './lib/lua/5.1/macros/lib/seq.so'
^Ino file './macros/lib/seq.so'
^Ino file './lib/lua/5.1/loadall.so'
eyboard interrupt
stack traceback:
  [C]: in function 'getcwd'
  ...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:25: in function 'wrap_macros'
  ...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:50: in function ?
  [C]: in function 'str'
  ...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:57: in function 'file'
  ...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:75: in function 'glob'
  .../nvim/site/pack/packer/start/aniseed/lua/aniseed/env.lua:57: in function 'init'
  [string ":lua"]:1: in main chunk
Compile error in /Users/shauryasingh/.config/nvim/fnl/core/keybinds.fnl:2
  .../shauryasingh/.config/nvim/fnl/macros/keybind-macros.fnl:1: module 'macros.lib.types' not found:
^Ino field package.preload['macros.lib.types']
^Ino file './share/lua/5.1/macros/lib/types.lua'
^Ino file './macros/lib/types.lua'
^Ino file './macros/lib/types/init.lua'
^Ino file './lib/lua/5.1/macros/lib/types.so'
^Ino file './macros/lib/types.so'
^Ino file './lib/lua/5.1/loadall.so'
^Ino file './lib/lua/5.1/macros.so'
^Ino file './macros.so'
^Ino file './lib/lua/5.1/loadall.so'
stack traceback:
^I[C]: in function 'require'
^I.../shauryasingh/.config/nvim/fnl/macros/keybind-macros.fnl:1: in function 'loader'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:1996: in function 'require-macros'
^Isrc/fennel/macros.fnl:322: in function <src/fennel/macros.fnl:317>
^I[C]: in function 'xpcall'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:2634: in function 'macroexpand_2a'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:2820: in function 'compile1'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:784: in function 'special'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:2658: in function 'compile1'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:3054: in function 'compileString'
^I...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:50: in function <...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:49>
^I[C]: in function 'str'
^I...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:57: in function 'file'
^I...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:75: in function 'glob'
^I.../nvim/site/pack/packer/start/aniseed/lua/aniseed/env.lua:57: in function 'init'
^I[string ":lua"]:1: in main chunk
(import-macros {: let!} :macros.variable-macros)

The text was updated successfully, but these errors were encountered:

bangedorrunt · 2022-08-07T18:35:11Z

Related #62

shaunsingh · 2022-08-07T18:36:15Z

I'll try putting everything under the macros/ folder, rather than some under macros/lib and some under macros/. It would be nice to have support for the later case as well though

shaunsingh · 2022-08-07T18:40:11Z

Nevermind, similar error even after doing that. I don't think #62 was actually fixed

Error detected while processing /Users/shauryasingh/.local/share/nvim/site/pack/packer/start/aniseed/plugin/aniseed.vim:
line    3:
Compile error in /Users/shauryasingh/.config/nvim/fnl/core/commands.fnl:2
  .../shauryasingh/.config/nvim/fnl/macros/command-macros.fnl:1: module 'macros.types-macros' not found:
^Ino field package.preload['macros.types-macros']
^Ino file './share/lua/5.1/macros/types-macros.lua'
^Ino file './macros/types-macros.lua'
^Ino file './macros/types-macros/init.lua'
^Ino file './lib/lua/5.1/macros/types-macros.so'
^Ino file './macros/types-macros.so'
^Ino file './lib/lua/5.1/loadall.so'
^Ino file './lib/lua/5.1/macros.so'
^Ino file './macros.so'
^Ino file './lib/lua/5.1/loadall.so'
stack traceback:
^I[C]: in function 'require'
^I.../shauryasingh/.config/nvim/fnl/macros/command-macros.fnl:1: in function 'loader'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:1996: in function 'require-macros'
^Isrc/fennel/macros.fnl:322: in function <src/fennel/macros.fnl:317>
^I[C]: in function 'xpcall'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:2634: in function 'macroexpand_2a'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:2820: in function 'compile1'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:784: in function 'special'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:2658: in function 'compile1'
^I...te/pack/packer/start/aniseed/lua/aniseed/deps/fennel.lua:3054: in function 'compileString'
^I...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:50: in function <...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:49>
^I[C]: in function 'str'
^I...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:57: in function 'file'
^I...m/site/pack/packer/start/aniseed/lua/aniseed/compile.lua:75: in function 'glob'
^I.../nvim/site/pack/packer/start/aniseed/lua/aniseed/env.lua:57: in function 'init'
^I[string ":lua"]:1: in main chunk
                                      ^
runtime error: module 'macros.seq-macros' not found:
^Ino field package.preload['macros.seq-macros']
^Ino file './share/lua/5.1/macros/seq-macros.lua'
^Ino file './macros/seq-macros.lua'
^Ino file './macros/seq-macros/init.lua'
^Ino file './lib/lua/5.1/macros/seq-macros.so'
^Ino file './macros/seq-macros.so'
^Ino file './lib/lua/5.1/loadall.so'

bangedorrunt · 2022-08-07T21:02:50Z

That's why I put everything into one macros file. Hope @Olical will fix later.

Olical · 2022-08-14T13:08:55Z

I'll try and fix this for good soon, just focusing on some Conjure stuff at the moment.

shaunsingh · 2022-08-14T16:41:08Z

Loving the work on conjure, for now I just moved everything into one file and it works perfectly. Take your time, no worries

Olical · 2022-08-24T18:34:25Z

So I think this was to do with Aniseed not creating intermediate directories when copying macro files over into your compiled lua output directory. I've since fixed that and I can now refer to nested macro files!

If you can still reproduce it, please let me know. I'm trying to reproduce it still but I feel like I'm missing something. I think I've fixed one issue but there may be another lurking that I can't quite repro.

Olical · 2022-08-24T19:00:37Z

So this works, but you NEED to use import-macros, if you just require then it's going to try and load a lua module which won't work since you can't have a macro file that is compiled lua. Macros MUST always be uncompiled fnl.

If this statement is wrong and this does work in some other system please let me know! But I think the only way to load other macro files into your macro file and use them is to use import-macros etc. I've fixed the nested dirs of macros issue, so maybe that was it?

Here's a small snippet of me requiring a sub-macro file and using it. It works for me! Although not super useful, I think it's just a limitation of Fennel, you can't just infinitely require and next macros and have it resolve them all as you would in Clojure... I think. I could just be wrong!

; (local {: bad-add} (require :./misc.other-macros.fnl))
(import-macros {: bad-add} :dotfiles.misc.other-macros)

{:adder-b (fn [a b]
            `(+ 1 ,a ,b ,(bad-add 1 2)))

...

bangedorrunt · 2022-08-24T23:03:28Z

@Olical
This might be in relation to
rktjmp/hotpot.nvim#76
and
rktjmp/hotpot.nvim#77

Olical · 2022-08-24T23:30:36Z

Yep! That sounds like the same problem! Which to me sounds like a Fennel usage problem? Like we need to work out patterns that work for Fennel. I don't think I can work around that limitation? I think I've fixed aniseeds bug to do with nested dirs though!

…

On Thu, 25 Aug 2022, 00:03 Thanh Dung TRUONG, ***@***.***> wrote: @Olical <https://github.com/Olical> Might related to rktjmp/hotpot.nvim#76 <rktjmp/hotpot.nvim#76> — Reply to this email directly, view it on GitHub <#128 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACM6XKKGA7GL66QETGSFCTV22S4VANCNFSM5522DVZA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

Olical · 2022-09-17T11:55:28Z

We're now on fennel 1.2.0 on the develop branch, not sure if that helps at all...

rktjmp · 2022-11-11T04:51:25Z

edit: sorry, just assumed this issue was about cljlib because it was linked!, not the broader (require :macro) behaviour! 😳

(x-post from hotpot issue)

I think if you use this fork https://github.com/rktjmp/fennel-cljlib/ cljlib should now work?

The install instructions have changed since this was opened, you how have to set the package.path.

Internally it makes a direct (require :lazy-seq) call, so you must update the package path as instructed. For me that was:

(let [root (vim.fn.stdpath :config)
      path (.. root :/fnl/cljlib)]
  (set package.path (string.format "%s;%s;%s"
                                   (.. path "/?/init.lua")
;; readme doesn't say to add this one but it's idiomatic to include both the `?.lua` and `?/init.lua` paths.
                                   (.. path "/?.lua")
                                   package.path)))

(local clj (require :cljlib))
(import-macros cljm :cljlib)

(print (clj.add 1 2 3))
(cljm.time (do
  (for [i 1 10000000] i)))

I just cloned it into .config/nvim/fnl/cljlib (and got the submodules) which effectively puts it in the main cljlib package in "neovims lua package root" if you want to call it that. If you put it in some nested subdir you might have to muck around with package.path a bit more.

Hopefully the patch can be up streamed, have to check with Andre if he has opinions on the change.

rktjmp/fennel-cljlib@8b58117

(note this doesn't fix calling `(require :some-macro) inside a import-macro in aniseed/hotpot obviously, imo that behaviour is out of spec)

shaunsingh · 2022-12-24T18:47:54Z

Considering require-macros is deprecated anyways, and that aniseed works just fine with multiple macro files and import-macros, I'll consider this fixed for my own use case.

Olical mentioned this issue Aug 20, 2022

require fails when requiring a macro-utils file from other macro files (Fennel Issue #406) Olical/conjure#286

Open

Olical added the bug Something isn't working label Sep 17, 2022

shaunsingh closed this as completed Dec 24, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Issues with loading macros #128

Issues with loading macros #128

shaunsingh commented Aug 7, 2022 •

edited

Loading

bangedorrunt commented Aug 7, 2022

shaunsingh commented Aug 7, 2022

shaunsingh commented Aug 7, 2022

bangedorrunt commented Aug 7, 2022

Olical commented Aug 14, 2022

shaunsingh commented Aug 14, 2022

Olical commented Aug 24, 2022

Olical commented Aug 24, 2022

bangedorrunt commented Aug 24, 2022 •

edited

Loading

Olical commented Aug 24, 2022 via email

Olical commented Sep 17, 2022

rktjmp commented Nov 11, 2022 •

edited

Loading

shaunsingh commented Dec 24, 2022

Issues with loading macros #128

Issues with loading macros #128

Comments

shaunsingh commented Aug 7, 2022 • edited Loading

bangedorrunt commented Aug 7, 2022

shaunsingh commented Aug 7, 2022

shaunsingh commented Aug 7, 2022

bangedorrunt commented Aug 7, 2022

Olical commented Aug 14, 2022

shaunsingh commented Aug 14, 2022

Olical commented Aug 24, 2022

Olical commented Aug 24, 2022

bangedorrunt commented Aug 24, 2022 • edited Loading

Olical commented Aug 24, 2022 via email

Olical commented Sep 17, 2022

rktjmp commented Nov 11, 2022 • edited Loading

shaunsingh commented Dec 24, 2022

shaunsingh commented Aug 7, 2022 •

edited

Loading

bangedorrunt commented Aug 24, 2022 •

edited

Loading

rktjmp commented Nov 11, 2022 •

edited

Loading