[Pkg-javascript-devel] Uploading Rainloop

[Yadd]

On 05/12/2021 23:07, Yadd wrote:
> On 05/12/2021 20:38, Daniel Ring wrote:
>> Hello,
>>
>> Is anyone available to sponsor-upload Rainloop? The current version 
>> has been sitting for quite a while due to the release freeze, so I'd 
>> like to get the bugfixes in now that bullseye is done.
>>
>> I'd also like to become an official DM for the package, if anyone here 
>> is willing to advocate for me in the New Maintainers process.
> 
> Hi,
> 
> Please drop embedded vendors directory from upload (in Files-Excluded), 
> at least:
>   * bootstrap, ckeditor, jquery-ui, lightgallery (already packaged) and
>     maybe some others, please verify
>   * minified files of others, even if not used in install
> 
> It's better than to have many lines in lintian-overrides and explain 
> that files aren't used (looking at your patch, you already patch build 
> to use Debian files).
> 
> Maybe this can help:
> 
>    Files-Excluded: vendors
>    Files-Includes:
>      .*\.css$
>      .*\.less$

Be more strict here^

>      vendors/jquery-ui/js/jquery-ui-1.10.3.custom.js
>      vendors/jquery-nanoscroller/jquery.nanoscroller.js
>      vendors/jquery-wakeup/jquery.wakeup.js
>      vendors/jquery-letterfx/jquery-letterfx.js
>      vendors/inputosaurus/inputosaurus.js
>      vendors/jua/jua.js
>      vendors/keymaster/keymaster.js
>      vendors/qr.js/qr.js
>      ...
> 
> You miss also to patch one "bootstrap" call in tasks/config.js.
> 
> Cheers,
> Yadd
>