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

Bug#916905: RFS: Jerry (Chess GUI) /3.1-0 -- looking for sponsor



On Fri, Dec 21, 2018 at 08:45:40PM +0000, Dominik Klein wrote:
> thanks very much for your feedback.
> 
> I've filed an ITP under #917027.
> 
> Moreover I've
> - changed the icon to a png, since the problem is likely that not all 
> window manager are able to display .ico
> - changed changelog to mark for unstable
> - added regexp to track new releases via the watch-file
> - modified the copyright file to clarify for all files who owns the 
> copyright and which license
> - The .bin is polyglot opening book. It's format is openly documented, 
> e.g. here http://hardy.uhasselt.be/Toga/book_format.html
> 
> I also changed the changelog 
> https://github.com/asdfjkl/jerry/releases/tag/v3.1.0 to automatically 
> close #917027, i.e. if feedback for the ITP is positive and a sponsor 
> would come forward, the current package might be suitable for inclusion...

Awesome.

There's one laughably small nit: the control file says:
Maintainer: Dominik Klein <dominik.klein@outlook.com>
while the changelog:
 -- Dominik Klein <Dominik.Klein@outlook.com>  Fri, 21 Dec 2018 10:15:05 +0100
with different capitalization of the email address.

For many mail servers, the user field is case-sentisive, thus tools in
general have to understand it this way.

Yet while it's just a detail, it would spook at least some of the archive
tools into thinking you're not the maintainer.  It'd be best to get this
right before the first upload.


Other stuff that would be nice to improve but are not blockers:

Tthe icon still doesn't work for me (xfce).

The copyright file would be a lot more readable if you placed you and Karl
into a "Files: *" stanza.  If two stanzas match the same file, only the
later one applies, allowing overriding a more global match.

Also, while missing a copyright holder is an error (not for a reasonable
person, but lawyers are not reasonable, thus the ftpmasters need to be
strict), being too generous is benign.  Thus, having both you and Karl for
the majority of files would allow coalescing all those entries into a single
one, leaving nothing but icons, pieces and books.

Likewise, the .install file could be made more readable using wildcards.
Of course, this is not mandatory, but would be nice to have.


Meow!
-- 
⢀⣴⠾⠻⢶⣦⠀ 
⣾⠁⢠⠒⠀⣿⡁ Ivan was a worldly man: born in St. Petersburg, raised in
⢿⡄⠘⠷⠚⠋⠀ Petrograd, lived most of his life in Leningrad, then returned
⠈⠳⣄⠀⠀⠀⠀ to the city of his birth to die.


Reply to: