"Quick Test" Instructions No Longer Working, Current Dir Not in Load-Path

[lumin-etherlight]

I tried to test nano-emacs using the "Quick test" instructions provided in the README (i.e. running emacs -q -l nano.el), but they didn't work. Emacs throws the following error:

eval-buffer: Cannot open load file: No such file or directory, nano-layout

This is a result of emacs not being able to find the module being required. I suspect this is a regression caused by commit 5ff5f58, which removes the code that adds the current directory to the load path. Applying the following patch fixes the issue, and allows the Quick test instructions to work again.

diff --git a/nano.el b/nano.el
index a55ddd0..5ad94a0 100644
--- a/nano.el
+++ b/nano.el
@@ -17,6 +17,8 @@
 ;; ---------------------------------------------------------------------
 (package-initialize)

+(add-to-list 'load-path ".")
+
 ;; Default layout (optional)
 (require 'nano-layout)

[rougier]

Thanks for the report and for finding the cause. Maybe we need to a different load path but the idea with . was to have it ready to start. With a different one it will be necessary for the user to change it before testing. I'm not sure what would be best.

[lumin-etherlight]

Maybe having a completely separate entry point for quick testing? e.g. A file called nano-quick-test.el which sets the load-path before loading nano.el proper. Resulting in something like:

emacs -Q -l ./nano-quick-test.el

Or, update the example perhaps to be something like:

emacs -Q --eval="(add-to-list 'load-path \".\")" -l ./nano.el

A bit uglier, but less mysterious. Or, as a mix of both option, something like:

emacs -Q -l ./nano-quick-test-setup.el -l ./nano.el

[c-alpha]

I just stumbled across the same problem. I'd be in favour of @lumin-etherlight's second suggestion:

emacs -q --execute "(normal-top-level-add-to-load-path '(\".\"))" -l nano.el

It's for a one-off test to see what things look like, so the extended command line will be ok. No need for any polished solution, IMHO.

[arunaugustine]

I had the same issue. Claude code suggested the following fix:

Addition to nano.el

+;; Add current directory to load path for -q usage
+(let* ((this-file  (or load-file-name (buffer-file-name)))
+       (this-dir  (file-name-directory this-file)))
+  (add-to-list 'load-path this-dir))

[rougier]

Sorry for delay. Claude's suggestion might be the best solution actually. Do you want to make a PR?

[c-alpha]

Do you want to make a PR?

Will do.

[c-alpha]

Do you want to make a PR?

Will do.

PR #171