Bug 116729

Summary: [PATCH] www/tomcat{41,55,6}: fix 2 wrapper script problems
Product: Ports & Packages Reporter: Alejandro Pulver <alepulver>
Component: Individual Port(s)Assignee: Alejandro Pulver <alepulver>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
tomcat6.diff none

Description Alejandro Pulver freebsd_committer freebsd_triage 2007-09-29 06:50:08 UTC

When I was taking code from the wrapper script for a new port (databases/exist), I saw 2 problems in it:

1) When the daemon is not running, 'procname' is empty and the call to check_pidfile fails.
2) The use of *$procname* causes shell expansion, and together with the previous one, generates an error because ** matches every file in the directory.

Note: these errors do not stop the script from working, but might produce strange results (for example the output) in some situations.

How-To-Repeat: 

Run when the daemon is not started:

${PREFIX}/etc/rc.d/tomcat66.sh forcestop

Apply the patch, and run again.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2007-09-29 12:25:31 UTC
Responsible Changed
From-To: freebsd-ports-bugs->alepulver

Submitter has GNATS access
Comment 2 Edwin Groothuis freebsd_committer freebsd_triage 2007-09-29 12:25:38 UTC
State Changed
From-To: open->feedback

Awaiting maintainers feedback
Comment 3 dfilter service freebsd_committer freebsd_triage 2007-09-30 23:04:11 UTC
alepulver    2007-09-30 22:04:02 UTC

  FreeBSD ports repository

  Modified files:
    www/tomcat41/files   tomcat41.sh.in 
    www/tomcat55/files   tomcat55.sh.in 
    www/tomcat6/files    tomcat6.sh.in 
  Log:
  - Fix problems with the "status" command in the wrapper script when the daemon
    is not started.
  
  PR:             ports/116729
  Submitted by:   alepulver (myself)
  Approved by:    "Tong Liu" <nemoliu@gmail.com> (maintainer of tomcat{41,6}),
                  Tomas Verbaitis <tomasv@megalogika.lt> (maintainer of tomcat55)
  
  Revision  Changes    Path
  1.6       +5 -2      ports/www/tomcat41/files/tomcat41.sh.in
  1.7       +5 -2      ports/www/tomcat55/files/tomcat55.sh.in
  1.2       +5 -2      ports/www/tomcat6/files/tomcat6.sh.in
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 4 Alejandro Pulver freebsd_committer freebsd_triage 2007-09-30 23:04:14 UTC
State Changed
From-To: feedback->closed

Committed. Thanks!