Comment 9 for bug 2013402

Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote (last edit ):

Hi William, sorry for the delay here. I got caught up in a work related trip and let this one fall a bit behind.

I had a short discussion about this SRU with some other Ubuntu core-devs and during that conversation, I was let know that the change requiring PHP >= 8 could be frowned upon when proposed as an SRU since it could introduce a regression for some very specific use cases.

Since those use cases are custom (odd/non-supported) setups, I believe that by adjusting the SRU template we can indeed have a good case for an SRU. Therefore, I went ahead and made some adjustments to the SRU paperwork in LP: #2016016.

I then made some minor adjustments to the patch and filed https://code.launchpad.net/~athos-ribeiro/ubuntu/+source/phpmyadmin/+git/phpmyadmin/+merge/442711 so you can review that before we land it in mantic and start SRUing it along with any of the other missing patches, when applicable.

Finally, in case you are interested, check out how I changed the SRU paperwork in LP: #2016016. There is absolutely nothing wrong with the initial paperwork file there. However, being very descriptive in the test plan should make the whole SRU process smoother since the SRU reviewers will want to have a deep understanding of the problem before they accept the change.