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

redesign package folders for DSM7 compatibility and use alternate TMPDIR for all packages #5094

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hgy59
Copy link
Contributor

@hgy59 hgy59 commented Jan 22, 2022

Description

This is a redesign of the framework generated installers

  1. define installer variables SYNOPKG_PKGVAR, SYNOPKG_PKGTMP and SYNOPKG_PKGHOME for DSM7 compatibility of DSM5 and DSM6 packages
  2. create home and tmp folders for DSM7 compatibility of DSM5 and DSM6 packages
  3. implement alternate TMPDIR for all packages and remove USE_ALTERNATE_TMPDIR variable (related to Internal updater of Radarr doesn't work to update to 4.x versions #5089)

§ 1. and 2. are based on the Package Filesystem, documented by synology (https://help.synology.com/developer-guide/integrate_dsm/fhs.html)

Checklist

  • Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Bug fix
  • New Package
  • Package update
  • Includes small framework changes
  • This change requires a documentation update (e.g. Wiki)

@hgy59 hgy59 changed the title redesign package folders for DSM7 compatibility redesign package folders for DSM7 compatibility and use alternate TMPDIR for all packages Jan 22, 2022
@bakerboy448
Copy link

this would also fix #5081

- implement alternate TMPDIR for all packages and remove USE_ALTERNATE_TMPDIR variable
- create home and tmp folders for DSM7 compatibility
- DSM6 and DSM5: define installer variables SYNOPKG_PKGVAR, SYNOPKG_PKGTMP and SYNOPKG_PKGHOME for DSM7 compatibility
@Safihre
Copy link
Contributor

Safihre commented Nov 2, 2022

@hgy59 Don't we still need this?

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

Successfully merging this pull request may close these issues.

3 participants