Page MenuHomeFreeBSD
Differential D3211

Create an rc.d script for seahub.
ClosedPublic
Actions

Authored by araujo on Jul 27 2015, 1:41 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Oct 17, 10:17 PM
Unknown Object (File)
Fri, Oct 17, 3:49 PM
Unknown Object (File)
Wed, Oct 8, 4:18 PM
Unknown Object (File)
Sun, Oct 5, 9:53 PM
Unknown Object (File)
Sun, Oct 5, 12:33 PM
Unknown Object (File)
Sun, Oct 5, 11:20 AM
Unknown Object (File)
Sun, Oct 5, 10:59 AM
Unknown Object (File)
Sun, Oct 5, 10:09 AM
View All Files
Subscribers
None

Details

Reviewers
koobs
bapt
Commits
rP393052: - Add an rc.d script to be able to start seahub.
Summary

This is a startup script that can be called on rc.conf to enable start
seahub at boot time as well as make it more FreeBSD style.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

araujo updated this revision to Diff 7354.Jul 27 2015, 1:41 AM
araujo retitled this revision from to Create an rc.d script for seafile..
araujo updated this object.
araujo edited the test plan for this revision. (Show Details)
araujo retitled this revision from Create an rc.d script for seafile. to Create an rc.d script for seahub..Jul 27 2015, 1:46 AM
araujo updated this object.
araujo added a reviewer: bapt.Jul 27 2015, 1:55 AM

I'm sending an email to the maintainer yan_jingfeng@yahoo.com.

araujo added a reviewer: koobs.Jul 27 2015, 2:00 AM
koobs accepted this revision.Jul 27 2015, 3:39 AM
koobs edited edge metadata.

Looks OK to me, I have not tested the rc.d script for functionality or correctness

www/seahub/Makefile
9

Independent of this revision, I'd like to see these depend on the port versions of these Python packages, or at least understand why they haven't been

This revision is now accepted and ready to land.Jul 27 2015, 3:39 AM
bapt requested changes to this revision.Jul 27 2015, 5:50 AM
bapt edited edge metadata.
bapt added inline comments.
www/seahub/files/seahub.in
33

unusal in rc script land

hardcodes /usr/local

btw there is no need to make seahub_path modifiable by end user

This revision now requires changes to proceed.Jul 27 2015, 5:50 AM
araujo updated this revision to Diff 7364.Jul 27 2015, 6:07 AM
araujo edited edge metadata.

Fix %%PREFIX%% spotted by @bapt.

araujo updated this revision to Diff 7366.Jul 27 2015, 6:18 AM
araujo edited edge metadata.

Address all issues spotted by @bapt.

Closed by commit rP393052: - Add an rc.d script to be able to start seahub. (authored by araujo). · Explain WhyJul 28 2015, 4:02 AM
This revision was automatically updated to reflect the committed changes.
araujo mentioned this in rP393052: - Add an rc.d script to be able to start seahub..

Revision Contents
Changeset List

PathSize
www/
seahub/
2 lines
files/
72 lines

Diff 7366

View Options

www/seahub/Makefile

Loading...
View Options

www/seahub/files/seahub.in

Loading...