Glow-Lang / glow

The Glow language for Blockchain Decentralized Applications.

Home Page:https://glow-lang.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Use bash for ggxi script - [merged]

kwannoel opened this issue · comments

In GitLab by @plotnick on Jun 8, 2021, 23:18

Merges afp/ggxi -> master

This script uses arrays, which are a Bash feature not available in a strict POSIX shell (e.g., Debian's /bin/sh).

If we would like to not depend on Bash, please just let me know in a comment, and I'll dispense with the use of arrays.

In GitLab by @plotnick on Jun 8, 2021, 23:18

assigned to @plotnick

In GitLab by @plotnick on Jun 8, 2021, 23:51

added 1 commit

Compare with previous version

In GitLab by @kwanzknoel on Jun 8, 2021, 23:51

Commented on ggxi line 1

#!/usr/bin/env bash

The executable bash does not exist at the path /usr/bash default on nix.

See: https://discourse.nixos.org/t/add-bin-bash-to-avoid-unnecessary-pain/5673

In GitLab by @plotnick on Jun 8, 2021, 23:52

Commented on ggxi line 1

Good suggestion, thanks. Fixed in 7e25e7a.

In GitLab by @kwanzknoel on Jun 8, 2021, 23:55

approved this merge request

In GitLab by @plotnick on Jun 9, 2021, 00:23

added 1 commit

Compare with previous version

In GitLab by @plotnick on Jun 9, 2021, 00:39

resolved all threads

In GitLab by @plotnick on Jun 9, 2021, 00:40

mentioned in commit 4634bdc