Bug 154942 - net/scribe: rc.d/thrift starts no matter what the rc.conf setting
Summary: net/scribe: rc.d/thrift starts no matter what the rc.conf setting
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: Greg Larkin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-21 17:00 UTC by Erik Steigler
Modified: 2011-05-13 15:00 UTC (History)
0 users

See Also:


Attachments
scribe_rc.patch (332 bytes, patch)
2011-02-21 17:00 UTC, Erik Steigler
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Erik Steigler 2011-02-21 17:00:18 UTC
  The scribe rc script seems to be missing the defination to check if it is enabled in /etc/rc.conf. This causes it to automatically start on boot and if someone runs /usr/local/etc/rc.d/scribe start. This also causes an unexpected operator error when it is run:
[root@erik /usr/local/etc/rc.d]# grep scribe /etc/rc.conf
[root@erik /usr/local/etc/rc.d]# ./scribe start
[: /usr/sbin/daemon: unexpected operator
Starting scribe.

Fix: The attached patch adds the rcvar variable which seems to fix the problem.
How-To-Repeat:   1. Install net/scribe
  2. Issue /usr/local/etc/rc.d/scribe start
Comment 1 Greg Larkin freebsd_committer freebsd_triage 2011-04-25 21:06:01 UTC
State Changed
From-To: open->feedback

I made some improvements to the rc.d script, because the error still 
appeared after I applied the patch.  I also noticed that the port was 
missing a dependency for devel/py-thrift.  That prevented the example 
scripts from working, although they also need some work in their shebang 
lines to reference the correct Python installation. 

Please check my diff here and let me know if it works for you: 
http://people.freebsd.org/~glarkin/diffs/scribe-2.2.diff
Comment 2 Greg Larkin freebsd_committer freebsd_triage 2011-04-25 21:08:18 UTC
Responsible Changed
From-To: freebsd-ports-bugs->glarkin

I'll take it.
Comment 3 dfilter service freebsd_committer freebsd_triage 2011-05-13 14:54:05 UTC
glarkin     2011-05-13 13:53:55 UTC

  FreeBSD ports repository

  Modified files:
    net/scribe           Makefile 
    net/scribe/files     scribe.in 
  Log:
  - Fixed rc.d script so it adheres to rc.conf setting [1]
  - Added missing dependency on devel/py-thrift
  - Bumped PORTREVISION
  
  PR:             ports/154942 (based on)
  Submitted by:   Erik Steigler <erik@okcupid.com> [1]
  Approved by:    maintainer timeout (18 days)
  
  Revision  Changes    Path
  1.6       +4 -2      ports/net/scribe/Makefile
  1.2       +16 -3     ports/net/scribe/files/scribe.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 Greg Larkin freebsd_committer freebsd_triage 2011-05-13 14:59:50 UTC
State Changed
From-To: feedback->closed

Committed with some modifications, thank you!