[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-09-26 Thread Paul Gevers
To all those involved. I am not in the position to test Ubuntu packages as I don't run that myself and my computer doesn't support hardware virtualization. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu.

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-05-07 Thread Mathew Hodson
** Changed in: cacti (Ubuntu Xenial) Importance: Undecided => Medium ** Changed in: cacti (Ubuntu Zesty) Importance: Undecided => Medium ** Changed in: cacti (Ubuntu Yakkety) Importance: Undecided => Medium ** Changed in: cacti (Ubuntu Artful) Importance: Undecided => Medium --

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-22 Thread Launchpad Bug Tracker
This bug was fixed in the package cacti - 0.8.8h+ds1-8ubuntu0.1 --- cacti (0.8.8h+ds1-8ubuntu0.1) zesty; urgency=medium * debian/patches/enable_faster_polling_than_cron.patch: split() is deprecated in PHP 7, causing the code to fail. Closes: #860271, LP: #1662027. Thanks

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-18 Thread Adam Conrad
Hello Joel, or anyone else affected, Accepted cacti into yakkety-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/cacti/0.8.8h+ds1-5ubuntu0.1 in a few hours, and then in the -proposed repository. Please help us by testing this new package. See

Re: [Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-18 Thread Nish Aravamudan
@Paul, because I didn't think of that at the time :) Sorry! -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1662027 Title: cacti vs PHP 7 vs split To manage notifications about this bug go to:

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-18 Thread Paul Gevers
@Nish, why didn't you (fake-) synced cacti in Zesty, there were no other delta's? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1662027 Title: cacti vs PHP 7 vs split To manage notifications about

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-18 Thread Adam Conrad
Hello Joel, or anyone else affected, Accepted cacti into zesty-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/cacti/0.8.8h+ds1-8ubuntu0.1 in a few hours, and then in the -proposed repository. Please help us by testing this new package. See

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-17 Thread Nish Aravamudan
I'm uploading the fix for x, y and z now. We can sync in aa once it's open. ** Description changed: + [Impact] + + * If the administrator configures cacti to poll data at a higher frequency + than the cron job that runs the poller, a fatal error is emitted. + + * As the highest cron

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-17 Thread Nish Aravamudan
Once we are able to sync the package from Debian in AA, Ubuntu will have the fix and we can SRU. ** Also affects: cacti (Ubuntu Aa-series) Importance: Undecided Status: New ** Changed in: cacti (Ubuntu Aa-series) Status: New => In Progress ** Changed in: cacti (Ubuntu

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-14 Thread Nish Aravamudan
** Changed in: cacti (Ubuntu Yakkety) Status: New => In Progress ** Changed in: cacti (Ubuntu Xenial) Status: New => In Progress ** Changed in: cacti (Ubuntu Zesty) Status: Confirmed => In Progress ** Changed in: cacti (Ubuntu Xenial) Assignee: (unassigned) => Nish

Re: [Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-14 Thread Nish Aravamudan
I will upload the same fix to Ubuntu tonight or tomorrow. Thanks Paul! -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1662027 Title: cacti vs PHP 7 vs split To manage notifications about this bug

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-14 Thread Bug Watch Updater
** Changed in: cacti (Debian) Status: Unknown => Fix Released -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1662027 Title: cacti vs PHP 7 vs split To manage notifications about this bug go

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-14 Thread Paul Gevers
Bug is now fixed in Debian and unblocked for the upcoming Stretch release. ** Bug watch added: Debian Bug tracker #860271 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=860271 ** Also affects: cacti (Debian) via http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=860271 Importance:

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-13 Thread Paul Gevers
Sleeping on it another night makes sense. I understand what I was doing wrong yesterday, I forgot to change the rrd file definition when changing the poller interval. Indeed, if someone wants to increase the frequency to something higher than once per minute, cacti fails to support this currently

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-12 Thread Paul Gevers
@Nish, if you want to fix this particular issue, I don't think you need to fix the other uses of explode. I think you could also use explode instead of split. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu.

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-12 Thread Paul Gevers
Hmm, I must also admit that I don't fully understand what this feature is for (it doesn't seem to do what I expected). Anybody that does now, can you explain why this should be fixed in older releases? -- You received this bug notification because you are a member of Ubuntu Bugs, which is

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-12 Thread Nish Aravamudan
@Paul: I was asking mostly because the upstream changes don't necessarily apply cleanly (the whole commit) because there seems to have been a global move to microtime() in the source. Is that something I should look at SRUing with this change? Or is it sufficient, in your opinion, to just backport

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-12 Thread Paul Gevers
Ouch, I forgot about this issue. @Nish: It looks like that indeed. Are you asking the last questions for a specific reason? It is the first time I see somebody ask me such a question from the Ubuntu side. I must admit that I underestimated the impact of this bug, please go ahead. -- You

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-11 Thread Nish Aravamudan
@Paul, I do see at least 6b5931130770cdfa5baa68c43a913dcabf1c3970 upstream that fixes the split() calls in question. Is it suitable to just make those changes in the versions in Ubuntu. Specifically: --- cacti-0.8.8f+ds1.orig/poller.php +++ cacti-0.8.8f+ds1/poller.php @@ -451,11 +451,9 @@ while

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-04-11 Thread Nish Aravamudan
** Also affects: cacti (Ubuntu Zesty) Importance: Undecided Status: New ** Also affects: cacti (Ubuntu Xenial) Importance: Undecided Status: New ** Also affects: cacti (Ubuntu Yakkety) Importance: Undecided Status: New -- You received this bug notification because

[Bug 1662027] Re: cacti vs PHP 7 vs split

2017-02-06 Thread Paul Gevers
Thanks for reporting this issue. The code was introduced to fix issue http://bugs.cacti.net/view.php?id=2446 in 0.8.8c. Looks like already before that time Debian and Ubuntu replaced split() by explode(), so that's why this one was missed. Code is not present in the recently released upstream