Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | ob_substvars(): Make package operand optional | Patrick McDermott | 2019-03-18 | 1 | -4/+12 |
| | |||||
* | ob_substvars(): Substitute binary package vars | Patrick McDermott | 2019-03-18 | 1 | -3/+16 |
| | |||||
* | ob_set_binary_substvar(): New function | Patrick McDermott | 2019-03-18 | 1 | -0/+34 |
| | |||||
* | ob_set_source_substvar(), ob_substvars(): Rename eval'ed var | Patrick McDermott | 2019-03-18 | 1 | -2/+2 |
| | |||||
* | ob_set_source_substvar(): Make sed script a global var | Patrick McDermott | 2019-03-18 | 1 | -9/+10 |
| | |||||
* | ob_set_source_substvar(): Replace here-documents with printf | Patrick McDermott | 2019-03-18 | 1 | -9/+3 |
| | |||||
* | ob_set_substvar(): Rename to ob_set_source_substvar() | Patrick McDermott | 2019-03-18 | 1 | -6/+7 |
| | |||||
* | lib/control.sh: Wrap long lines | Patrick McDermott | 2019-03-18 | 1 | -7/+10 |
| | |||||
* | ob_set_substvar(): Replace echo with a here-document | Patrick McDermott | 2019-03-18 | 1 | -2/+5 |
| | |||||
* | ob_set_substvar(): No it can't | Patrick McDermott | 2019-03-18 | 1 | -1/+0 |
| | | | | | | read only reads one line. It could be put in a loop, but then it would trim whitespace from every line (without some hacks to flag the first line and detect the last line), which is undesired. | ||||
* | Revert "ob_set_substvar(): Replace sed with read" | Patrick McDermott | 2019-03-18 | 1 | -3/+10 |
| | | | | This reverts commit 96851fd522ae3540314ac5c5d4d4fbec1448ca4b. | ||||
* | _OB_LF: Make public | Patrick McDermott | 2019-03-17 | 1 | -1/+1 |
| | |||||
* | ob_parse_control(): Fix another variable name in expansion | Patrick McDermott | 2019-03-17 | 1 | -1/+1 |
| | |||||
* | _ob_parse_changelog_error(), _ob_parse_control_error(): Drop field width | Patrick McDermott | 2019-03-17 | 1 | -2/+2 |
| | |||||
* | ob_parse_control(): Fix variable name in expansion | Patrick McDermott | 2019-03-16 | 1 | -1/+1 |
| | | | | Bug found with "set -u" in test suite. | ||||
* | Add braces to param exps in arith substs | Patrick McDermott | 2019-03-16 | 1 | -2/+2 |
| | |||||
* | Fix missing "()" after function names in @details | Patrick McDermott | 2019-03-14 | 1 | -4/+4 |
| | | | | | Also fix function name in ob_get_system_plat()'s @details and update ob_error()'s @details. | ||||
* | Protect against cmd operands beginning with "-" | Patrick McDermott | 2019-03-13 | 1 | -1/+1 |
| | |||||
* | ob_parse_changelog(), ob_parse_control(): Explicitly assign null strings | Patrick McDermott | 2019-03-13 | 1 | -1/+1 |
| | | | | | POSIX say, "If value is not specified, the variable shall be given a null value." Make this explicit, for consistency. | ||||
* | ob_substvars(): s/while true/while :/ | Patrick McDermott | 2019-03-13 | 1 | -1/+1 |
| | |||||
* | ob_substvars(), ob_init_package(): Call _ob_error_msg() | Patrick McDermott | 2019-03-13 | 1 | -1/+1 |
| | |||||
* | libopkbuild: Abort on invalid function arguments | Patrick McDermott | 2019-03-13 | 1 | -46/+21 |
| | | | | | | | | | | 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. | ||||
* | lib/control.sh, lib/package.sh: Replace echo with here-document in tr commands | Patrick McDermott | 2019-03-12 | 1 | -2/+8 |
| | |||||
* | 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_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_substvar(): Replace sed with read | Patrick McDermott | 2019-03-12 | 1 | -10/+3 |
| | |||||
* | _ob_parse_control_error(): Use _ob_warn_msg() | Patrick McDermott | 2019-03-12 | 1 | -12/+3 |
| | |||||
* | Use (or plan to use) _ob_warn_msg() | Patrick McDermott | 2019-03-11 | 1 | -2/+4 |
| | |||||
* | Revert "ob_parse_control(): Run parse error callback" | Patrick McDermott | 2019-03-11 | 1 | -6/+35 |
| | | | | This reverts commit d80290a2f8c9f069261da78afd2f275e84923d0f. | ||||
* | ob_parse_control(): Run parse error callback | Patrick McDermott | 2019-01-05 | 1 | -35/+6 |
| | |||||
* | Revert "[WIP] ob_parse_control(): Run an error callback" | Patrick McDermott | 2019-01-05 | 1 | -19/+41 |
| | | | | This reverts commit d1f79411c618a7a3d078430450631dfca85c2a07. | ||||
* | [WIP] ob_parse_control(): Run an error callback | Patrick McDermott | 2019-01-05 | 1 | -41/+19 |
| | |||||
* | _OB_LF: Define and use newline constant | Patrick McDermott | 2019-01-05 | 1 | -2/+1 |
| | |||||
* | ob_parse_control(): Replace [ commands after callbacks | Patrick McDermott | 2019-01-05 | 1 | -4/+2 |
| | |||||
* | ob_parse_control(): Update/improve field_cb documentation | Patrick McDermott | 2019-01-05 | 1 | -8/+11 |
| | |||||
* | ob_parse_control(): Add a user_data operand | Patrick McDermott | 2019-01-05 | 1 | -6/+10 |
| | |||||
* | ob_parse_control(): Pass field name and value as args | Patrick McDermott | 2019-01-05 | 1 | -17/+8 |
| | |||||
* | ob_parse_control(): Replace param prefix/suffix removals with read | Patrick McDermott | 2019-01-05 | 1 | -5/+4 |
| | |||||
* | ob_parse_control(): Replace if construct with case | Patrick McDermott | 2019-01-05 | 1 | -54/+55 |
| | |||||
* | lib/changelog.sh, lib/control.sh, lib/locale.sh: Update copyright years | Patrick McDermott | 2018-12-29 | 1 | -1/+1 |
| | |||||
* | ob_substvars(): Add a TODO comment | Patrick McDermott | 2018-12-29 | 1 | -0/+1 |
| | |||||
* | ob_substvars(): Document | Patrick McDermott | 2018-12-29 | 1 | -0/+9 |
| | |||||
* | ob_set_substvar(): Improve eval command | Patrick McDermott | 2018-12-29 | 1 | -4/+1 |
| | |||||
* | ob_set_substvar(): Document that 125 is returned if name is empty | Patrick McDermott | 2018-12-29 | 1 | -1/+1 |
| | |||||
* | ob_set_substvar(): Add a TODO comment | Patrick McDermott | 2018-12-28 | 1 | -0/+1 |
| | |||||
* | ob_set_substvar(): Fix successful return value | Patrick McDermott | 2018-12-28 | 1 | -2/+3 |
| |