Re: Wheezy update of lintian?

[Niels Thykier]

Ola Lundqvist:
> Dear maintainer(s),
> 
> The Debian LTS team would like to fix the security issues which are
> currently open in the Wheezy version of lintian:
> https://security-tracker.debian.org/tracker/CVE-2017-8829
> 
> []
> 

Hi,

The bug (#861958) says it only affects lintian from version 2.5.41 or
later.  Wheezy has 2.5.10.4 and thus unaffected.  Even Wheezy-backports
at 2.5.30 should be fine (only -sloppy is affected).

IOW, I think there is nothing for either of us to do (except update the
metadata about CVE-2017-8829). :)

Thanks,
~Niels

[lintian] branch master updated (01b97ac -> eda105a)

[Paul Wise]

This is an automated email from the git hooks/post-receive script.

pabs pushed a change to branch master
in repository lintian.

  from  01b97ac   c/files.pm: Perltidy again after perlcritic (but adding 
an extra space before '{')
   new  eda105a   spelling: Add several corrections

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.


Summary of changes:
 data/spelling/corrections | 4 
 1 file changed, 4 insertions(+)

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/lintian/lintian.git

[lintian] 01/01: spelling: Add several corrections

[Paul Wise]

This is an automated email from the git hooks/post-receive script.

pabs pushed a commit to branch master
in repository lintian.

commit eda105a4dd4738f6dac69e56e8eb496c25857b39
Author: Paul Wise 
Date:   Mon May 22 08:38:08 2017 +0800

spelling: Add several corrections
---
 data/spelling/corrections | 4 
 1 file changed, 4 insertions(+)

diff --git a/data/spelling/corrections b/data/spelling/corrections
index 831dc27..3d1835a 100644
--- a/data/spelling/corrections
+++ b/data/spelling/corrections
@@ -679,6 +679,8 @@ clasifying||classifying
 classess||classes
 classs||class
 classses||classes
+cleanpu||cleanup
+cleanpus||cleanups
 clearified||clarified
 clearifies||clarifies
 clearify||clarify
@@ -1774,6 +1776,7 @@ guranteed||guaranteed
 gurantee||guarantee
 guranteeing||guaranteeing
 gurantees||guarantees
+hadling||handling
 halfs||halves
 hander||handler
 handfull||handful
@@ -3638,6 +3641,7 @@ syncronizing||synchronizing
 syncronously||synchronously
 syncronous||synchronous
 syncronus||synchronous
+syncting||syncing
 synopsys||synopsis
 syste||system
 sytems||systems

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/lintian/lintian.git

Wheezy update of lintian?

[Ola Lundqvist]

Dear maintainer(s),

The Debian LTS team would like to fix the security issues which are
currently open in the Wheezy version of lintian:
https://security-tracker.debian.org/tracker/CVE-2017-8829

Would you like to take care of this yourself?

If yes, please follow the workflow we have defined here:
https://wiki.debian.org/LTS/Development

If that workflow is a burden to you, feel free to just prepare an
updated source package and send it to debian-...@lists.debian.org
(via a debdiff, or with an URL pointing to the source package,
or even with a pointer to your packaging repository), and the members
of the LTS team will take care of the rest. Indicate clearly whether you
have tested the updated package or not.

If you don't want to take care of this update, it's not a problem, we
will do our best with your package. Just let us know whether you would
like to review and/or test the updated package before it gets released.

You can also opt-out from receiving future similar emails in your
answer and then the LTS Team will take care of lintian updates
for the LTS releases.

Thank you very much.

Ola Lundqvist,
  on behalf of the Debian LTS team.

PS: A member of the LTS team might start working on this update at
any point in time. You can verify whether someone is registered
on this update in this file:
https://anonscm.debian.org/viewvc/secure-testing/data/dla-needed.txt?view=markup

Bug#863068: lintian: Merge pages from same contributor (but different email) into one page

[Niels Thykier]

Package: lintian
Version: 2.5.50.3
Severity: wishlist

Many contributors have multiple emails (common case being "non-DD"
email and a "DD" email).  However, lintian.d.o generates a maintainer
page per unique email and not per contributor.

We can do this by exporting the following data from
contributors.debian.org[1].  Using this dataset we can merge multiple
emails into one contributor by comparing the "user.email" value.  If
two entries have the same "user.email" value, then they are related to
the same contributor.

Access to the dataset is a privacy concern, so:
 * The data set (and related log files) should preferably at most be readable 
by the
   lintian maintainers on lindsay.d.o
- technically, lindsay is DD-only, but I would still feel better with 0750
  over a 0755 permission.
 * We should not present any data from the dataset except where this is already 
public and
   exposed by the current implementation.
   - Example: Currently, I don't use my nthyk...@debian.org email for 
packaging, so that
 email must not appear on lintian.d.o even though the dataset lists it as 
an email
 associated to me.
   - On the flip side, lintian.d.o would expose ni...@thykier.net as I use that 
for packaging
 (like it did previously).

In the short term, we can do manual exports of the data (for
prototyping/testing).  Long term, we should setup some sort of batch
job with a service account to pull this data.  The latter probably needs
DSA and/or maintainers of contributor.d.o

Thanks,
~Niels

[1] 
https://contributors.debian.org/api/identifiers/?format=json&type=email&limit=none

 * Warning: large data set - your browser/editor might not like it (omit 
"limit=no" if you are going
   to click on the link)
 * Access restrictions: DD-only (Privacy)
 * Authentification: SSL certificate (from sso.d.o)
 * Code: 
https://anonscm.debian.org/cgit/nm/dc.git/commit/?id=7c58b6ce8092fea3c6902dfe8a10428f7c4d1795
   - Plus some later commits
 * Example in below using "?user__email=nthykier%40debian.org&type=email" as 
filter
   - Data is about me, declassified by me, so its exposure is a non-concern.

JSON example:

{
"count": 2,
"next": null,
"previous": null,
"results": [
{
"type": "email",
"name": "ni...@thykier.net",
"user": {
"email": "nthyk...@debian.org",
"full_name": "Niels Thykier"
}
},
{
"type": "email",
"name": "nthyk...@debian.org",
"user": {
"email": "nthyk...@debian.org",
"full_name": "Niels Thykier"
}
}
]
}

Jenkins build is back to normal : lintian-tests_sid #1676

[jenkins]

See 

Build failed in Jenkins: lintian-tests_sid #1675

[jenkins]

See 


Changes:

[lamby] c/files.pm: Avoid "Mixed high and low-precedence booleans" perlcritic

--
[...truncated 260.78 KB...]
Adding debian:China_Internet_Network_Information_Center_EV_Certificates_Root.pem
Adding debian:Chambers_of_Commerce_Root_-_2008.pem
Adding debian:Certum_Trusted_Network_CA_2.pem
Adding debian:Certum_Trusted_Network_CA.pem
Adding debian:Certum_Root_CA.pem
Adding debian:Certplus_Root_CA_G2.pem
Adding debian:Certplus_Root_CA_G1.pem
Adding debian:Certplus_Class_2_Primary_CA.pem
Adding debian:Certinomis_-_Root_CA.pem
Warning: there was a problem reading the certificate file 
/etc/ssl/certs/Certinomis_-_Autorit?_Racine.pem. Message:
  /etc/ssl/certs/Certinomis_-_Autorit?_Racine.pem (No such file or directory)
Adding debian:Certigna.pem
Adding debian:Certification_Authority_of_WoSign_G2.pem
Adding debian:Camerfirma_Global_Chambersign_Root.pem
Adding debian:Camerfirma_Chambers_of_Commerce_Root.pem
Adding debian:COMODO_RSA_Certification_Authority.pem
Adding debian:COMODO_ECC_Certification_Authority.pem
Adding debian:COMODO_Certification_Authority.pem
Adding debian:CNNIC_ROOT.pem
Adding debian:CFCA_EV_ROOT.pem
Adding debian:CA_WoSign_ECC_Root.pem
Adding debian:CA_Disig_Root_R2.pem
Adding debian:CA_Disig_Root_R1.pem
Adding debian:Buypass_Class_3_Root_CA.pem
Adding debian:Buypass_Class_2_Root_CA.pem
Adding debian:Buypass_Class_2_CA_1.pem
Adding debian:Baltimore_CyberTrust_Root.pem
Adding debian:Autoridad_de_Certificacion_Firmaprofesional_CIF_A62634068.pem
Adding debian:Atos_TrustedRoot_2011.pem
Adding debian:ApplicationCA_-_Japanese_Government.pem
Adding debian:AffirmTrust_Premium_ECC.pem
Adding debian:AffirmTrust_Premium.pem
Adding debian:AffirmTrust_Networking.pem
Adding debian:AffirmTrust_Commercial.pem
Adding debian:AddTrust_Qualified_Certificates_Root.pem
Adding debian:AddTrust_Public_Services_Root.pem
Adding debian:AddTrust_Low-Value_Services_Root.pem
Adding debian:AddTrust_External_Root.pem
Adding debian:Actalis_Authentication_Root_CA.pem
Warning: there was a problem reading the certificate file 
/etc/ssl/certs/AC_Ra?z_Certic?mara_S.A..pem. Message:
  /etc/ssl/certs/AC_Ra?z_Certic?mara_S.A..pem (No such file or directory)
Adding debian:ACEDICOM_Root.pem
Adding debian:ACCVRAIZ1.pem
done.
Setting up openjdk-8-jdk-headless:amd64 (8u131-b11-2) ...
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/idlj to 
provide /usr/bin/idlj (idlj) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jdeps to 
provide /usr/bin/jdeps (jdeps) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/wsimport to 
provide /usr/bin/wsimport (wsimport) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/rmic to 
provide /usr/bin/rmic (rmic) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jinfo to 
provide /usr/bin/jinfo (jinfo) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jsadebugd to 
provide /usr/bin/jsadebugd (jsadebugd) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/native2ascii 
to provide /usr/bin/native2ascii (native2ascii) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jstat to 
provide /usr/bin/jstat (jstat) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javac to 
provide /usr/bin/javac (javac) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javah to 
provide /usr/bin/javah (javah) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jstack to 
provide /usr/bin/jstack (jstack) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jrunscript to 
provide /usr/bin/jrunscript (jrunscript) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javadoc to 
provide /usr/bin/javadoc (javadoc) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jhat to 
provide /usr/bin/jhat (jhat) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javap to 
provide /usr/bin/javap (javap) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jar to provide 
/usr/bin/jar (jar) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/xjc to provide 
/usr/bin/xjc (xjc) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/schemagen to 
provide /usr/bin/schemagen (schemagen) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jps to provide 
/usr/bin/jps (jps) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/extcheck to 
provide /usr/bin/extcheck (extcheck) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jmap to 
provide /usr/bin/jmap (jmap) in auto mode
update-alterna

[lintian] 01/01: c/files.pm: Perltidy again after perlcritic (but adding an extra space before '{')

[Chris Lamb]

This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit 01b97ac77a839d32d04de7b42053d7ed65784400
Author: Chris Lamb 
Date:   Sun May 21 10:30:57 2017 +0200

c/files.pm: Perltidy again after perlcritic (but adding an extra space 
before '{')
---
 checks/files.pm | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/checks/files.pm b/checks/files.pm
index 9d65d0a..0f51425 100644
--- a/checks/files.pm
+++ b/checks/files.pm
@@ -748,8 +748,7 @@ sub run {
 elsif ($fname =~ m,^usr/lib/R/site-library/\S,) {
 if (   $file->is_file
 && $file =~ m,\.(?:rda|Rda|rdata|Rdata)$,
-&& !$info->index_resolved_path('debian/README.source'))
-{
+&& !$info->index_resolved_path('debian/README.source')) {
 tag 'r-data-without-readme-source', $file;
 }
 }

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/lintian/lintian.git

[lintian] branch master updated (9f298e9 -> 01b97ac)

[Chris Lamb]

This is an automated email from the git hooks/post-receive script.

lamby pushed a change to branch master
in repository lintian.

  from  9f298e9   c/files.pm: Avoid "Mixed high and low-precedence 
booleans" perlcritic warning.
   new  01b97ac   c/files.pm: Perltidy again after perlcritic (but adding 
an extra space before '{')

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.


Summary of changes:
 checks/files.pm | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/lintian/lintian.git

Build failed in Jenkins: lintian-tests_sid #1674

[jenkins]

See 


Changes:

[lamby] Detect RData without README.source. (Closes: #815233)

--
[...truncated 260.81 KB...]
Adding debian:Chambers_of_Commerce_Root_-_2008.pem
Adding debian:Certum_Trusted_Network_CA_2.pem
Adding debian:Certum_Trusted_Network_CA.pem
Adding debian:Certum_Root_CA.pem
Adding debian:Certplus_Root_CA_G2.pem
Adding debian:Certplus_Root_CA_G1.pem
Adding debian:Certplus_Class_2_Primary_CA.pem
Adding debian:Certinomis_-_Root_CA.pem
Warning: there was a problem reading the certificate file 
/etc/ssl/certs/Certinomis_-_Autorit?_Racine.pem. Message:
  /etc/ssl/certs/Certinomis_-_Autorit?_Racine.pem (No such file or directory)
Adding debian:Certigna.pem
Adding debian:Certification_Authority_of_WoSign_G2.pem
Adding debian:Camerfirma_Global_Chambersign_Root.pem
Adding debian:Camerfirma_Chambers_of_Commerce_Root.pem
Adding debian:COMODO_RSA_Certification_Authority.pem
Adding debian:COMODO_ECC_Certification_Authority.pem
Adding debian:COMODO_Certification_Authority.pem
Adding debian:CNNIC_ROOT.pem
Adding debian:CFCA_EV_ROOT.pem
Adding debian:CA_WoSign_ECC_Root.pem
Adding debian:CA_Disig_Root_R2.pem
Adding debian:CA_Disig_Root_R1.pem
Adding debian:Buypass_Class_3_Root_CA.pem
Adding debian:Buypass_Class_2_Root_CA.pem
Adding debian:Buypass_Class_2_CA_1.pem
Adding debian:Baltimore_CyberTrust_Root.pem
Adding debian:Autoridad_de_Certificacion_Firmaprofesional_CIF_A62634068.pem
Adding debian:Atos_TrustedRoot_2011.pem
Adding debian:ApplicationCA_-_Japanese_Government.pem
Adding debian:AffirmTrust_Premium_ECC.pem
Adding debian:AffirmTrust_Premium.pem
Adding debian:AffirmTrust_Networking.pem
Adding debian:AffirmTrust_Commercial.pem
Adding debian:AddTrust_Qualified_Certificates_Root.pem
Adding debian:AddTrust_Public_Services_Root.pem
Adding debian:AddTrust_Low-Value_Services_Root.pem
Adding debian:AddTrust_External_Root.pem
Adding debian:Actalis_Authentication_Root_CA.pem
Warning: there was a problem reading the certificate file 
/etc/ssl/certs/AC_Ra?z_Certic?mara_S.A..pem. Message:
  /etc/ssl/certs/AC_Ra?z_Certic?mara_S.A..pem (No such file or directory)
Adding debian:ACEDICOM_Root.pem
Adding debian:ACCVRAIZ1.pem
done.
Setting up openjdk-8-jdk-headless:amd64 (8u131-b11-2) ...
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/idlj to 
provide /usr/bin/idlj (idlj) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jdeps to 
provide /usr/bin/jdeps (jdeps) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/wsimport to 
provide /usr/bin/wsimport (wsimport) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/rmic to 
provide /usr/bin/rmic (rmic) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jinfo to 
provide /usr/bin/jinfo (jinfo) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jsadebugd to 
provide /usr/bin/jsadebugd (jsadebugd) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/native2ascii 
to provide /usr/bin/native2ascii (native2ascii) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jstat to 
provide /usr/bin/jstat (jstat) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javac to 
provide /usr/bin/javac (javac) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javah to 
provide /usr/bin/javah (javah) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jstack to 
provide /usr/bin/jstack (jstack) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jrunscript to 
provide /usr/bin/jrunscript (jrunscript) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javadoc to 
provide /usr/bin/javadoc (javadoc) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jhat to 
provide /usr/bin/jhat (jhat) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/javap to 
provide /usr/bin/javap (javap) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jar to provide 
/usr/bin/jar (jar) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/xjc to provide 
/usr/bin/xjc (xjc) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/schemagen to 
provide /usr/bin/schemagen (schemagen) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jps to provide 
/usr/bin/jps (jps) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/extcheck to 
provide /usr/bin/extcheck (extcheck) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jmap to 
provide /usr/bin/jmap (jmap) in auto mode
update-alternatives: using /usr/lib/jvm/java-8-openjdk-amd64/bin/jstatd to 
provide /usr/bin/jstatd (jstatd) in

[lintian] 01/01: c/files.pm: Avoid "Mixed high and low-precedence booleans" perlcritic warning.

[Chris Lamb]

This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit 9f298e96673a11965b16645cb34720c23b9ead4b
Author: Chris Lamb 
Date:   Sun May 21 10:15:48 2017 +0200

c/files.pm: Avoid "Mixed high and low-precedence booleans" perlcritic 
warning.
---
 checks/files.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/checks/files.pm b/checks/files.pm
index 74c86df..9d65d0a 100644
--- a/checks/files.pm
+++ b/checks/files.pm
@@ -748,7 +748,7 @@ sub run {
 elsif ($fname =~ m,^usr/lib/R/site-library/\S,) {
 if (   $file->is_file
 && $file =~ m,\.(?:rda|Rda|rdata|Rdata)$,
-&& not $info->index_resolved_path('debian/README.source'))
+&& !$info->index_resolved_path('debian/README.source'))
 {
 tag 'r-data-without-readme-source', $file;
 }

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/lintian/lintian.git

[lintian] branch master updated (c00ed2e -> 9f298e9)

[Chris Lamb]

This is an automated email from the git hooks/post-receive script.

lamby pushed a change to branch master
in repository lintian.

  from  c00ed2e   Detect RData without README.source. (Closes: #815233)
   new  9f298e9   c/files.pm: Avoid "Mixed high and low-precedence 
booleans" perlcritic warning.

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.


Summary of changes:
 checks/files.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/lintian/lintian.git

Processed: Re: detect RData without README.source

[Debian Bug Tracking System]

Processing commands for cont...@bugs.debian.org:

> tags 815233 + pending
Bug #815233 [lintian] detect RData without README.source
Added tag(s) pending.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
815233: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=815233
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems

Bug#815233: detect RData without README.source

[Chris Lamb]

Chris Lamb wrote:

> tags 815233 + pending
> thanks
> 
> Fixed in Git:

Sorry, wrong URL. Corrected:


https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=c00ed2e78e09e617b00933b4b548a37fd6c675ac



Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-

Bug#815233: detect RData without README.source

[Chris Lamb]

tags 815233 + pending
thanks

Fixed in Git:

  
https://anonscm.debian.org/git/lintian/lintian.git/commit/?id=8654944937da135487df1d98618b46599f396142


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-

[lintian] 01/01: Detect RData without README.source. (Closes: #815233)

[Chris Lamb]

This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit c00ed2e78e09e617b00933b4b548a37fd6c675ac
Author: Chris Lamb 
Date:   Sun May 21 09:49:08 2017 +0200

Detect RData without README.source. (Closes: #815233)
---
 checks/files.desc  | 11 +++
 checks/files.pm| 10 ++
 debian/changelog   |  1 +
 t/tests/files-r-data/debian/debian/install |  1 +
 t/tests/files-r-data/debian/src/test.Rda   |  0
 t/tests/files-r-data/debian/src/test.Rdata |  0
 t/tests/files-r-data/debian/src/test.rda   |  0
 t/tests/files-r-data/debian/src/test.rdata |  0
 t/tests/files-r-data/desc  |  6 ++
 t/tests/files-r-data/tags  |  4 
 10 files changed, 33 insertions(+)

diff --git a/checks/files.desc b/checks/files.desc
index 7113ac3..ff2f227 100644
--- a/checks/files.desc
+++ b/checks/files.desc
@@ -1741,3 +1741,14 @@ Info: The package is architecture-dependent, ships a 
shared library in
  same package, but this package does not contain any programs.
  .
  Please remove the Multi-Arch: foreign stanza.
+
+Tag: r-data-without-readme-source
+Severity: important
+Certainty: certain
+Info: Many modules packaged for the R Project for Statistical Computing contain
+ data files with names as *.rda, *.Rda, *.rdata, *.Rdata, etc.
+ .
+ When such files exist, the FTP masters expect them to be explained in
+ debian/README.source, which this package is missing.
+ .
+ Please add a README.source documenting the origins of these files.
diff --git a/checks/files.pm b/checks/files.pm
index 5a696c5..74c86df 100644
--- a/checks/files.pm
+++ b/checks/files.pm
@@ -744,6 +744,16 @@ sub run {
 }
 }
 
+#  /usr/lib/R/site-library
+elsif ($fname =~ m,^usr/lib/R/site-library/\S,) {
+if (   $file->is_file
+&& $file =~ m,\.(?:rda|Rda|rdata|Rdata)$,
+&& not $info->index_resolved_path('debian/README.source'))
+{
+tag 'r-data-without-readme-source', $file;
+}
+}
+
 #  /usr/lib/sgml
 elsif ($fname =~ m,^usr/lib/sgml/\S,) {
 tag 'file-in-usr-lib-sgml', $file;
diff --git a/debian/changelog b/debian/changelog
index 0f520bb..eaf7331 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -47,6 +47,7 @@ lintian (2.5.51) UNRELEASED; urgency=medium
   files that should be ignored.  Thanks to Helmut Grohne for all the
   research behind it.  (Closes: #856857)
 + [BR] Detect symlink pointing to builddir.  (Closes: #860419)
++ [CL] Detect RData without README.source.  (Closes: #815233)
   * checks/gir.{desc,pm}:
 + [NT] Add check for gir packages written by Simon McVittie.
   (Closes: #695345)
diff --git a/t/tests/files-r-data/debian/debian/install 
b/t/tests/files-r-data/debian/debian/install
new file mode 100644
index 000..7cd27d5
--- /dev/null
+++ b/t/tests/files-r-data/debian/debian/install
@@ -0,0 +1 @@
+src/* usr/lib/R/site-library/files-r-data
diff --git a/t/tests/files-r-data/debian/src/test.Rda 
b/t/tests/files-r-data/debian/src/test.Rda
new file mode 100644
index 000..e69de29
diff --git a/t/tests/files-r-data/debian/src/test.Rdata 
b/t/tests/files-r-data/debian/src/test.Rdata
new file mode 100644
index 000..e69de29
diff --git a/t/tests/files-r-data/debian/src/test.rda 
b/t/tests/files-r-data/debian/src/test.rda
new file mode 100644
index 000..e69de29
diff --git a/t/tests/files-r-data/debian/src/test.rdata 
b/t/tests/files-r-data/debian/src/test.rdata
new file mode 100644
index 000..e69de29
diff --git a/t/tests/files-r-data/desc b/t/tests/files-r-data/desc
new file mode 100644
index 000..b897784
--- /dev/null
+++ b/t/tests/files-r-data/desc
@@ -0,0 +1,6 @@
+Testname: files-r-data
+Sequence: 6000
+Version: 1.0
+Description: Checks for R statistical data files
+Test-For:
+ r-data-without-readme-source
diff --git a/t/tests/files-r-data/tags b/t/tests/files-r-data/tags
new file mode 100644
index 000..1a03bbd
--- /dev/null
+++ b/t/tests/files-r-data/tags
@@ -0,0 +1,4 @@
+E: files-r-data: r-data-without-readme-source 
usr/lib/R/site-library/files-r-data/test.Rda
+E: files-r-data: r-data-without-readme-source 
usr/lib/R/site-library/files-r-data/test.Rdata
+E: files-r-data: r-data-without-readme-source 
usr/lib/R/site-library/files-r-data/test.rda
+E: files-r-data: r-data-without-readme-source 
usr/lib/R/site-library/files-r-data/test.rdata

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/lintian/lintian.git

[lintian] branch master updated (020e3d7 -> c00ed2e)

[Chris Lamb]

This is an automated email from the git hooks/post-receive script.

lamby pushed a change to branch master
in repository lintian.

  from  020e3d7   c/cruft: Apply perltidy
   new  c00ed2e   Detect RData without README.source. (Closes: #815233)

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.


Summary of changes:
 checks/files.desc | 11 +++
 checks/files.pm   | 10 ++
 debian/changelog  |  1 +
 t/tests/files-r-data/debian/debian/install|  1 +
 .../files-r-data/debian/src/test.Rda} |  0
 .../files-r-data/debian/src/test.Rdata}   |  0
 .../files-r-data/debian/src/test.rda} |  0
 .../files-r-data/debian/src/test.rdata}   |  0
 t/tests/files-r-data/desc |  6 ++
 t/tests/files-r-data/tags |  4 
 10 files changed, 33 insertions(+)
 create mode 100644 t/tests/files-r-data/debian/debian/install
 copy t/{changes/changes-directory-traversal.tags => 
tests/files-r-data/debian/src/test.Rda} (100%)
 copy t/{changes/changes-directory-traversal.tags => 
tests/files-r-data/debian/src/test.Rdata} (100%)
 copy t/{changes/changes-directory-traversal.tags => 
tests/files-r-data/debian/src/test.rda} (100%)
 copy t/{changes/changes-directory-traversal.tags => 
tests/files-r-data/debian/src/test.rdata} (100%)
 create mode 100644 t/tests/files-r-data/desc
 create mode 100644 t/tests/files-r-data/tags

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/lintian/lintian.git