[Bug 926433] Re: indicator-weather leaves .pid file in /tmp and will not restart

2012-05-24 Thread Neal McBurnett
I agree that, given how often indicator-weather is crashing (daily for me it seems), this bug will make it hard for a lot of people to even restart the indicator. So please consider releasing this for precise. In the meantime, people can work around the problem by simply removing the file in

[Bug 926433] Re: indicator-weather leaves .pid file in /tmp and will not restart

2012-05-01 Thread Mantas Kriaučiūnas
** Also affects: baltix Importance: Undecided Status: New -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/926433 Title: indicator-weather leaves .pid file in /tmp and will not restart To

[Bug 926433] Re: indicator-weather leaves .pid file in /tmp and will not restart

2012-05-01 Thread Paul Broadhead
Looks like this fix didn't make it into precise. Any chance it could be updated? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/926433 Title: indicator-weather leaves .pid file in /tmp and will not

[Bug 926433] Re: indicator-weather leaves .pid file in /tmp and will not restart

2012-02-11 Thread Vadim Rutkovsky
This is not actual for 3.0 series, as we use DBus there, so no pids are created ** Changed in: weather-indicator/3.0 Status: New = Invalid -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/926433

[Bug 926433] Re: indicator-weather leaves .pid file in /tmp and will not restart

2012-02-11 Thread Vadim Rutkovsky
Thanks, Paul! Commited this as rev. 305 ** Also affects: weather-indicator Importance: Undecided Status: New ** Also affects: weather-indicator/2.0 Importance: Undecided Status: New ** Also affects: weather-indicator/3.0 Importance: Undecided Status: New **

[Bug 926433] Re: indicator-weather leaves .pid file in /tmp and will not restart

2012-02-11 Thread Paul Broadhead
Vadim. Thanks for commiting the change but you have not used my patch. In fact, I think the change you have made instead breaks the pid detection even more. Please could you have another look at my patch and how you have made the change. My patch uses the ls -1 to list just the file names, one

[Bug 926433] Re: indicator-weather leaves .pid file in /tmp and will not restart

2012-02-11 Thread Vadim Rutkovsky
Yes, you are correct. I hope, I've commited this correctly at rev. 308 --- Ubuntu Bug Squad volunteer triager http://wiki.ubuntu.com/BugSquad -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/926433

[Bug 926433] Re: indicator-weather leaves .pid file in /tmp and will not restart

2012-02-11 Thread Paul Broadhead
Much better. Thanks for the quick response. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/926433 Title: indicator-weather leaves .pid file in /tmp and will not restart To manage notifications

[Bug 926433] Re: indicator-weather leaves .pid file in /tmp and will not restart

2012-02-03 Thread Paul Broadhead
I've now checked the source code. Unfortunatly, I did not check the contents of the .pid file before deleting it. However, I can see from my directory listing that the length was only 3 characters. Looking at the pid checking code, perhaps the 3 characters matched in part some other process

[Bug 926433] Re: indicator-weather leaves .pid file in /tmp and will not restart

2012-02-03 Thread Ubuntu Foundation's Bug Bot
The attachment patch of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug