~jaawerth

Recent activity

Re: [PATCH 2/2] fix macrodebug sandboxing regression 19 days ago

From Jesse to ~technomancy/fennel

On Fri, Oct 2, 2020 at 11:58 AM Phil Hagelberg <phil@hagelb.org> wrote:
>
> Thanks for looking into this! I've applied and pushed these patches.
>
> For clarity, was the raw* change needed to fix the bug, or was that an
> unrelated change that you just happened to notice while working on this?

You guessed it! I initially added the raw* functions after getting an using
rawget to check a key in the compiler-env while attempting to treat the two
problems as one. I removed the usage after isolating the fix for macrodebug,
but left the raw* functions in place under the supposition that the omission
was an oversight. Plus, rawget could end up being useful for checking the
env without firing the metatable warning/error.

[PATCH 2/2] fix macrodebug sandboxing regression 20 days ago

From Jesse Wertheim to ~technomancy/fennel

Addresses [sourcehut issue 11](https://todo.sr.ht/~technomancy/fennel/11).

Macrodebug was failing to require fennelview in the standalone CLI due
to the use of `package.preload` generated from --require-as-include.

To address this, added the native `require` to the compiler-env for
built-in macros.
---
 changelog.md   |  2 ++
 src/fennel.fnl |  5 +++--
 test/macro.fnl | 20 +++++++++++++++++++-
 3 files changed, 24 insertions(+), 3 deletions(-)

diff --git a/changelog.md b/changelog.md
[message trimmed]

[PATCH 1/2] safe-compiler-env: add missing raw* globals 20 days ago

From Jesse Wertheim to ~technomancy/fennel

Added `rawget`, `rawset`, `rawlen`, and `rawequal` to safe-compiler-env
---
 src/fennel/specials.fnl | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/fennel/specials.fnl b/src/fennel/specials.fnl
index 555e3c8..a3a25c3 100644
--- a/src/fennel/specials.fnl
+++ b/src/fennel/specials.fnl
@@ -833,7 +833,8 @@ Method name doesn't have to be known at compile-time; if it is, use
(local safe-compiler-env
       (setmetatable {: table : math : string : pairs : ipairs : assert : error
                      : select : tostring : tonumber : pcall : xpcall : next
                      : print : type :bit _G.bit : setmetatable : getmetatable}
[message trimmed]

Re: Issues with readline a month ago

From Jesse Wertheim to ~technomancy/fennel

On Tue, Sep 8, 2020 at 5:44 PM Changa Damany Imara <cdimara@gmail.com> wrote:
>
> I've installed fennel and readline in my LuaRocks and fennel still won't
> recognize it.
> ..
> Any ideas on what might be happening. I'm on a Mac. Installed Lua via
> homebrew. Everything else came in through LuaRocks.

Thanks for bringing this up! On my (arch linux) system, a global
luarocks install puts things in /usr/share/lua/5.x/readline.lua, which
is also one of the paths Lua will automatically set on package.path
when LUA_PATH is unset or contains ';;'. However, the default
locations and where luarocks puts its global installs is likely
somewhat system-dependent, and it sounds like on homebrew there may be