Feedback on the grub2-themes-ubuntustudio package in NEW:
 - please review the output of 'lintian -I *changes' on packages.  In 
particular, in this case:
W: grub2-themes-ubuntustudio source: newer-standards-version 4.3.0 (current is 
4.2.1)
   (there should be no reason for an out-of-date standards version on a 
completely new package)
 - why is src/grub2-themes-ubuntu-mate.svg present in the source? this is 
obviously not even the right package name.
 - debian/copyright includes a lot of redundancy.  Files: *, Files: debian/*, 
and Files: theme.txt all have the same copyright and license; there is no 
reason for separate stanzas vs. a single glob. (I realize this is inherited 
from the grub2-themes-ubuntu-mate source package, but that is not the package 
I'm currently reviewing in the NEW queue, this one is.)
 - I don't know what the debian/post{inst,rm} scripts here are supposed to do - 
they are not inherited from the ubuntu-mate theme package, and there is nothing 
in the packaging which contains integration that would change the grub theme 
selection.  I also see that you are calling update-grub2; but this is not the 
preferred name for the command, you should be invoking it as 'update-grub'.  
Furthermore, since the package depends on grub-common, I don't think there's 
any reason for guarding the update-grub call with a 'which' check for the 
command.

Since this last point is a potential integration issue, and we are past
FF, I am going to reject the package currently in the NEW queue.  Once
this is resolved, I can either sponsor a new upload or rescue the
previous package from the rejected queue as appropriate.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1818373

Title:
  [needs-packaging] [FFe] grub2-themes-ubuntustudio

To manage notifications about this bug go to:
https://bugs.launchpad.net/grub2-theme-ubuntustudio/+bug/1818373/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to