Bug 130157 - net-p2p/verlihub - Wrong rc start order - verlihub and mysql
Summary: net-p2p/verlihub - Wrong rc start order - verlihub and mysql
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Dmitry Marakasov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-01-04 10:30 UTC by Morgan Wesström
Modified: 2009-01-11 22:50 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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"