Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmdline should not be cached #331

Closed
giampaolo opened this issue May 23, 2014 · 8 comments
Closed

cmdline should not be cached #331

giampaolo opened this issue May 23, 2014 · 8 comments

Comments

@giampaolo
Copy link
Owner

From Psihon...@gmail.com on September 24, 2012 17:27:01

After version 0.5, property `cmdline` of Process is cacheable. 
Could you explain why so? 
There are a lot of situations, when process `cmdline` contains some valuable 
information, which should be processed/validated. 

Thanks.

p.s. update psutil from 0.4.1 to 0.6.1, with cacheable `cmdline`, completely 
crashed our mail fetching monitoring system :)

Original issue: http://code.google.com/p/psutil/issues/detail?id=331

@giampaolo
Copy link
Owner Author

From g.rodola on September 24, 2012 10:15:08

Cmdline is cached bacause is not supposed to change during the entire lifetime 
of the process (the same applies for process pid, ppid, name an exe).
How exactly your code was affected by this change?
The only difference in terms of compatibility should be that in case the 
process disappears NoSuchProcess is no longer raised.

@giampaolo
Copy link
Owner Author

From ruib...@gmail.com on January 21, 2013 19:18:38

runsvdir (of runit) alters its .cmdline to include recent stderr messages from 
all child processes.

@giampaolo
Copy link
Owner Author

From g.rodola on January 22, 2013 01:34:21

Status: Accepted

@giampaolo
Copy link
Owner Author

From g.rodola on February 27, 2013 07:25:17

Fixed in r1570 .

Status: FixedInSVN
Labels: Milestone-0.6.2

@giampaolo
Copy link
Owner Author

From g.rodola on March 02, 2013 04:13:18

Updated csets after the SVN -> Mercurial migration: r1570 == revision 
9d2f1e41eee3

@giampaolo
Copy link
Owner Author

From g.rodola on April 07, 2013 18:21:49

Status: FixedInHG

@giampaolo
Copy link
Owner Author

From g.rodola on April 11, 2013 02:17:39

Labels: -Milestone-0.6.2 Milestone-0.7.0

@giampaolo
Copy link
Owner Author

From g.rodola on April 12, 2013 11:21:26

Status: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant