Bug 130157

Summary: net-p2p/verlihub - Wrong rc start order - verlihub and mysql
Product: Ports & Packages Reporter: Morgan Wesström <freebsd-ports-bugs>
Component: Individual Port(s)Assignee: Dmitry Marakasov <amdmi3>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   

Description Morgan Wesström 2009-01-04 10:30:03 UTC 
net-p2p/verlihub is dependent on a working MySQL-server but Verlihub's startup script is missing that information so rc starts Verlihub before MySQL which doesn't work.

Fix: This small patch to Verlihub's startup script solves the problem:



However, I don't know if it's correct to add "mysql" to Verlihub's REQUIRE or add "DAEMON" to MYSQL's PROVIDE or maybe even some third alternative.--9AsF5FuzTzB3H4y8uTrWSkmTzh0wzpwoXpwbDuLi9GaaUh0j
Content-Type: text/plain; name="file.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="file.diff"

--- verlihub.orig	2009-01-04 11:10:23.000000000 +0100
+++ verlihub	2009-01-04 11:10:32.000000000 +0100
@@ -1,7 +1,7 @@
 #!/bin/sh
 
 # PROVIDE: verlihub
-# REQUIRE: DAEMON
+# REQUIRE: DAEMON mysql
 #
 # Add the following lines to /etc/rc.conf.local or /etc/rc.conf
 # to enable this service:
How-To-Repeat: Install databases/mysql51-server and net-p2p/verlihub from ports. Configure both applications and add them to /etc/rc.conf and reboot. Verlihub will start before MySQL and exit.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2009-01-04 10:50:54 UTC 
Maintainer of net-p2p/verlihub,

Please note that PR ports/130157 has just been submitted.

If it contains a patch for an upgrade, an enhancement or a bug fix
you agree on, reply to this email stating that you approve the patch
and a committer will take care of it.

The full text of the PR can be found at:
    http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/130157

-- 
Edwin Groothuis via the GNATS Auto Assign Tool
edwin@FreeBSD.org
Comment 2 Edwin Groothuis freebsd_committer freebsd_triage 2009-01-04 10:50:58 UTC 
State Changed
From-To: open->feedback

Awaiting maintainers feedback (via the GNATS Auto Assign Tool)
Comment 3 Mikle Davidkin 2009-01-10 19:50:09 UTC 
Small but needed fix! Works OK - suggested patch surely needs to be
commited....
Comment 4 Mark Linimon freebsd_committer freebsd_triage 2009-01-10 20:28:33 UTC 
State Changed
From-To: feedback->open

Maintainer approved.
Comment 5 Dmitry Marakasov freebsd_committer freebsd_triage 2009-01-11 15:28:24 UTC 
Responsible Changed
From-To: freebsd-ports-bugs->amdmi3

I'll take it.
Comment 6 Dmitry Marakasov freebsd_committer freebsd_triage 2009-01-11 22:49:36 UTC 
State Changed
From-To: open->closed

Committed. Thanks!
Comment 7 dfilter service freebsd_committer freebsd_triage 2009-01-11 22:49:41 UTC 
amdmi3      2009-01-11 22:49:27 UTC

  FreeBSD ports repository

  Modified files:
    net-p2p/verlihub     Makefile 
    net-p2p/verlihub/files verlihub.in 
  Log:
  - Add mysql dependency to the rc script
  
  PR:             130157
  Submitted by:   Morgan Wesström <freebsd-ports-bugs at pp dot dyndns dot biz>
  Approved by:    "Mikle Davidkin" <skylord at linkline dot ru> (maintainer)
  
  Revision  Changes    Path
  1.25      +1 -1      ports/net-p2p/verlihub/Makefile
  1.2       +1 -1      ports/net-p2p/verlihub/files/verlihub.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"