Bug 194465 - sysutils/linux-crashplan overwrites its run.conf
Summary: sysutils/linux-crashplan overwrites its run.conf
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: William Grzybowski
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-19 12:52 UTC by Julie Koubova
Modified: 2014-10-23 00:44 UTC (History)
2 users (show)

See Also:
julie: maintainer-feedback? (wg)


Attachments
set poll selector from rc script instead of during install (1.98 KB, patch)
2014-10-19 12:52 UTC, Julie Koubova
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Julie Koubova 2014-10-19 12:52:03 UTC
Created attachment 148464 [details]
set poll selector from rc script instead of during install

Hi! CrashPlan recently started trying to use epoll, and I noticed that its run.conf file doesn't contain the -D....PollSelector flag. Maybe CrashPlan's update mechanism modifies that file, overwriting the port changes?

Attached is a patch that updates the run.conf file from the rc.d start script instead during install. It seems to work on my machine.

Thanks
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2014-10-19 12:52:03 UTC
Auto-assigned to maintainer wg@FreeBSD.org
Comment 2 Bradley T. Hughes freebsd_committer freebsd_triage 2014-10-20 10:49:41 UTC
I ran into the same problem. I think this happened as part of the 3.6.4 auto-update.
Comment 3 Julie Koubova 2014-10-21 13:20:04 UTC
the patch breaks "status" command. it needs to contain the following line to forward it to the CrashPlanEngine script:

status_cmd="${command} status"
Comment 4 commit-hook freebsd_committer freebsd_triage 2014-10-23 00:37:04 UTC
A commit references this bug:

Author: wg
Date: Thu Oct 23 00:36:48 UTC 2014
New revision: 371372
URL: https://svnweb.freebsd.org/changeset/ports/371372

Log:
  sysutils/linux-crashplan: make sure poll selector is set from rc

  - CrashPlan in place update can overwrite our run.conf preventing it
    from working [1]
  - Update to 3.6.4

  PR:		194465 [1]

Changes:
  head/sysutils/linux-crashplan/Makefile
  head/sysutils/linux-crashplan/distinfo
  head/sysutils/linux-crashplan/files/crashplan.in
Comment 5 Julie Koubova 2014-10-23 00:44:06 UTC
Perfect! Thank you, William! :)