jeans/LibreELEC.tv
jeans/LibreELEC.tv
1
0
Fork 0
mirror of https://github.com/LibreELEC/LibreELEC.tv.git synced 2025-07-24 11:16:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

packages/readme.md: add reference to tools/pkgcheck

Browse Source
This commit is contained in:
MilhouseVH 2018-12-30 00:32:45 +00:00
parent 1765a1013d
commit b93923854d
1 changed files with 14 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
packages/readme.md
View File

@ -212,7 +212,7 @@ Lastly, the following variables are assigned during `scripts/build` but some pac
BOOTSTRAP_MESON_OPTS
```
###### Example
#### Example
```
configure_package() {
# now we know where we're building, assign a value
@ -235,6 +235,19 @@ post_makeinstall_target() {
}
```
#### tools/pkgcheck
Use `tools/pkgcheck` to verify packages. It detects the following issues:
Issue | Level | Meaning |
| :--- | :----: | ---- |
| late binding violation | FAIL | Late binding variables referenced outside of a function - see [late binding](https://github.com/LibreELEC/LibreELEC.tv/blob/master/packages/readme.md#late-binding-variable-assignment) |
| duplicate function def | FAIL | Function defined multiple times, only last definition will be used |
| bad func - missing brace | FAIL | Opening brace (`{`) for function definition should be on same line as the function def, ie. `pre_configure_target() {` |
| intertwined vars & funcs | WARN | Variable assignments and logic is intertwined with functions - this is cosmetic, but variables and logic should be specified before all functions |
| unknown function | WARN | Could be a misspelled function, ie. `per_configure_target() {` which might fail silently.|
## Add a new package to the Image
1. Think about, why you need it in the image.
* new multimedia tool