Bug 224266 - www/tomcat85: add support for jsvc umask
Summary: www/tomcat85: add support for jsvc umask
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Lars Engels
URL: https://issues.apache.org/jira/browse...
Keywords:
Depends on:
Blocks:
 
Reported: 2017-12-12 09:33 UTC by Lapo Luchini
Modified: 2017-12-19 09:31 UTC (History)
3 users (show)

See Also:
vvd: maintainer-feedback+
vvd: maintainer-feedback+


Attachments
Add support for jsvc umask (1.08 KB, patch)
2017-12-12 09:33 UTC, Lapo Luchini
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lapo Luchini 2017-12-12 09:33:41 UTC
Created attachment 188744 [details]
Add support for jsvc umask

jsvc support `-umask` parameter since 1.0.8, it can be useful e.g. to set to 0027 to have group-readable Catalina log files.

This patch (and feature request) probably applies identically also to:
- www/tomcat-devel
- www/tomcat7
- www/tomcat8
Comment 1 Vladimir Druzenko freebsd_committer freebsd_triage 2017-12-12 12:24:54 UTC
Tested - work as expected.

Can you make same report for www/tomcat-devel?
I'm the maintainer of www/tomcat85 and www/tomcat-devel ports and I'll set maintainer-feedback.
Comment 2 Lapo Luchini 2017-12-18 10:36:15 UTC
As requested, bug #224321 for www/tomcat-devel.
Comment 3 Vladimir Druzenko freebsd_committer freebsd_triage 2017-12-18 19:04:59 UTC
Waiting Lars Engels <lme@FreeBSD.org>…
Comment 4 commit-hook freebsd_committer freebsd_triage 2017-12-19 09:31:11 UTC
A commit references this bug:

Author: lme
Date: Tue Dec 19 09:30:11 UTC 2017
New revision: 456713
URL: https://svnweb.freebsd.org/changeset/ports/456713

Log:
  www/tomcat{85,-devel}

  - Add support for jsvc umask
  - Bump PORTREVISION

  jsvc support `-umask` parameter since 1.0.8, it can be useful e.g. to set to
  0027 to have group-readable Catalina log files.

  PR:		224266, 224321
  Submitted by:	Lapo Luchini
  Approved by:	maintainer

Changes:
  head/www/tomcat-devel/Makefile
  head/www/tomcat-devel/files/tomcat_devel.in
  head/www/tomcat85/Makefile
  head/www/tomcat85/files/tomcat85.in
Comment 5 Lars Engels freebsd_committer freebsd_triage 2017-12-19 09:31:29 UTC
Committed, thanks!