self-service-password icon indicating copy to clipboard operation
self-service-password copied to clipboard

deb cleanup

Open davidcoutadeur opened this issue 1 year ago • 4 comments

This is the issue equivalent to spec cleanup https://github.com/ltb-project/self-service-password/issues/846, but for debian:

Here is the list of main adaptations done in the specfile, which must be brought to deb packaging:

  • [x] the dependencies are now explicitly listed in the RPM, including the bundled ones.
  • [x] the configuration files are now in /etc/self-service-password directory
  • [x] the previous configuration files present in /usr/share/self-service-password/conf (all .php files) are migrated to /etc/self-service-password/ during the upgrade process. config.inc.php is migrated as a config.inc.php.bak file, all other php file names are preserved.
  • [x] the very old configuration files, present in /usr/share/self-service-password/ are NOT migrated during the upgrade process, and must be upgraded manually. These files have been deprecated since version 0.9, release in 2015 of October. The release note must define how to migrate these configuration files
  • [x] smarty is now a required package See smarty4: https://github.com/ltb-project/self-service-password/issues/850
  • [x] some bundled dependencies have been updated. See smarty4: https://github.com/ltb-project/self-service-password/issues/850, bootstrap 5.3: https://github.com/ltb-project/self-service-password/issues/834, and these dependency list: https://github.com/ltb-project/self-service-password/issues/849
  • [x] unit tests are now run by the build process, only for fedora distribution and debian packaging
  • [x] a minimum version of PHP is now required by the package: >=7.3
  • [x] new basic dependency is now required by the package: sed
  • [x] hidden files (.gitignore,...) from bundled dependencies are now removed from the package
  • [x] now, the cache is being cleaned-up during self-service-password upgrade / install

davidcoutadeur avatar Mar 08 '24 18:03 davidcoutadeur

Done in #858

davidcoutadeur avatar Mar 11 '24 19:03 davidcoutadeur

This feature should be ready. Only thing missing is updating the requirement of smarty4. It can be done in https://github.com/ltb-project/self-service-password/issues/850

davidcoutadeur avatar Mar 12 '24 18:03 davidcoutadeur

TODO:

  • modify documentation (installation process)
  • add a release note in the documentation to describe the update process
  • require optionally smarty4 (preferred) or smarty3

davidcoutadeur avatar Mar 14 '24 15:03 davidcoutadeur

Last TODO should be ok with last commit

davidcoutadeur avatar Mar 14 '24 17:03 davidcoutadeur