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

Bug#409124: marked as done ([aesthetics] report trailing blank lines and whitespace)

Your message dated Mon, 22 Jul 2013 14:58:10 +0200
with message-id <51ED2C62.1020802@thykier.net>
and subject line Re: Bug#409124: [aesthetics] report trailing blank lines and whitespace
has caused the Debian Bug report #409124,
regarding [aesthetics] report trailing blank lines and whitespace
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
409124: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=409124
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: lintian
Version: 1.23.27
Severity: wishlist

It would be nice if there were and INFO level notice to report:

- Extra empty newlines at the end of file
- Trailining whitepace at the end of lines

Usually these are just careless mistakes that are easily overlooked and
they appear in patch. Paying attention to these could help making files
in debian/* more strict.

-- System Information:
Debian Release: 4.0
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'stable')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/dash
Kernel: Linux 2.6.18-3-686
Locale: LANG=C, LC_CTYPE=C (charmap=ISO-8859-1) (ignored: LC_ALL set to en_US)

Versions of packages lintian depends on:
ii  binutils               2.17-3            The GNU assembler, linker and bina
ii  diffstat               1.43-2            produces graph of changes introduc
ii  dpkg-dev               1.13.25           package building tools for Debian
ii  file                   4.17-5            Determines file type using "magic"
ii  gettext                0.16.1-1          GNU Internationalization utilities
ii  intltool-debian        0.35.0+20060710.1 Help i18n of RFC822 compliant conf
ii  libparse-debianchangel 1.0-1             parse Debian changelogs and output
ii  man-db                 2.4.3-5           The on-line manual pager
ii  perl [libdigest-md5-pe 5.8.8-7           Larry Wall's Practical Extraction 

lintian recommends no packages.

-- no debconf information

--- End Message ---
--- Begin Message --- 
On 2011-12-22 22:44, jari wrote:
> 2009-03-08 Russ Allbery <rra@debian.org>:
> | Anything that doesn't handle whitespace in that way is buggy and needs to
> | be fixed.  I don't think warning (even as a pedantic warning) about
> | packages that follow Policy's requirements for debian/control is a good
> | idea.  I have added a pedantic tag for fields that don't use a single
> | space after the colon; that's the one thing that's present in Policy as a
> | pseudo-recommendation.
> 
> I's not about Debian tools to be unable to handle these things. 
> 
> It is just logical to add these checks to "Lintian" to help keeping
> packages in best possible shape.
> 
> Who would deliberately add EOL whitespaces?
> 
> It's usually added by mistake because the editor may be unable (or not
> configured to) to show trailing white spaces.
> 
> When data is exchanged between parties:
> 
> 	- By email
> 	- By patches (NMUs, diffs, etc)
> 	- Injected into VCS (if configured to warn EOL whitespaces)
> 
> It helps when there are no EOL whitespaces to worry about.
> 
> Jari
> 

I think we have to say no to including this tag in Lintian itself.
However, you can still write a check for it (even if only for
local/personal use)[1].

~Niels

[1] http://lintian.debian.org/manual/section-3.3.html

--- End Message ---
Reply to: