[Twisted-Python] [INCOMPATIBLE CHANGE] Removing Gadfly support from adbapi

[Amber "Hawkie" Brown]

So, apparently, a long time ago, there was a database called Gadfly, and Twisted used to test adbapi against it. However, since then, we've moved on from Python versions that gadfly supports, so it outright doesn't work now -- not buggy, not weird -- just doesn't even import.

This doesn't change any user facing code, but does change some tests, and even though it's covered under the compatibility policy, I thought I'd put it on the ML anyway. :)

You can find the code at https://twistedmatrix.com/trac/ticket/8119. We do not require any signoffs to remove it, this email is purely informational.

- Amber

[Jason Williams]

We don't use Gadfly but I really appreciate you mentioning the changes like this on the ML. :)

Sent via iPhone

Post by Amber "Hawkie" Brown
So, apparently, a long time ago, there was a database called Gadfly, and Twisted used to test adbapi against it. However, since then, we've moved on from Python versions that gadfly supports, so it outright doesn't work now -- not buggy, not weird -- just doesn't even import.
This doesn't change any user facing code, but does change some tests, and even though it's covered under the compatibility policy, I thought I'd put it on the ML anyway. :)
You can find the code at https://twistedmatrix.com/trac/ticket/8119. We do not require any signoffs to remove it, this email is purely informational.
- Amber
_______________________________________________
Twisted-Python mailing list
http://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python

[Glyph Lefkowitz]

Post by Amber "Hawkie" Brown
So, apparently, a long time ago, there was a database called Gadfly, and Twisted used to test adbapi against it. However, since then, we've moved on from Python versions that gadfly supports, so it outright doesn't work now -- not buggy, not weird -- just doesn't even import.
This doesn't change any user facing code, but does change some tests, and even though it's covered under the compatibility policy, I thought I'd put it on the ML anyway. :)
You can find the code at https://twistedmatrix.com/trac/ticket/8119. We do not require any signoffs to remove it, this email is purely informational.
- Amber

I think the all-caps policy-compliant "INCOMPATIBLE CHANGE" was maybe a little overkill, given it's not actually incompatible under the policy definition, but thanks for announcing nonetheless :).

The purpose of testing with gadfly was just to have a database that didn't require an external process set up; I don't think anyone seriously used Gadfly ever. So the modern equivalent would be sqlite3 - do we test against that? (I seem to recall that we do...)

-glyph

[Amber Brown]

We don't! Those tests never run, because "import sqlite" doesn't work on 2.7. Today's plan is to put up a patch to test stdlib sqlite3 and pysqlite2 :)

Post by Glyph Lefkowitz

Post by Amber "Hawkie" Brown
So, apparently, a long time ago, there was a database called Gadfly, and Twisted used to test adbapi against it. However, since then, we've moved on from Python versions that gadfly supports, so it outright doesn't work now -- not buggy, not weird -- just doesn't even import.
This doesn't change any user facing code, but does change some tests, and even though it's covered under the compatibility policy, I thought I'd put it on the ML anyway. :)
You can find the code at https://twistedmatrix.com/trac/ticket/8119. We do not require any signoffs to remove it, this email is purely informational.
- Amber

I think the all-caps policy-compliant "INCOMPATIBLE CHANGE" was maybe a little overkill, given it's not actually incompatible under the policy definition, but thanks for announcing nonetheless :).
The purpose of testing with gadfly was just to have a database that didn't require an external process set up; I don't think anyone seriously used Gadfly ever.  So the modern equivalent would be sqlite3 - do we test against that?  (I seem to recall that we do...)
-glyph
_______________________________________________
Twisted-Python mailing list
http://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python