Sign me up Login

Details about package findent

Name: findent (PTS)
Uploader: Willem Vermin <wvermin@gmail.com> (Debian QA page)
Description: findent - indents/converts Fortran sources

Package uploads

Upload #2

Information

Version: 4.3.6-1
Uploaded: 2025-07-08 09:27
Source package: findent_4.3.6-1.dsc
Distribution: unstable
Section: devel
Priority: optional
Homepage: https://sourceforge.net/projects/findent/

Changelog

 findent (4.3.6-1) unstable; urgency=low
 .
   * copyright: removed vim/findent.vim and vim/fortran.vim
     These are already covered by 'Files: *'
   * Changed Standards-Version: 4.7.2
   * Remove trailing space from debian/changelog
   * Remove debian/patches
   * Add debian/source/include-binaries: doc/design/findent.pdf

QA information

Comments

No comments

Upload #1

Information

Version: 4.3.6-1
Uploaded: 2025-07-07 10:12
Source package: findent_4.3.6-1.dsc
Distribution: unstable
Section: devel
Priority: optional
Homepage: https://sourceforge.net/projects/findent/

Changelog

 findent (4.3.6-1) unstable; urgency=low
 .
   * copyright: removed vim/findent.vim and vim/fortran.vim
     These are already covered by 'Files: *'
   * Changed Standards-Version: 4.7.2

QA information

Comments

  1. Willem,

Review of upload: 2025-07-07 10:12

For information about the tests run, see:

https://wiki.debian.org/PhilWyett/DebianMentoring

Note: If test line removed, it indicates a test pass.

Test 4 (sbuild): Information only

Lintian:

P: findent source: trailing-whitespace [debian/changelog:12]
N: 
N:   This file contains lines with trailing whitespace characters.
N:   
N:   Whilst often harmless and unsightly, such extra whitespaces can also cause
N:   tools to interpret the whitespace characters literally. The tool diff(1)
N:   does not like them, either. They are best avoided.
N:   
N:   Some of these problems can be hard to track down.
N:   
N:   Whitespace at the end of lines may be removed with the following:
N:   
N:    $ sed -i -e 's@[[:space:]]*$@@g' debian/control debian/changelog
N:   
N:   If you use Emacs, you can also use "M-x wh-cl" (whitespace-cleanup).
N:   
N:   However, if you wish to only remove trailing spaces and leave trailing
N:   tabs (eg. for Makefiles), you can use the following code snippet:
N:   
N:    $ sed -i -e 's@[ ]*$@@g' debian/rules
N:   
N:   To remove empty lines from the end of a file, you can use:
N:   
N:    $ sed -i -e :a -e '/^\n*$/{$d;N;};/\n$/ba' debian/rules
N: 
N:   Visibility: pedantic
N:   Show-Always: no
N:   Check: debian/trailing-whitespace
N:   Renamed from: file-contains-trailing-whitespace

Test 5 (ratt): Information only
Test 6 (debian/watch): Information only
Test 7 (licenserecon): Information only
Test 8 (first run): Information only

Summary
=======

Looks good. I would certainly not block the package for anything here. If you wish to make changes prior to sponsorship or filing an RFS that would be good.

Could you remove the unused 'debian/patches' folder. Looks nicer and avoids confusion.

Regards

Phil
    Phil Wyett at July 7, 2025, 11:55 a.m.