python-daemon

#6 is_detach_process_context_required return value description reversed
Closed: Fixed Opened by zware.

Closed: Fixed
Reopen Issue

The description of the return value of is_detach_process_context_required is backwards:

 def is_detach_process_context_required():
     """ Determine whether detaching the process context is required.
-        :return: ``True`` iff the process is already detached; otherwise
-            ``False``.
+        :return: ``False`` iff the process is already detached; otherwise
+            ``True``.
         The process environment is interrogated for the following:

This is corroborated by the last line of the docstring and the code.

(I attempted to submit a pull request for this, but that functionality doesn't seem to be all there. So you get a badly hand-rendered diff in an issue instead :) )

The description of the return value of
is_detach_process_context_required is backwards

Thank you for the report. This is now corrected in ‘master’ (at commit f201d4ca8852bdcf) which will be part of the next release.
Edited by bignose

Metadata Update from @bignose:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)
Metadata
None
None
None
None
None
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.