Re: checks/scripts / checkbashsims updates
Adam D. Barratt [EMAIL PROTECTED] writes: Sorry to be a pain, but we've tightened this further in svn to '(?![\$\(])\(\(.*\)\)' so as not to match constructs such as progress_size=$(((100 - $PROGRESS_STATE) / 3)) What *is* this construct trying to catch, anyway? dash supports things like $((4 + 5)), and if the above is also okay, I'm not sure when this would not be a false positive. -- Russ Allbery ([EMAIL PROTECTED]) http://www.eyrie.org/~eagle/ -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Re: checks/scripts / checkbashsims updates
On Tue, 2008-02-19 at 09:07 -0800, Russ Allbery wrote: Adam D. Barratt [EMAIL PROTECTED] writes: Sorry to be a pain, but we've tightened this further in svn to '(?![\$\(])\(\(.*\)\)' so as not to match constructs such as progress_size=$(((100 - $PROGRESS_STATE) / 3)) What *is* this construct trying to catch, anyway? dash supports things like $((4 + 5)), and if the above is also okay, I'm not sure when this would not be a false positive. So far as I can tell, it's intended to be catching use of (( as a synonym for let, instead of $((, for performing arithmetic expansion. (The accompanying comment simply says (( should be $((.) The check was added approximately 18 months ago but unfortunately there's no clue in the check in comment, which refers to other changes made in the same commit. Julian? Adam -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
lintian: r1221 - trunk/checks
Author: rra
Date: 2008-02-19 20:19:41 +0100 (Tue, 19 Feb 2008)
New Revision: 1221
Modified:
trunk/checks/scripts
Log:
One more regex update from checkbashisms.
Modified: trunk/checks/scripts
===
--- trunk/checks/scripts2008-02-19 10:45:45 UTC (rev 1220)
+++ trunk/checks/scripts2008-02-19 19:19:41 UTC (rev 1221)
@@ -561,7 +561,7 @@
'(?:^|\s+)echo\s+-e',# echo -e
'(?:^|\s+)exec\s+-[acl]',# exec -c/-l/-a name
'(?:^|\s+)let\s',# let ...
- '(?!\$)\(\(.*\)\)', # '((' should be '$(('
+ '(?![\$\(])\(\(.*\)\)', # '((' should be '$(('
'(\[|test)\s+-a',# test with unary -a (should be
-e)
);
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
lintian 1.23.45 MIGRATED to testing
FYI: The status of the lintian source package in Debian's testing distribution has changed. Previous version: 1.23.42 Current version: 1.23.45 -- This email is automatically generated; [EMAIL PROTECTED] is responsible. See http://people.debian.org/~henning/trille/ for more information. -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Bug#466617: lintian: Wrong spelling-error-in-copyright for publically
Package: lintian Version: 1.23.45 Severity: normal Lintian warns about a mis-spelling in one of my packages: W: libsvnqt4: spelling-error-in-copyright publically publicly Imho, both spellings are correct [1] and the check is wrong. Cheers, Michael [1] http://www.merriam-webster.com/cgi-bin/dictionary?book=Dictionaryva=publically -- System Information: Debian Release: lenny/sid APT prefers unstable APT policy: (500, 'unstable'), (300, 'experimental') Architecture: i386 (i686) Kernel: Linux 2.6.24.2 Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Versions of packages lintian depends on: ii binutils2.18.1~cvs20080103-1 The GNU assembler, linker and bina ii diffstat1.45-2 produces graph of changes introduc ii dpkg-dev1.14.16.6package building tools for Debian ii file4.23-2 Determines file type using magic ii gettext 0.17-2 GNU Internationalization utilities ii intltool-debian 0.35.0+20060710.1Help i18n of RFC822 compliant conf ii libparse-debianchan 1.1.1-2 parse Debian changelogs and output ii liburi-perl 1.35.dfsg.1-1Manipulates and accesses URI strin ii man-db 2.5.1-2 on-line manual pager ii perl [libdigest-md5 5.8.8-12 Larry Wall's Practical Extraction lintian recommends no packages. -- no debconf information -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Bug#466617: Acknowledgement (lintian: Wrong spelling-error-in-copyright for publically)
I have to add, that I'm not a native speaker. So in case I'm wrong, please let me know. The lintian warning was not verbose enough to explain why it thinks publically is not correct. Cheers, Michael -- Why is it that all of the instruments seeking intelligent life in the universe are pointed away from Earth? signature.asc Description: OpenPGP digital signature
lintian: r1222 - in trunk: debian lib
Author: rra Date: 2008-02-20 01:32:56 +0100 (Wed, 20 Feb 2008) New Revision: 1222 Modified: trunk/debian/changelog trunk/lib/Spelling.pm Log: + [RA] Merriam-Webster and the OED have both given up on publically vs. publicly, so we should as well. (Closes: #466617) Modified: trunk/debian/changelog === --- trunk/debian/changelog 2008-02-19 19:19:41 UTC (rev 1221) +++ trunk/debian/changelog 2008-02-20 00:32:56 UTC (rev 1222) @@ -48,6 +48,8 @@ * lib/Spelling.pm: + [RA] Add another spelling correction. ++ [RA] Merriam-Webster and the OED have both given up on publically + vs. publicly, so we should as well. (Closes: #466617) * reporting/html_reports: + [RA] Load the correct configuration file. Modified: trunk/lib/Spelling.pm === --- trunk/lib/Spelling.pm 2008-02-19 19:19:41 UTC (rev 1221) +++ trunk/lib/Spelling.pm 2008-02-20 00:32:56 UTC (rev 1222) @@ -250,7 +250,6 @@ pronouce pronounce protcol protocol protocoll protocol - publically publicly recieve receive recieved received redircet redirect -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Processed: Lintian bugs fixed in revision r1222
Processing commands for [EMAIL PROTECTED]: package lintian Ignoring bugs not assigned to: lintian # Fixed in r1222 by rra tag 466617 + pending Bug#466617: lintian: Wrong spelling-error-in-copyright for publically There were no tags set. Tags added: pending thanks Stopping processing here. Please contact me if you need assistance. Debian bug tracking system administrator (administrator, Debian Bugs database) -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
