Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 691772 - net-libs/tox - Mismatch between configs for pidfile
Summary: net-libs/tox - Mismatch between configs for pidfile
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal minor (vote)
Assignee: JoMull01
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks:
 
Reported: 2019-08-08 19:18 UTC by МАН69К
Modified: 2020-01-13 06:09 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description МАН69К 2019-08-08 19:18:03 UTC
Hello!
We have problem between 'tox-bootstrapd.conf' and 'initd' in /usr/portage/net-libs/tox/files/.

tox-bootstrapd.conf:
...
pid_file_path = "/var/run/tox-bootstrapd/tox-bootstrapd.pid"
...

initd:
...
PIDFILE="${PIDDIR}"/tox-bootstrap.pid
...

Problem with "tox-bootstrap(d).pid". I think variant with "d" more right, so need correct 'initd' file.

Without this command '/etc/init.d/tox-dht-daemon status' say: 'status: crashed'

Reproducible: Always

Steps to Reproduce:
1. install net-libs/tox
2. /etc/init.d/tox-dht-daemon start
3. /etc/init.d/tox-dht-daemon status
Actual Results:  
status: crashed

Expected Results:  
status: started
Comment 1 МАН69К 2019-10-07 12:49:09 UTC
Hi! Any progress? How I can help fix this?
Comment 2 JoMull01 2019-10-22 01:31:39 UTC
Hello
 Terribly sorry for my unreasonable delay in replying. I have been trying to get tox-0.2.10 merged into the main tree. Once that happens I will start investigating this bug and others. I think that the new version is almost ready to be merged though.
Thanks for your offer,
  Josiah M. Mullins
Comment 3 Larry the Git Cow gentoo-dev 2020-01-13 06:09:56 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=606a78b496eff09146c70ba53059d5a5c755216c

commit 606a78b496eff09146c70ba53059d5a5c755216c
Author:     Josiah Mullins <jomull01@protonmail.com>
AuthorDate: 2020-01-09 17:31:44 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2020-01-13 06:09:43 +0000

    net-libs/tox: fix to openrc init script.
    
    This commit unifies the location of the PID file for
    the daemon in both the openrc script and the daemon's
    config file. In addition,the script now checks the keys
    directory for existence and correct permissions.
    
    Signed-off-by: Josiah M. Mullins <JoMull01@protonmail.com>
    
    Closes: https://bugs.gentoo.org/691772
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 net-libs/tox/files/initd               |  6 +++++-
 net-libs/tox/files/tox-bootstrapd.conf | 11 +++++++----
 2 files changed, 12 insertions(+), 5 deletions(-)