Code review comment for lp:~bones/ubuntu/oneiric/gnomebaker/fix-for-818364

Revision history for this message
Michael Terry (mterry) wrote :

Hi! So I would normally approve this, except for two things:

1) Most importantly, this package has been removed from Debian and Ubuntu because it is "obsolete, dead upstream, unmaintained, doesn't work with current kernels."

2) Even if it hadn't been removed, this was a package that we got from Debian. Since Ubuntu loves to be in sync with Debian where possible, this change isn't important enough to carry a delta from Debian for it. So I would have forwarded this patch on to Debian for you, but not committed it to Ubuntu. https://wiki.ubuntu.com/Debian/ForUbuntuDevelopers

Regardless, thanks for the effort and I'm glad you're learning Ubuntu development!

review: Disapprove

« Back to merge proposal