Allow "simple names" to start with an integer #3

Parzival-3141 · 2024-01-17T22:01:19Z

This would enable expansions for shell script positional arguments, e.g.

> cat print.sh
echo $1

> print.sh foo
foo # $1 expanded to foo

It would directly solve our use case in Wanix where we use go-posix for shell preprocessing.

mapping := posix.Map{}
for i, scriptArg := range os.Args {
    mapping[strconv.Itoa(i)] = scriptArg
}
posix.Expand(shellScript, mapping)

Here's a possible implementation:

func lexStartExpansion(l *lexer) stateFn {
	c := l.next()
	switch {
	case c == eof:
		l.emit(itemText("$"))
		return nil
	case c == '{':
		l.ignore()
		l.depth++
		return lexBracketName
-	case isAlpha(c):
+	case isAlphaNum(c):
		return lexSimpleName
	}
	return nil // FIXME
}

Parzival-3141 changed the title ~~Allow "simple names to start with an integer~~ Allow "simple names" to start with an integer Jan 17, 2024

Parzival-3141 linked a pull request Jan 17, 2024 that will close this issue

Allow simple expansion variables to start with a number #4

Open

mgood mentioned this issue Jan 24, 2024

ExpandEnv should handle positional args #5

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Allow "simple names" to start with an integer #3

Allow "simple names" to start with an integer #3

Parzival-3141 commented Jan 17, 2024 •

edited

Loading

Allow "simple names" to start with an integer #3

Allow "simple names" to start with an integer #3

Comments

Parzival-3141 commented Jan 17, 2024 • edited Loading

Parzival-3141 commented Jan 17, 2024 •

edited

Loading