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

Bug#872996: RFS: node-unicode-tr51/9.0.0-1 ITP



control: tags -1 + moreinfo

On Wed, Aug 23, 2017 at 9:55 PM, roucaries bastien
<roucaries.bastien+debian@gmail.com> wrote:
> control: owner -1 rouca@debian.org
>
> I take it
>
> On Wed, Aug 23, 2017 at 2:43 PM, Julien Puydt <julien.puydt@laposte.net> wrote:
>> Package: sponsorship-requests
>> Severity: wishlist
>>
>>   Dear mentors,
>>
>>   I am looking for a sponsor for my package "node-unicode-tr51"
>>
>>  * Package name    : node-unicode-tr51
>>    Version         : 9.0.0-1
>>    Upstream Author : Mathias Bynens
>>  * URL             : https://mths.be/unicode-tr51
>>  * License         : Expat
>>    Section         : web
>>
>>   It builds those binary packages:
>>
>>     node-unicode-tr51 - Emoji data for Node.js
>>
>>   To access further information about this package, please visit the
>> following URL:
>>
>>   https://mentors.debian.net/package/node-unicode-tr51
>>
>>
>>   Alternatively, one can download the package with dget using this command:
>>
>>     dget -x
>> https://mentors.debian.net/debian/pool/main/n/node-unicode-tr51/node-unicode-tr51_9.0.0-1.dsc
>>
>>   It is packaged within the Debian Javascript Maintainers team:
>>     Vcs-Git:
>> https://anonscm.debian.org/git/pkg-javascript/node-unicode-tr51.git
>>     Vcs-Browser:
>> https://anonscm.debian.org/cgit/pkg-javascript/node-unicode-tr51.git
>>
>> Snark on #debian-js
>>
* debian/control:
   + section is javascript
   + you should use package.json version in source depends
   + Description is too short. Define emoji for instance. What contain
emoji data ?
 * copyright:
     please mention in general comment what I have just opened #873028
for adding emoji to unicode-data ....
 * rules
      They are no guarantee that .js file are overwritten by script
and even in this case your package does not build cleanly. Better is
to create a debian/build directory (rm -rf during clean) that will get
a symlink of script and data. Than run script on it.
 * debian/test/mocha: please you a copy on my runtestsuite script. It
will use installed version instead of source
* debian/watch is 4

Bastien


Reply to: