Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
... | |||||
* | opkbuild: Add more return statements | Patrick McDermott | 2019-03-13 | 1 | -23/+35 |
| | |||||
* | opkbuild: Add return statements after ob_error() calls | Patrick McDermott | 2019-03-13 | 1 | -13/+35 |
| | |||||
* | ob-*: Add return statements after ob_error() calls | Patrick McDermott | 2019-03-13 | 4 | -18/+67 |
| | |||||
* | ob_parse_changelog(), ob_parse_control(): Explicitly assign null strings | Patrick McDermott | 2019-03-13 | 2 | -2/+2 |
| | | | | | POSIX say, "If value is not specified, the variable shall be given a null value." Make this explicit, for consistency. | ||||
* | TODO: Add xz-related tasks | Patrick McDermott | 2019-03-13 | 1 | -0/+5 |
| | |||||
* | opkbuild, ob-*: set -e | Patrick McDermott | 2019-03-13 | 11 | -1/+20 |
| | |||||
* | ob_substvars(): s/while true/while :/ | Patrick McDermott | 2019-03-13 | 1 | -1/+1 |
| | |||||
* | ob_parse_dep(): Fix crash on dep without version, arches, or plats | Patrick McDermott | 2019-03-13 | 1 | -4/+4 |
| | |||||
* | tests/*.sh: set -e | Patrick McDermott | 2019-03-13 | 7 | -0/+14 |
| | |||||
* | Don't let libopkbuild errors cause exit with -e | Patrick McDermott | 2019-03-13 | 7 | -41/+38 |
| | |||||
* | ob_substvars(), ob_init_package(): Call _ob_error_msg() | Patrick McDermott | 2019-03-13 | 3 | -3/+2 |
| | |||||
* | ob_error(): Make non-fatal | Patrick McDermott | 2019-03-13 | 2 | -5/+4 |
| | |||||
* | libopkbuild: Abort on invalid function arguments | Patrick McDermott | 2019-03-13 | 10 | -356/+159 |
| | | | | | | | | | | Shift arguments and abort instead of returning 125. Incorrect numbers of function arguments suggest application/library incompatibilities or serious errors by the application developer. Either way, the application developer should be made immediately aware of this (and not allowed to simply not check return values), and continuing to run and handle any further API calls may be unsafe. | ||||
* | _ob_abort(): New internal function to crash hard | Patrick McDermott | 2019-03-13 | 1 | -0/+7 |
| | |||||
* | TODO: echo is gone and IFS assignments are OK | Patrick McDermott | 2019-03-13 | 1 | -3/+0 |
| | |||||
* | tests/ob_parse_dep.sh: Replace echo and improve quotes in eval cmd | Patrick McDermott | 2019-03-13 | 1 | -2/+2 |
| | |||||
* | ob-unpacksource: Replace echo with here-document | Patrick McDermott | 2019-03-13 | 1 | -1/+5 |
| | | | | And drop the unnecessary grep. | ||||
* | ob-unpacksource: Make sed BRE POSIX-conformant | Patrick McDermott | 2019-03-13 | 1 | -1/+1 |
| | |||||
* | ob-unpacksource: Improve upstream archive counting | Patrick McDermott | 2019-03-13 | 1 | -6/+6 |
| | |||||
* | ob-gencontrol: Replace echo with here-document | Patrick McDermott | 2019-03-13 | 1 | -1/+4 |
| | |||||
* | ob-buildopk: Replace echo with printf | Patrick McDermott | 2019-03-13 | 1 | -1/+1 |
| | |||||
* | ob_match_arch(): Use clearer variable names | Patrick McDermott | 2019-03-13 | 1 | -9/+9 |
| | |||||
* | ob_match_arch(): Replace generated ERE with native SCL | Patrick McDermott | 2019-03-13 | 1 | -8/+20 |
| | |||||
* | _ob_validate_*(): Replace echo with here-document in grep commands | Patrick McDermott | 2019-03-12 | 1 | -6/+11 |
| | |||||
* | _ob_get_upstreamver(), _ob_get_distrev(): Replace echo with printf | Patrick McDermott | 2019-03-12 | 1 | -2/+2 |
| | |||||
* | _ob_get_doc_package_2(): Replace echo with printf | Patrick McDermott | 2019-03-12 | 1 | -1/+1 |
| | |||||
* | ob_reduce_deps(): Replace echo with printf | Patrick McDermott | 2019-03-12 | 1 | -1/+1 |
| | |||||
* | lib/package.sh: Update copyright years | Patrick McDermott | 2019-03-12 | 1 | -1/+1 |
| | |||||
* | ob_get_binary_packages(): Drop echo along with unnecessary assignment | Patrick McDermott | 2019-03-12 | 1 | -4/+2 |
| | |||||
* | ob_get_binary_packages(), _ob_set_binary_parameter(): Replace echo and grep ↵ | Patrick McDermott | 2019-03-12 | 1 | -9/+10 |
| | | | | with case | ||||
* | ob_get_binary_packages(): Fix indentation | Patrick McDermott | 2019-03-12 | 1 | -2/+2 |
| | |||||
* | lib/control.sh, lib/package.sh: Replace echo with here-document in tr commands | Patrick McDermott | 2019-03-12 | 2 | -15/+41 |
| | |||||
* | ob_parse_control(): Rewrite missing fields handling | Patrick McDermott | 2019-03-12 | 1 | -6/+7 |
| | | | | Get rid of echo and sed. | ||||
* | ob_parse_control(): Rewrite field checking | Patrick McDermott | 2019-03-12 | 1 | -23/+26 |
| | | | | | | | Get rid of the echo and sed pipeline and use case constructs instead of "[" commands with "${parameter%word}" expansions. The new logic also will only report duplicate unknown fields once. | ||||
* | *ob_[gs]et_*_parameter(): Make validation more immediately clear | Patrick McDermott | 2019-03-12 | 1 | -20/+12 |
| | |||||
* | TODO: eval commands have been evaluated | Patrick McDermott | 2019-03-12 | 1 | -1/+0 |
| | |||||
* | _ob_set_*_parameter(): Don't expand escaped value early in eval | Patrick McDermott | 2019-03-12 | 1 | -8/+2 |
| | |||||
* | ob_get_*_parameter(): Drop echo and improve quotes in eval cmds | Patrick McDermott | 2019-03-12 | 1 | -2/+2 |
| | |||||
* | ob_parse_version(): Improve eval safety | Patrick McDermott | 2019-03-12 | 1 | -4/+4 |
| | | | | | | | There likely wasn't a vulnerability here since versions are validated first, but unnecessary expansions like this in eval commands are bad practice, and a bug in the validation could have led to a vulnerability here. | ||||
* | ob_parse_dep(): Drop echo and improve quotes in eval cmds | Patrick McDermott | 2019-03-12 | 1 | -2/+2 |
| | |||||
* | TODO: Add non-fatal on_error() task | Patrick McDermott | 2019-03-12 | 1 | -1/+3 |
| | | | | Also update copyright years. | ||||
* | en_US/libopkbuild_2: Add invalid substvar message | Patrick McDermott | 2019-03-12 | 1 | -0/+1 |
| | | | | Should have been done in commit b857ead. | ||||
* | ob_substvars(): Replace echo in eval command | Patrick McDermott | 2019-03-12 | 1 | -1/+1 |
| | |||||
* | ob_set_substvar(): Don't allow "_" in variable names | Patrick McDermott | 2019-03-12 | 1 | -2/+2 |
| | | | | | The character was allowed due to the details of the implementation, but the SPF 2.0 specification doesn't allow it. | ||||
* | ob_substvars(): Validate variable names | Patrick McDermott | 2019-03-12 | 1 | -0/+11 |
| | |||||
* | ob_set_substvar(): Make validation more immediately clear | Patrick McDermott | 2019-03-12 | 1 | -4/+2 |
| | | | | | | The original version in 000a27ff used a "${parameter:-word}" default value expansion to detect an empty string. Instead, let the case construct match empty strings directly. | ||||
* | ob_set_text_domain(): Update @return doc | Patrick McDermott | 2019-03-12 | 1 | -1/+1 |
| | |||||
* | lib/output.sh: Update copyright years | Patrick McDermott | 2019-03-12 | 1 | -1/+1 |
| | | | | Approximately none of the 2012 code remains. | ||||
* | ob_error(), ob_warn(), ob_info(): Internationalize | Patrick McDermott | 2019-03-12 | 3 | -4/+8 |
| | |||||
* | ob_error(), ob_warn(), ob_info(): Rewrite | Patrick McDermott | 2019-03-12 | 1 | -17/+12 |
| |