[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#765179: RFS: yubikey-neo-manager/0.2.2-1 [ITP] -- YubiKey NEO management graphical user interface



I'm resolving the issue of pixmaps in /usr/lib, as well as replacing the menuicon with one that is 32x32 as we speak. It looks bad when I test it on Ubuntu (which uses 48x48), but it seems Ubuntu has an app-install-data package with higher res icons for all of the included packages, so we can worry about that later I suppose.

Dain Nilsson
Senior Software Developer, Yubico 
dain@yubico.com
yubico.com

On Tue, Oct 14, 2014 at 11:54 AM, Simon Josefsson <simon@josefsson.org> wrote:
> On Mon, 2014-10-13 at 21:35 +0200, Simon Josefsson wrote:
> > Dear mentors,
> >
> > I am looking for a sponsor for our package "yubikey-neo-manager":
>
> Hello Simon!
>
> Thank you for your help in packaging the yubikey neo manager for
> Debian! I've got a few things that might need fixing up for you to
> take a look at.

Thank you!  Cc'ing Dain who has wrote the project.

> - The file neoman/libloader has a different copyright than the rest of
> the code, both in license and copyright holder.  It needs to be
> specified in d/copyright

Good catch!  Dain, is it needed?  I added it to debian/copyright
meanwhile.

> - Do you want debian/* to be copyrighted by Yubico, or should it be
> copyrighted by you?  (This may or may not be a question for legal
> counsel.)

Having debian/* be owned by Yubico is fine, we are the same group of
people working on both upstream and the packaging, and all are bound to
Yubico.

> - There are several pixmap files being installed under /usr/lib;
> architecture independent files should be installed under /usr/share.
>
> - Icons in the Debian menu system should be 32x32 at the largest -
> the /usr/share/pixmap/neoman.xpm, file is 128x128.

Yep, we are trying to figure these two out -- do you happen to know how
to get high-res menu icons?  32x32 is really crappy, and there ought to
be a way to provide a better image.

> -  There is a substitution variable ${python:Provides}, but there
> isn't a substitution variable for that.  What are you trying to get
> from that?

Likely cut'n'paste issue, I dropped it.

I have built and uploaded a new version to mentors.

> Thanks, again, for packaging these tools.  I have a NEO on my
> keychain, so this definitely caught my eye!

Glad to hear that :-)  Btw, are you a DD?  If you would be willing to
sponsor this, that would be appreciated.

/Simon


Reply to: