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

deb cleanup #142

Closed
davidcoutadeur opened this issue Jul 18, 2024 · 1 comment · Fixed by #161
Closed

deb cleanup #142

davidcoutadeur opened this issue Jul 18, 2024 · 1 comment · Fixed by #161
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@davidcoutadeur
Copy link

The idea is to do the same thing as what have been done in self-service-password:

@davidcoutadeur davidcoutadeur added the enhancement New feature or request label Jul 18, 2024
@davidcoutadeur davidcoutadeur added this to the 0.6 milestone Jul 18, 2024
@davidcoutadeur davidcoutadeur self-assigned this Jul 18, 2024
davidcoutadeur pushed a commit that referenced this issue Jul 31, 2024
@davidcoutadeur
Copy link
Author

davidcoutadeur commented Jul 31, 2024

Here is the list of breaking change in the #161 PR, which must be brought to .deb package, and notified in the changelog / in the release notes / documentation:

  • the dependencies are now explicitly listed in the package description, including the bundled ones.
  • the configuration files are now in /etc/service-desk directory
  • the previous configuration files present in /usr/share/service-desk/conf (all .php files) are migrated to /etc/service-desk/ during the upgrade process. config.inc.php is migrated as a config.inc.php.bak file, all other php file names are preserved.
  • smarty is now a required package. See consideration about smarty 3 / smarty 4 in Update to smarty4 #158
  • some bundled dependencies have been updated:
        "components/jquery": "v3.7.1",
        "datatables.net/datatables.net-bs5": "2.0.8",
        "datatables.net/datatables.net-buttons-bs5": "3.0.2",
        "fortawesome/font-awesome": "v6.5.2",
        "ltb-project/ltb-common": "0.3.0",
        "twbs/bootstrap": "v5.3.2"
  • a minimum version of PHP is now required by the package: >=7.3
  • dependencies have been removed: php (php module for apache), apache2 (indirect dependency)
  • new basic dependencies are now required by the package: php-fpm, smarty4 | smarty3
  • hidden files (.gitignore,...) from bundled dependencies are now removed from the package
  • now, the cache is being cleaned-up during self-service-password upgrade / install
  • use relative path in unit tests
  • configuration of smarty4 will be automatically made in config.inc.php if available
  • php-fpm configuration for apache will be automatically made if apache is available during installation

davidcoutadeur pushed a commit that referenced this issue Aug 1, 2024
davidcoutadeur pushed a commit that referenced this issue Aug 1, 2024
davidcoutadeur pushed a commit that referenced this issue Aug 1, 2024
davidcoutadeur pushed a commit that referenced this issue Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant