Merge ~rbalint/ubuntu-seeds/+git/platform:demote-devscripts-lintian into ~ubuntu-core-dev/ubuntu-seeds/+git/platform:groovy

Proposed by Balint Reczey
Status: Rejected
Rejected by: Balint Reczey
Proposed branch: ~rbalint/ubuntu-seeds/+git/platform:demote-devscripts-lintian
Merge into: ~ubuntu-core-dev/ubuntu-seeds/+git/platform:groovy
Diff against target: 15 lines (+0/-2)
1 file modified
supported-development-common (+0/-2)
Reviewer Review Type Date Requested Status
Steve Langasek Pending
Dimitri John Ledkov Pending
Ubuntu Core Development Team Pending
Review via email: mp+391464@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Balint Reczey (rbalint) wrote :

Devscripts recommends lintian and it doing so makes sense, so devscripts should be demoted together with lintian.

Revision history for this message
Steve Langasek (vorlon) wrote :

I'm uncomfortable with demoting devscripts, because it contains many tools which are critical to the Ubuntu development process and I think it should be covered by security support.

Revision history for this message
Balint Reczey (rbalint) wrote :

I agree that devscripts tools are critical, but we can support it in universe including providing security support. The delta we carry is very small, too, thus I expect little maintenance work to be done on the package going forward.
Ideally devscripts should be in main, but IMO MIRs for the ever growing lintian dependencies create too much work for very little gain and an extra cost of main gaining newer and newer Perl packages.

Revision history for this message
Balint Reczey (rbalint) wrote :

Lintian had one security update since Precise:

lintian (2.5.43ubuntu0.1) xenial-security; urgency=medium

  * SECURITY UPDATE: code execution via YAML parsing
    - checks/upstream-metadata.pm: disable YAML parser.
    - t/tests/upstream-metadata-invalid-yml/skip: skip test.
    - 0a2f38ecbc70d34a4b77c93a030555b310bd34ff
    - CVE-2017-8829

 -- Marc Deslauriers <email address hidden> Mon, 05 Jun 2017 14:33:13 -0400

Devscripts had a few:

devscripts (2.17.12ubuntu1.1) bionic-security; urgency=medium

  * SECURITY UPDATE: Code execution through unsafe YAML loading
    - CVE-2018-13043
---
devscripts (2.14.1ubuntu0.1) trusty-security; urgency=medium

  * SECURITY UPDATE: directory traversal issue in uupdate
    - scripts/uupdate.sh: remove symlinks before applying patches, and
      restore them afterwards.
    - http://anonscm.debian.org/cgit/collab-maint/devscripts.git/commit/?id=0fef671
    - CVE-2014-1833

---
devscripts (2.11.6ubuntu1.7) precise-security; urgency=medium

  * SECURITY UPDATE: directory traversal issue in uupdate
    - scripts/uupdate.sh: remove symlinks before applying patches, and
      restore them afterwards.
    - http://anonscm.debian.org/cgit/collab-maint/devscripts.git/commit/?id=0fef671
    - CVE-2014-1833

 -- Marc Deslauriers <email address hidden> Mon, 15 Jun 2015 13:15:39 -0400

devscripts (2.11.6ubuntu1.6) precise-security; urgency=low

  * SECURITY UPDATE: arbitrary code execution in uscan via crafted tarball
    - scripts/uscan.pl: improve tarball handling.
    - 02c6850d973e3e1246fde72edab27f03d63acc52
    - 4b7e58ee6000cdefac0682601cec6ecce0137467
    - CVE-2013-6888

 -- Marc Deslauriers <email address hidden> Fri, 10 Jan 2014 13:02:15 -0500

devscripts (2.11.6ubuntu1.4) precise-security; urgency=low

  * SECURITY UPDATE: arbitrary code execution via insufficient validation
    in dscverify
    - scripts/dscverify.pl: perform better validation.
    - 9fba4788933475185df5e58b7fa557e5e3fb15e4
    - CVE-2012-2240
  * SECURITY UPDATE: arbitrary file deletion via insufficient validation
    in dget
    - scripts/dget.pl: strip invalid characters
    - 0fd15bdec07b085f9ef438dacd18e159ac60b810
    - CVE-2012-2241
  * SECURITY UPDATE: file alteration via TOCTOU in annotate-output
    - scripts/annotate-output.sh: prevent symlink attack.
    - 4d23a5e6c90f7a37b0972b30f5d31dce97a93eb0
    - CVE-2012-3500
  * REGRESSION FIX: improper exit code in CVE-2012-0212 debdiff.pl fix
    - f9a1a4c468671827d2650161cc33324fe0247a98

Revision history for this message
Balint Reczey (rbalint) wrote :

For now lintian and devscripts stay in main because there would be no easy way of monitoring the security problems if they were demoted, according to the Security Team.

This could be revisited later.

Unmerged commits

5d19d46... by Balint Reczey

unseed devscripts and lintian

While both are needed for package maintenance they don't have to be
supported in main and lintian keeps expanding its dependency list
growing main with packages not needed by any other package in main.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/supported-development-common b/supported-development-common
2index 655c2cc..9b47a45 100644
3--- a/supported-development-common
4+++ b/supported-development-common
5@@ -38,10 +38,8 @@
6 == Package Maintainance ==
7
8 * debhelper # ColinWatson
9- * devscripts
10 * dh-make
11 * dupload # ColinWatson
12- * lintian
13 * apt-listchanges
14 * dput
15 * pkgstriptranslations # for removing gettext data on the buildds

Subscribers

People subscribed via source and target branches