Bug 101962

Summary: Enforce net/asterisk dependencies
Product: Ports & Packages Reporter: amistry
Component: Individual Port(s)Assignee: Maxim Sobolev <sobomax>
Status: Closed FIXED    
Severity: Affects Only Me CC: sobomax
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
asterisk-depends.patch none

Description amistry 2006-08-13 08:30:15 UTC 

Enforce asterisk startup dependencies.  I'm not sure if this is correct.
Requires:
http://www.freebsd.org/cgi/query-pr.cgi?pr=101958
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2006-08-13 08:47:08 UTC 
Responsible Changed
From-To: freebsd-ports-bugs->sobomax

Over to maintainer
Comment 2 amistry 2006-08-13 18:28:58 UTC 
The iaxmodem was not correct.  We just need to depend on zaptel.

--- files/asterisk.sh.in.orig   Sun Aug 13 03:17:20 2006
+++ files/asterisk.sh.in        Sun Aug 13 13:25:45 2006
@@ -1,7 +1,7 @@
 #!/bin/sh
 
 # PROVIDE: asterisk
-# REQUIRE: DAEMON
+# REQUIRE: DAEMON zaptel
 # KEYWORD: shutdown
 
 #

-- 
Anish Mistry
amistry@am-productions.biz
AM Productions http://am-productions.biz/
Comment 3 Maxim Sobolev 2006-08-14 19:23:00 UTC 
This is not correct since asterisk can be built with WITHOUT_ZAPTEL or 
zaptel loading can be disabled in the rc.conf. In both those cases 
asterisk should be able to run. Especially considering that zaptel is 
still i386-only package, while asterisk is not.

-Maxim
Comment 4 Thomas Sandford 2006-10-09 14:07:03 UTC 
As Maxim states, it is incorrect for Asterisk to

# REQUIRE: zaptel

The correct way of doing this would be for Zaptel to have a line

# BEFORE: asterisk

and for Asterisk to

#PROVIDE: asterisk

-- 
Thomas Sandford
Comment 5 Maxim Sobolev freebsd_committer freebsd_triage 2007-04-16 08:57:38 UTC 
State Changed
From-To: open->closed

Asterisk is now "PROVIDE: zaptel" while zaptel is "BEFORE: asterisk", which 
should solve the problem.