Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix domoticz configuration #4730

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

hgy59
Copy link
Contributor

@hgy59 hgy59 commented Jul 10, 2021

Motivation: Current domoticz package does not find the scrips folder and is missing dzVents runtime
Linked issues: closes #4728, fixes #5131

Checklist

  • Build rule all-supported completed successfully
  • Package upgrade completed successfully
  • New installation of package completed successfully

Remarks

This fixes the userdata folder for domoticz, as scripts are expected under /var/packages/domoticz/target/scripts.

But another issue is uncovered here.
As scripts might be user specific and/or manually created, at least the following folders need backup/restore implementation for package updates - or all scripts are lost at package update.

  • /var/packages/domoticz/target/scripts/dzVents (all subfolders except examples)
  • /var/packages/domoticz/target/scripts/python
  • maybe more...

TODO

  • fix folder permission on DSM 7 (2021-11-26 08:24:19.267 Error: EventSystem: problem writing file: /volume1/@appstore/domoticz/scripts/dzVents/generated_scripts/dzVents Test.lua)
  • backup/restore script folders on package update
  • Validate status of Domoticz 2020.2-10 failed to install #5131

hgy59 added 7 commits April 26, 2023 20:50
- use SYNOPKG_PKGDEST instead of SYNOPKG_PKGVAR folder for userdata
- introduce PKG_DOWNLOAD_METHOD = git-repository to download full repository
- domoticz needs to be built within the repository to retrieve version information
- move scripts folder to var for backup/restore on package update
- link scripts folder into userdata that must be located in SYNOPKG_PKGDEST
- fix privileges for backup/restore on package update
- domoticz 2021.1 can only be built with gcc available in DSM 7+ toolchains
- update c-ares to v1.18.1
- update jsoncpp to v1.9.5
@th0ma7
Copy link
Contributor

th0ma7 commented Jul 3, 2023

Asking once more, any intent in pursuing this PR or can I mark domoticz as broken and erase entirely python38? Anyway, when someone will reconvene (if ever) this will have to be using a newer python...

@hgy59
Copy link
Contributor Author

hgy59 commented Jul 5, 2023

Asking once more, any intent in pursuing this PR or can I mark domoticz as broken and erase entirely python38? Anyway, when someone will reconvene (if ever) this will have to be using a newer python...

@th0ma7 please do not remove cross/python38, but you do not need to update/build/publish spk/python38 as domoticz does not need installed python 3.8 package but bundles python 3.8 into the domoticz package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Domoticz 2020.2-10 failed to install Domoticz 2020.2-10 package does not contain dzVents runtime files
2 participants