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

Bug#672237: RFS: vim-vimerl/1.4.1+git20120509.89111c7-1 [ITP]



Hi!

On Fri, May 11, 2012 at 10:53 AM, Benoît Knecht <benoit.knecht@fsfe.org> wrote:
> Hi Per,
>
> Per Andersson wrote:
>> I am looking for a sponsor for my package "vim-vimerl"
>>
>> * Package name    : vim-vimerl
>>   Version         : 1.4+git20120210.89111c7-1
>>    Upstream Author : Ricardo Catalinas Jiménez <jimenezrick@gmail.com>
>> * URL             : http://github.com/jimenezrick/vimerl
>> * License         : Vim License
>>   Section         : editors
>>
>> It builds those binary packages:
>>
>>  vim-vimerl - Erlang plugin for Vim
>>  vim-vimerl-syntax - Erlang syntax for Vim
>>
>> To access further information about this package, please visit the
>> following URL:
>>
>>   http://mentors.debian.net/package/vim-vimerl
>>
>>
>> Alternatively, one can download the package with dget using this command:
>>
>>     dget -x http://mentors.debian.net/debian/pool/main/v/vim-vimerl/vim-vimerl_1.4+git20120210.4c4d1f0-2.dsc

The above URL is wrong, there exists later packages.


> I took a look at your package:

Thanks!


>  - lintian reports the following:
>
>      P: vim-vimerl source: unversioned-copyright-format-uri http://dep.debian.net/deps/dep5
>      W: vim-vimerl source: syntax-error-in-dep5-copyright line 27: Continuation line outside a paragraph.

This was (and is) fixed in the later packages.


>      P: vim-vimerl-syntax: no-upstream-changelog
>      P: vim-vimerl: no-upstream-changelog

Upstream has no changelog. Should I do a Lintian override? (Lintian man page
states that overriding pedantic tags are probably not worth the effort.)


>  - In debian/control, ${shlibs:Depends} is not defined, just remove it.
>
>  - I don't think the upstream README contains any useful information
>    for debian users (features are in the long description, installation
>    instructions are irrelevant, and bugs should normally be reported in
>    debian's BTS), don't install it.

I'll remove these then.


>  - You should remove the template comments in debian/rules.

Removed.


>  - According to uscan, there's a more recent upstream version (1.4.1).

Yes! Already packaged, as stated above.


I have prepared a new package with the fixes suggested above.

Have a look at the latest package at

    http://mentors.debian.net/package/vim-vimerl

Or download it with dget:

    dget -x http://mentors.debian.net/debian/pool/main/v/vim-vimerl/vim-vimerl_1.4.1+git20120509.89111c7-2.dsc


Changes since last upload:

 vim-vimerl (1.4.1+git20120509.89111c7-2) unstable; urgency=low

   * debian/control: Remove unused ${shlib:Depends}.
   * debian/rules: Remove template comments.
   * Do not install README.

  -- Per Andersson <avtobiff@gmail.com>  Sun, 13 May 2012 02:00:43 +0200


Best,
Per

> Cheers,
>
> --
> Benoît Knecht



Reply to: