[Bug 1711197] Re: distro-info returns an object instead of a dictionary which breaks applications relying on it
Closing as those _ attributes were private APIs. ** Changed in: distro-info (Ubuntu) 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/1711197 Title: distro-info returns an object instead of a dictionary which breaks applications relying on it To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/distro-info/+bug/1711197/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1711197] Re: distro-info returns an object instead of a dictionary which breaks applications relying on it
_date and _rows were never part of the API since these properties are private. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1711197 Title: distro-info returns an object instead of a dictionary which breaks applications relying on it To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/distro-info/+bug/1711197/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1711197] Re: distro-info returns an object instead of a dictionary which breaks applications relying on it
FWIW, while this change may things nicer moving forward, this makes it very difficult to maintain code that needs to work on both, Xenial and Artful+. Having the old API still available would allow applications to continue to rely on the module without having to do any hackery, specially, since the module doesn't seem to expose any versioning information. While I acknowledge that the change is for the better, to address these cases it should at least expose version information so applications can adapt the work on the different behavior of the module in a different release. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1711197 Title: distro-info returns an object instead of a dictionary which breaks applications relying on it To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/distro-info/+bug/1711197/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1711197] Re: distro-info returns an object instead of a dictionary which breaks applications relying on it
Benjamin, we see that you made this API change as part of addressing bug #1253208. Do you see a fundamental reason not to continue supporting the old API? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1711197 Title: distro-info returns an object instead of a dictionary which breaks applications relying on it To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/distro-info/+bug/1711197/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1711197] Re: distro-info returns an object instead of a dictionary which breaks applications relying on it
Other areas this doesn't work anymore: ubuntu = UbuntuDistroInfo() ubuntu._rows ... AttributeError: 'UbuntuDistroInfo' object has no attribute '_rows' -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1711197 Title: distro-info returns an object instead of a dictionary which breaks applications relying on it To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/distro-info/+bug/1711197/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
