Opened 12 years ago
Closed 2 years ago
#10131 closed enhancement (invalid)
Add option to specifiy path for twistd's process ID (.pid) file
Reported by: | Christopher Schwan | Owned by: | tba |
---|---|---|---|
Priority: | minor | Milestone: | sage-duplicate/invalid/wontfix |
Component: | notebook | Keywords: | |
Cc: | Frédéric Chapoton | Merged in: | |
Authors: | Reviewers: | Dima Pasechnik | |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
I have created a small patch which allows one to specify the location of twistd's PID file - this is useful when running the notebook as a service (which place their pid files e.g. at /var/run/).
The patch does not change the default behavior (placing it to ${HOME}/.sage/sage_notebook.sagenb/twistd.pid) and documents the new option.
Attachments (1)
Change History (11)
Changed 12 years ago by
Attachment: | sagenb-0.8.2-add-pidfile-option.patch added |
---|
comment:1 follow-up: 2 Changed 12 years ago by
comment:2 Changed 10 years ago by
Replying to fbissey:
Will need rebasing for 4.7.
For Sage 5.x as well I guess.
Go ahead, kiwifb! ;-)
comment:3 Changed 10 years ago by
We never actually used that in sage-on-gentoo but I'll see if I can get it into shape soonish. I have a number of things on my plate and that's not near the top.
comment:4 Changed 9 years ago by
Milestone: | sage-5.11 → sage-5.12 |
---|
comment:5 Changed 9 years ago by
Milestone: | sage-6.1 → sage-6.2 |
---|
comment:6 Changed 9 years ago by
Milestone: | sage-6.2 → sage-6.3 |
---|
comment:7 Changed 8 years ago by
Milestone: | sage-6.3 → sage-6.4 |
---|
comment:8 Changed 2 years ago by
Cc: | Frédéric Chapoton added |
---|---|
Milestone: | sage-6.4 → sage-duplicate/invalid/wontfix |
Status: | new → needs_review |
Proposing to close all sagenb tickets as outdated, so that all remaining open tickets in the notebook component are about the Jupyter notebook.
comment:9 Changed 2 years ago by
Reviewers: | → Dima Pasechnik |
---|---|
Status: | needs_review → positive_review |
comment:10 Changed 2 years ago by
Resolution: | → invalid |
---|---|
Status: | positive_review → closed |
Will need rebasing for 4.7.