Bug 131922 - net/asterisk16 add some OPTION, remove unnecessary dependency, fix pkg-plist
Summary: net/asterisk16 add some OPTION, remove unnecessary dependency, fix pkg-plist
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: Maxim Sobolev
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-21 02:10 UTC by Dmitry
Modified: 2009-03-18 06:23 UTC (History)
0 users

See Also:


Attachments
file.diff (2.91 KB, patch)
2009-02-21 02:10 UTC, Dmitry
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dmitry 2009-02-21 02:10:01 UTC
Add optional support curl, gsm, remove unnecessary dependency mpg123

Fix: Patch attached with submission follows:
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2009-02-21 02:10:11 UTC
Responsible Changed
From-To: freebsd-ports-bugs->sobomax

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 dfilter service freebsd_committer freebsd_triage 2009-02-27 02:08:07 UTC
sobomax     2009-02-27 02:07:57 UTC

  FreeBSD ports repository

  Modified files:
    net/asterisk16       Makefile pkg-plist 
  Log:
  Make GSM and CURL options. They are on by default, so that no PORTREVISION
  bump has been performed.
  
  PR:             ports/131922
  Submitted by:   Dmitry <dmitry2006@yandex.ru>
  
  Revision  Changes    Path
  1.117     +21 -3     ports/net/asterisk16/Makefile
  1.39      +3 -3      ports/net/asterisk16/pkg-plist
_______________________________________________
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 3 Maxim Sobolev freebsd_committer freebsd_triage 2009-02-27 02:08:47 UTC
Dmitry,

I am not sure what makes you think mpg123 dependency can be removed - 
it's still referenced from several places in the code. Please explain.

I have checked in the other part of the change (making curl and gsm an 
option).

-Maxim
Comment 4 Dmitry 2009-03-04 18:54:41 UTC
>I am not sure what makes you think mpg123 dependency can be removed -
>it's still referenced from several places in the code. Please explain.
mpg123 - used as external application, for build just do not need.
For run - using only in app_mp3, res_musiconhold as external application.
Probably better to do MPG123 option.

>I have checked in the other part of the change (making curl and gsm an option).
please fix pkg-plist:
-lib/asterisk/modules/cdr_sqlite3_custom.so
+%%WITH_SQLITE%%lib/asterisk/modules/cdr_sqlite3_custom.so

--
/Dmitry
Comment 5 Mark Linimon freebsd_committer freebsd_triage 2009-03-18 06:22:41 UTC
State Changed
From-To: open->closed

commmitted 2009-02-27.