only consider lsbdistcodename for apt-transport-https
authorAntoine Beaupré <anarcat@debian.org>
Wed, 12 Feb 2020 19:03:15 +0000 (14:03 -0500)
committerAntoine Beaupré <anarcat@debian.org>
Wed, 12 Feb 2020 19:03:15 +0000 (14:03 -0500)
commitf862889455c4c9994d0afc21a84715f4bc051a9c
tree8b85bc5f811db901be52d2ced9200dd678defce9
parentd5884ab0608ee1d47f7ad922aa163f4721bb9038
only consider lsbdistcodename for apt-transport-https

It does not matter if we have *one* older source from (say) jessie or
stretch, we could still be running buster or later. The latter is more
reliably indicated by the fact than by the provided "release"
parameter, which is really just an arbitrary string that does not
necessarily match a Debian suite, especially for third-party
repositories.

I have had a problem with this setting when deploying a "stretch"
repository on a "buster" system because I needed to keep MongoDB
running (which is gone from buster). The "stretch" line triggered this
line, which conflicted with a `ensure_packages` I had elsewhere in our
code base, to *remove* the `apt-transport-https` package from buster
and later.

An alternative implementation might prefer to remove the package
unconditionnally if we run a newer release, but I figured I would keep
the changes to a minimum.
manifests/source.pp