Bug 291306 - www/redmine60: Use puma 7, fix install directories
Summary: www/redmine60: Use puma 7, fix install directories
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Xin LI
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-11-30 19:04 UTC by Matt Kempe
Modified: 2025-11-30 23:05 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (delphij)


Attachments
Proposed patch (9.75 KB, patch)
2025-11-30 19:05 UTC, Matt Kempe
no flags Details | Diff
Updated patch (9.76 KB, patch)
2025-11-30 19:07 UTC, Matt Kempe
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Kempe 2025-11-30 19:04:08 UTC
This proposed patch corrects an unreported error wherein a public/assets directory is not created, causing significant runtime errors. Starting with redmine 6.0, the public/plugin_assets directory is not used, but public/assets instead. The tmp/pdf directory is also needed, per the redmine installation instructions.

Additionally, now that the port uses rails-72, puma 7 is automatically included, and redmine runs successfully without puma6. The other rubygem dependencies are updated in the Makefile to reflect the contents of the Gemfile.

The DB option is broken out to one of the four supported by redmine, via Makefile edits to standardize Postgres option prefix, and add sqlite and sql server options. Default remains mysql.

Also, the database.yml.sample file is patched to be more informative, and include all the available information contained in the redmine installation instructions. 

Lastly, re-ordered Makefile to remove lint found by portclippy.

Build logs:

With SQL server option set, and all other options unset:
https://pkg.compasses.xyz/data/143Ramd64-default-baseSSL/2025-11-29_22h40m07s/logs/redmine60-6.0.7_1.log

With sqlite, markdown and puma options set, magick unset:
https://pkg.compasses.xyz/data/143Ramd64-default-baseSSL/2025-11-26_09h21m21s/logs/redmine60-6.0.7_1.log

With PGSQL, magick, markdown, and puma set:
https://pkg.compasses.xyz/data/143Ramd64-default-baseSSL/2025-11-30_13h45m58s/logs/redmine60-6.0.7_1.log

With default options set:
https://pkg.compasses.xyz/data/143Ramd64-default-baseSSL/2025-11-30_13h48m09s/logs/redmine60-6.0.7_1.log
Comment 1 Matt Kempe 2025-11-30 19:05:06 UTC
Created attachment 265734 [details]
Proposed patch
Comment 2 Matt Kempe 2025-11-30 19:07:27 UTC
Created attachment 265735 [details]
Updated patch

Fixed typo in commit message.
Comment 3 commit-hook freebsd_committer freebsd_triage 2025-11-30 23:01:29 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=da111ef51c2f5c7d4058ced2475a19d0eab95c19

commit da111ef51c2f5c7d4058ced2475a19d0eab95c19
Author:     Matt Kempe <fsbruva@yahoo.com>
AuthorDate: 2025-11-30 19:02:24 +0000
Commit:     Xin LI <delphij@FreeBSD.org>
CommitDate: 2025-11-30 23:00:29 +0000

    www/redmine60: Use puma 7, fix install directories

    This commit also modernizes the port's dependencies and configuration
    options.

    Redmine 6.0 has changed to use a slightly different directory structure
    from previous versions.  The installation now creates public/assets and
    tmp/pdf directories that Redmine 6.x expects, preventing runtime errors
    related to missing directories.

    Support for Puma 7 allows users to benefit from the latest web server
    performance improvements among other enhancements.

    Database configuration is now clearer with mutually exclusive options that
    prevent misconfiguration. SQLite and SQL Server support give users more
    deployment flexibility, though SQLite is clearly marked as unsuitable for
    production use.

    Use the standardized "PGSQL" naming for PostgreSQL which aligns with
    other ports.

    The new database.yml.sample patch helps users avoid common configuration
    mistakes by providing inline documentation for connection settings, ports,
    and encoding options.

    PR:             291306

 www/redmine60/Makefile                             | 50 ++++++++++++++--------
 .../files/patch-config_database.yml.sample (new)   | 39 +++++++++++++++++
 www/redmine60/files/sql.rb (new)                   |  2 +
 www/redmine60/files/sqlite3.rb (new)               |  1 +
 www/redmine60/pkg-message                          |  6 +++
 www/redmine60/pkg-plist                            |  7 ++-
 6 files changed, 85 insertions(+), 20 deletions(-)
Comment 4 Xin LI freebsd_committer freebsd_triage 2025-11-30 23:05:42 UTC
Committed, thanks!