Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 5 Sep 2023 02:28:38 GMT
From:      Kyle Evans <kevans@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 5bc1e0c2285e - main - stand: lua: module names should permit more
Message-ID:  <202309050228.3852ScQk082330@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by kevans:

URL: https://cgit.FreeBSD.org/src/commit/?id=5bc1e0c2285e73fe8455bb6c72b2b40e33f5477e

commit 5bc1e0c2285e73fe8455bb6c72b2b40e33f5477e
Author:     Kyle Evans <kevans@FreeBSD.org>
AuthorDate: 2023-09-05 02:21:34 +0000
Commit:     Kyle Evans <kevans@FreeBSD.org>
CommitDate: 2023-09-05 02:28:11 +0000

    stand: lua: module names should permit more
    
    The module entries should generally allow whatever is allowed as an
    env_var in the pattern table.  Notably, we're missing periods which
    would allow proper entries for .dtb files in loader.conf that don't need
    to specify a module_name entry for it.
    
    %d in this expression is actually redundant as %w is actually
    "all alphanumerics," but I've included it for now to match the env_var
    entry.  We should really remove it from both.
    
    Reported by:    "aribi" on the forums via allanjude@
    MFC after:      1 week
---
 stand/lua/config.lua | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/stand/lua/config.lua b/stand/lua/config.lua
index 3f8fbc52cb40..8fdc805ee983 100644
--- a/stand/lua/config.lua
+++ b/stand/lua/config.lua
@@ -61,7 +61,10 @@ local MSG_FAILSYN_EOLESC = "Stray escape at end of line"
 local MSG_FAILSYN_EOLVAR = "Unescaped $ at end of line"
 local MSG_FAILSYN_BADVAR = "Malformed variable expression at position '%d'"
 
-local MODULEEXPR = '([-%w_]+)'
+-- MODULEEXPR should more or less allow the exact same set of characters as the
+-- env_var entries in the pattern table.  This is perhaps a good target for a
+-- little refactoring.
+local MODULEEXPR = '([%w%d-_.]+)'
 local QVALEXPR = '"(.*)"'
 local QVALREPL = QVALEXPR:gsub('%%', '%%%%')
 local WORDEXPR = "([-%w%d][-%w%d_.]*)"



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202309050228.3852ScQk082330>