Opened 15 years ago
Closed 15 years ago
#517 closed defect (fixed)
Running gli.sh script (Linux) throws errors - needs modifications
| Reported by: | ak19 | Owned by: | oranfry |
|---|---|---|---|
| Priority: | moderate | Milestone: | Book Release |
| Component: | GLI | Severity: | major |
| Keywords: | Cc: |
Description
The function definitions for "get_version" and "check_installation" need parentheses after them (get_version() and check_installation()) else the GLI script throws errors upon running. The removal of the () seems to have been a recent change to do with the gli.sh commit "make sure gli.sh is run with bash not sh or other interpretters, and took out erroneous brackets from function definitions". There is obviously an important reasons for why the () have been removed, but as the script is at present it doesn't run without them being reintroduced.
Change History (2)
comment:1 by , 15 years ago
comment:2 by , 15 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Oran checked over the file and found that the round brackets could be reintroduced as they're just as acceptable in the bash world as no brackets in function definition. Gli.sh launches fine now.