Skip to content
Snippets Groups Projects
Verified Commit aa053d8d authored by Clayton Craft's avatar Clayton Craft :speech_balloon:
Browse files

Use "local" for variable scoping 

This introduces using "local" in the scripts to scope variables
locally. "local" is not defined in POSIX, so this breaks compatibility.
At least in Alpine, which uses ash as it's "POSIX shell", this is
supported.

We re-use a lot of variable names (e.g. "$file") and this can lead to
some interesting bugs if we're not careful, so it seems worth breaking
POSIX compatibility in order to get real variable scoping to avoid this
class of bugs.
parent 9a981eef
No related branches found
No related tags found
1 merge request!26bug fixes, and support gummiboot
Hide whitespace changes
Inline Side-by-side
  • Administrator @root

    mentioned in commit c3d33d99

    By Newbyte on 2023-05-28T12:12:42

     · Imported

    mentioned in commit c3d33d99

    By Newbyte on 2023-05-28T12:12:42

    Edited by Ghost User
    Toggle commit list
  • Administrator @root

    mentioned in merge request !33 (merged)

    By Newbyte on 2023-05-28T12:13:05

     · Imported

    mentioned in merge request !33 (merged)

    By Newbyte on 2023-05-28T12:13:05

    Edited by Ghost User
    Toggle commit list
  • Administrator @root

    mentioned in commit ee3e2ec2

    By Newbyte on 2023-05-28T12:13:21

     · Imported

    mentioned in commit ee3e2ec2

    By Newbyte on 2023-05-28T12:13:21

    Edited by Ghost User
    Toggle commit list
  • Administrator @root

    mentioned in commit f3583f60

    By Newbyte on 2023-05-29T16:34:50

     · Imported

    mentioned in commit f3583f60

    By Newbyte on 2023-05-29T16:34:50

    Edited by Ghost User
    Toggle commit list
  • Administrator @root

    mentioned in commit 1b58e3da

    By Pablo Correa Gomez on 2023-06-16T22:15:26

     · Imported

    mentioned in commit 1b58e3da

    By Pablo Correa Gomez on 2023-06-16T22:15:26

    Edited by Ghost User
    Toggle commit list
  • Administrator @root

    mentioned in commit b4faf154

    By Pablo Correa Gomez on 2023-06-16T22:50:41

     · Imported

    mentioned in commit b4faf154

    By Pablo Correa Gomez on 2023-06-16T22:50:41

    Edited by Ghost User
    Toggle commit list
  • Administrator @root

    mentioned in commit a1582b6a

    By Pablo Correa Gomez on 2023-06-18T16:59:25

     · Imported

    mentioned in commit a1582b6a

    By Pablo Correa Gomez on 2023-06-18T16:59:25

    Edited by Ghost User
    Toggle commit list
  • Administrator @root

    mentioned in commit 59d5d1a7

    By clayton craft on 2023-06-22T00:30:42

     · Imported

    mentioned in commit 59d5d1a7

    By clayton craft on 2023-06-22T00:30:42

    Edited by Ghost User
    Toggle commit list
  • Administrator @root

    mentioned in merge request !34 (merged)

    By clayton craft on 2023-06-22T00:31:41

     · Imported

    mentioned in merge request !34 (merged)

    By clayton craft on 2023-06-22T00:31:41

    Edited by Ghost User
    Toggle commit list
  • Administrator @root

    mentioned in commit 3ca7edec

    By clayton craft on 2023-06-23T17:34:05

     · Imported

    mentioned in commit 3ca7edec

    By clayton craft on 2023-06-23T17:34:05

    Edited by Ghost User
    Toggle commit list
  • Arnav Singh @Arnavion

    mentioned in issue pmaports#3362 (moved)

     ·

    mentioned in issue pmaports#3362 (moved)

    Toggle commit list
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment