Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.debian.devel > #101500 > unrolled thread
| Started by | Phil Morrell <debian@emorrp1.name> |
|---|---|
| First post | 2021-09-03 00:40 +0200 |
| Last post | 2021-09-15 17:40 +0200 |
| Articles | 13 — 9 participants |
Back to article view | Back to linux.debian.devel
Finding rough consensus on level of vendoring for large upstreams Phil Morrell <debian@emorrp1.name> - 2021-09-03 00:40 +0200
Re: Finding rough consensus on level of vendoring for large upstreams Jérémy Lal <kapouer@melix.org> - 2021-09-03 01:20 +0200
Re: Finding rough consensus on level of vendoring for large upstreams Phil Morrell <debian@emorrp1.name> - 2021-09-03 03:10 +0200
Re: Finding rough consensus on level of vendoring for large upstreams Gunnar Wolf <gwolf@debian.org> - 2021-09-03 17:30 +0200
Re: Finding rough consensus on level of vendoring for large upstreams Jonas Smedegaard <jonas@jones.dk> - 2021-09-03 02:50 +0200
Re: Finding rough consensus on level of vendoring for large upstreams Phil Morrell <debian@emorrp1.name> - 2021-09-03 03:40 +0200
Re: Finding rough consensus on level of vendoring for large upstreams Jonas Smedegaard <jonas@jones.dk> - 2021-09-03 05:00 +0200
Re: Finding rough consensus on level of vendoring for large upstreams Pirate Praveen <praveen@onenetbeyond.org> - 2021-09-03 18:50 +0200
Bug#907051: Finding rough consensus on level of vendoring for large upstreams Simon McVittie <smcv@debian.org> - 2021-09-03 11:30 +0200
Re: Finding rough consensus on level of vendoring for large upstreams Paul Wise <pabs@debian.org> - 2021-09-03 05:30 +0200
Re: Finding rough consensus on level of vendoring for large upstreams "Bastien Roucariès" <roucaries.bastien@gmail.com> - 2021-09-03 10:20 +0200
Re: Finding rough consensus on level of vendoring for large upstreams Adrian Bunk <bunk@debian.org> - 2021-09-12 19:20 +0200
Re: Finding rough consensus on level of vendoring for large upstreams Phil Morrell <debian@emorrp1.name> - 2021-09-15 17:40 +0200
| From | Phil Morrell <debian@emorrp1.name> |
|---|---|
| Date | 2021-09-03 00:40 +0200 |
| Subject | Finding rough consensus on level of vendoring for large upstreams |
| Message-ID | <CT3xg-4QP-9@gated-at.bofh.it> |
[Multipart message — attachments visible in raw view] — view raw
Over this last year there seems to have been a noticeable divergence of maintainer opinion, on what has become known as vendoring, from a strict reading of [policy 4.13]. I think it's notable that the heading is [Embedded] copies and was [Convenience] copies since its inception, thankfully I found a request to expand this section using [vendoring]. [policy 4.13]: https://www.debian.org/doc/debian-policy/ch-source.html#embedded-code-copies [Embedded]: https://bugs.debian.org/955036 [Convenience]: https://bugs.debian.org/392362 [vendoring]: https://bugs.debian.org/907051 It is my reading of the situation that not only has this practice become more prevalent across multiple ecosystems since 2008, but that it can be a good thing when upstreams use it to better modularise their code. As a consequence, and in particular for large upstream projects, it is not a good use of maintainer time to package every single vendored library as a separate source package. See e.g. [kubernetes], [python BoF @25mins], [android-platform-tools], and even [uscan] grouping used by nodejs. [kubernetes]: https://bugs.debian.org/971515#172 [python BoF @25mins]: https://meetings-archive.debian.net/pub/debian-meetings/2021/DebConf21/debconf21-97-python-team-bof.webm [android-platform-tools]: https://salsa.debian.org/android-tools-team/admin/-/issues/40 [uscan]: https://manpages.debian.org/uscan#grouped_package Is there any objection to the following summary? 1. If the reused code is small and intended to be embedded into a package, then this MUST be documented in the [security-tracker]. 2. If the included project has already been packaged, then the Debian version SHOULD be used instead. 3. If modifications have been made, then those changes SHOULD be forwarded and/or the package ported to the official version. 4. When 2 or 3 are too onerous to maintain, the package MAY use the convenience copy but MUST document why in README.source and SHOULD be included in the [security-tracker]. 5. Where only a small number of unrelated projects are bundled, they SHOULD be uploaded as separate source packages. 6. If the upstream authors are largely the same, then vendored sub-projects MAY simply be built together as the same source. 7. A large number of vendored dependencies used only together for a single Debian package MAY be grouped into a single source upload. 8. If 6 or 7 are used initially but a new package has some overlap, then the new package MUST NOT duplicate the vendoring. The duplication SHOULD be packaged separately, then the original package SHOULD be updated to use the Debian version instead. 9. When upstream has a proven track record of promptly handling security vulnerabilities inside their vendored dependencies, then maintainers SHOULD follow the same practice, updating versions in lockstep. I might be misusing the MUST/SHOULD/MAY, so those can be dropped as needed, but I tried to capture the accepted practice and deliberately used all the different historical terms. For comparison, there's also [Fedora] policy, but apart from not having an in-band "bundled", I also think that the line has ended up being drawn marginally differently. [security-tracker]: https://salsa.debian.org/security-tracker-team/security-tracker/-/blob/master/data/embedded-code-copies [Fedora] https://docs.fedoraproject.org/en-US/packaging-guidelines/#bundling
[toc] | [next] | [standalone]
| From | Jérémy Lal <kapouer@melix.org> |
|---|---|
| Date | 2021-09-03 01:20 +0200 |
| Message-ID | <CT40h-5gQ-1@gated-at.bofh.it> |
| In reply to | #101500 |
[Multipart message — attachments visible in raw view] — view raw
Le ven. 3 sept. 2021 à 00:39, Phil Morrell <debian@emorrp1.name> a écrit : > Over this last year there seems to have been a noticeable divergence of > maintainer opinion, on what has become known as vendoring, from a strict > reading of [policy 4.13]. I think it's notable that the heading is > [Embedded] copies and was [Convenience] copies since its inception, > thankfully I found a request to expand this section using [vendoring]. > > [policy 4.13]: > https://www.debian.org/doc/debian-policy/ch-source.html#embedded-code-copies > [Embedded]: https://bugs.debian.org/955036 > [Convenience]: https://bugs.debian.org/392362 > [vendoring]: https://bugs.debian.org/907051 > > It is my reading of the situation that not only has this practice become > more prevalent across multiple ecosystems since 2008, but that it can be > a good thing when upstreams use it to better modularise their code. As a > consequence, and in particular for large upstream projects, it is not a > good use of maintainer time to package every single vendored library as > a separate source package. See e.g. [kubernetes], [python BoF @25mins], > [android-platform-tools], and even [uscan] grouping used by nodejs. > > [kubernetes]: https://bugs.debian.org/971515#172 > [python BoF @25mins]: > https://meetings-archive.debian.net/pub/debian-meetings/2021/DebConf21/debconf21-97-python-team-bof.webm > [android-platform-tools > <https://meetings-archive.debian.net/pub/debian-meetings/2021/DebConf21/debconf21-97-python-team-bof.webm%5Bandroid-platform-tools>]: > https://salsa.debian.org/android-tools-team/admin/-/issues/40 > [uscan]: https://manpages.debian.org/uscan#grouped_package > > Is there any objection to the following summary? > > 1. If the reused code is small and intended to be embedded into a > package, then this MUST be documented in the [security-tracker]. > 2. If the included project has already been packaged, then the Debian > version SHOULD be used instead. > 3. If modifications have been made, then those changes SHOULD be > forwarded and/or the package ported to the official version. > 4. When 2 or 3 are too onerous to maintain, the package MAY use the > convenience copy but MUST document why in README.source and SHOULD be > included in the [security-tracker]. > 5. Where only a small number of unrelated projects are bundled, they > SHOULD be uploaded as separate source packages. > 6. If the upstream authors are largely the same, then vendored > sub-projects MAY simply be built together as the same source. > 7. A large number of vendored dependencies used only together for a > single Debian package MAY be grouped into a single source upload. > 8. If 6 or 7 are used initially but a new package has some overlap, then > the new package MUST NOT duplicate the vendoring. The duplication > SHOULD be packaged separately, then the original package SHOULD be > updated to use the Debian version instead. > 9. When upstream has a proven track record of promptly handling security > vulnerabilities inside their vendored dependencies, then maintainers > SHOULD follow the same practice, updating versions in lockstep. > a couple naive questions: - should a package debian/control list bundled dependencies to make sure to avoid duplications ? - when a bundled package dependency is already available in debian, and is the same (unpatched), should the upstream source tarball be repacked without that dependency, or kept inside the source tarball ? Jérémy
[toc] | [prev] | [next] | [standalone]
| From | Phil Morrell <debian@emorrp1.name> |
|---|---|
| Date | 2021-09-03 03:10 +0200 |
| Message-ID | <CT5Sp-6s4-1@gated-at.bofh.it> |
| In reply to | #101501 |
[Multipart message — attachments visible in raw view] — view raw
On Fri, Sep 03, 2021 at 01:03:35AM +0200, Jérémy Lal wrote: > - should a package debian/control list bundled dependencies to make > sure to avoid duplications ? Maybe? I noted in my final paragraph that Fedora has a mechanism for this that we don't, but perhaps Provides is sufficient. > - when a bundled package dependency is already available in debian, > and is the same (unpatched), should the upstream source tarball be > repacked without that dependency, or kept inside the source tarball ? I omitted this from the policy side, because it seems like this is already answered in ftp-master practice. Provided the vendored copy is not used during the build and unless there is a *different* reason for repacking with Files-Excluded, then I see no reason to remove it. -- emorrp1
[toc] | [prev] | [next] | [standalone]
| From | Gunnar Wolf <gwolf@debian.org> |
|---|---|
| Date | 2021-09-03 17:30 +0200 |
| Message-ID | <CTjiF-77Q-3@gated-at.bofh.it> |
| In reply to | #101505 |
[Multipart message — attachments visible in raw view] — view raw
Phil Morrell dijo [Fri, Sep 03, 2021 at 02:04:44AM +0100]: > On Fri, Sep 03, 2021 at 01:03:35AM +0200, Jérémy Lal wrote: > > - should a package debian/control list bundled dependencies to make > > sure to avoid duplications ? > > Maybe? I noted in my final paragraph that Fedora has a mechanism for > this that we don't, but perhaps Provides is sufficient. Although it very seldom the case IMO. Even if we don't take into account the horrible practice of vendoring *and then patching* libraries done by some upstreams, we do ship (and our shipped packages depend on) specific versions of libraries. One of the ugly things about vendoring is that they bundle _other_ specific versions of libraries -- and dependencies are often quite hard to update without wrecking havoc in its whole ecosystem :-( > I omitted this from the policy side, because it seems like this is > already answered in ftp-master practice. Provided the vendored copy is > not used during the build and unless there is a *different* reason for > repacking with Files-Excluded, then I see no reason to remove it. Completely agree. Many packages vendor i.e. rendering libraries to produce their documentation at build time, and such libraries are not needed for the package once built. My example might not be great, because we still like building the documentation (and thus they would not qualify for Files-Excluded, only for omitting them from the binary package), but you get the idea :)
[toc] | [prev] | [next] | [standalone]
| From | Jonas Smedegaard <jonas@jones.dk> |
|---|---|
| Date | 2021-09-03 02:50 +0200 |
| Message-ID | <CT5z3-65c-5@gated-at.bofh.it> |
| In reply to | #101500 |
[Multipart message — attachments visible in raw view] — view raw
Hi Phil, First of all, thanks for compiling the list of reasonings. I get the impression that you are framing current state of embedding as a generally good thing to do, and if I understand that correctly then I disagree with it. I suspect that it helps if separating reasons for _encouraging_ embedding (tiny upstream projects and deeply integrated sets of upstreams, I guess) from reasons for _discouraging_ embdding (all other cases, I guess). Quoting Phil Morrell (2021-09-03 00:38:35) > 5. Where only a small number of unrelated projects are bundled, they > SHOULD be uploaded as separate source packages. Concretely I think not I but ftpmaster objects to the above: Node.js packages embed unrelated packages to meet ftpmaster requirement of a minimum size source package. - Jonas -- * Jonas Smedegaard - idealist & Internet-arkitekt * Tlf.: +45 40843136 Website: http://dr.jones.dk/ [x] quote me freely [ ] ask before reusing [ ] keep private
[toc] | [prev] | [next] | [standalone]
| From | Phil Morrell <debian@emorrp1.name> |
|---|---|
| Date | 2021-09-03 03:40 +0200 |
| Message-ID | <CT6lr-6C5-1@gated-at.bofh.it> |
| In reply to | #101504 |
[Multipart message — attachments visible in raw view] — view raw
On Fri, Sep 03, 2021 at 02:46:20AM +0200, Jonas Smedegaard wrote: > First of all, thanks for compiling the list of reasonings. Thanks for taking the time to read through it, I was hoping it would be a useful observation. > I get the impression that you are framing current state of embedding as > a generally good thing to do, and if I understand that correctly then I > disagree with it. ish? I mostly tried to document current practice rather than have an opinion on it being good. I do think that the evidence of multiple independent maintainer teams coming to similar conclusions on the basis of lack of user benefit and drag on new version velocity indicates the positive side. I believe, based on only a day's investigation, that you are in the minority here. I don't mean that as a bad thing - 1/3 of DDs disagree(d) with offering non-free alongside main - but I'd like to hear why you think the maintainers I gave as examples should use their Debian time to unvendor everything? > I suspect that it helps if separating reasons for _encouraging_ > embedding (tiny upstream projects and deeply integrated sets of > upstreams, I guess) from reasons for _discouraging_ embdding (all other > cases, I guess). I think the expanded points I gave empower maintainers to make the best decision for their own packages. By laying out the permitted reasons clearly, it's implied other reasons are not valid, but there's probably something I haven't thought of. However #907051 also wanted more background on _why_ one might choose one way or the other, so please do elaborate on this if you can. > Quoting Phil Morrell (2021-09-03 00:38:35) > > 5. Where only a small number of unrelated projects are bundled, they > > SHOULD be uploaded as separate source packages. > > Concretely I think not I but ftpmaster objects to the above: Node.js > packages embed unrelated packages to meet ftpmaster requirement of a > minimum size source package. No, I think Node.js is covered by #7 (large number of deps). With #5 I was attempting to capture the current policy for when _not_ to bundle. Thanks for the additional background about why the bundling happens. -- emorrp1
[toc] | [prev] | [next] | [standalone]
| From | Jonas Smedegaard <jonas@jones.dk> |
|---|---|
| Date | 2021-09-03 05:00 +0200 |
| Message-ID | <CT7AR-7rd-1@gated-at.bofh.it> |
| In reply to | #101508 |
[Multipart message — attachments visible in raw view] — view raw
Quoting Phil Morrell (2021-09-03 03:30:04) > On Fri, Sep 03, 2021 at 02:46:20AM +0200, Jonas Smedegaard wrote: > > First of all, thanks for compiling the list of reasonings. > > Thanks for taking the time to read through it, I was hoping it would > be a useful observation. > > > I get the impression that you are framing current state of embedding > > as a generally good thing to do, and if I understand that correctly > > then I disagree with it. > > ish? I mostly tried to document current practice rather than have an > opinion on it being good. I do think that the evidence of multiple > independent maintainer teams coming to similar conclusions on the > basis of lack of user benefit and drag on new version velocity > indicates the positive side. > > I believe, based on only a day's investigation, that you are in the > minority here. I don't mean that as a bad thing - 1/3 of DDs > disagree(d) with offering non-free alongside main - but I'd like to > hear why you think the maintainers I gave as examples should use their > Debian time to unvendor everything? I do not think that those maintainers you gave as example should do differently. My point is that those you gave as example are *exceptions* to a general practice in Debian of _avoiding_ embedding. > > I suspect that it helps if separating reasons for _encouraging_ > > embedding (tiny upstream projects and deeply integrated sets of > > upstreams, I guess) from reasons for _discouraging_ embdding (all > > other cases, I guess). > > I think the expanded points I gave empower maintainers to make the > best decision for their own packages. By laying out the permitted > reasons clearly, it's implied other reasons are not valid, but there's > probably something I haven't thought of. > > However #907051 also wanted more background on _why_ one might choose > one way or the other, so please do elaborate on this if you can. I am very worried about how complex node-* packages in Debian have become since ftpmasters explicitly stated a not-too-small rule and we began more aggressively embedding. E.g. version of each embedded project is hidden by default, and those packages manually adding virtual packages has no mechanism to ensure that versions don't jump backwards or disappear due to a typos. - Jonas -- * Jonas Smedegaard - idealist & Internet-arkitekt * Tlf.: +45 40843136 Website: http://dr.jones.dk/ [x] quote me freely [ ] ask before reusing [ ] keep private
[toc] | [prev] | [next] | [standalone]
| From | Pirate Praveen <praveen@onenetbeyond.org> |
|---|---|
| Date | 2021-09-03 18:50 +0200 |
| Message-ID | <CTky6-7Rw-3@gated-at.bofh.it> |
| In reply to | #101510 |
2021, സെപ്റ്റംബർ 3 8:22:51 AM IST, Jonas Smedegaard <jonas@jones.dk>ൽ എഴുതി >I am very worried about how complex node-* packages in Debian have >become since ftpmasters explicitly stated a not-too-small rule and we >began more aggressively embedding. E.g. version of each embedded >project is hidden by default, and those packages manually adding virtual >packages has no mechanism to ensure that versions don't jump backwards >or disappear due to a typos. Apparently reducing package metadata size is more important than every other consideration. It has become very difficult to get new people into js team after we started embedding as the complexity has increased very much from earlier days. -- Sent from my Android device with K-9 Mail. Please excuse my brevity.
[toc] | [prev] | [next] | [standalone]
| From | Simon McVittie <smcv@debian.org> |
|---|---|
| Date | 2021-09-03 11:30 +0200 |
| Subject | Bug#907051: Finding rough consensus on level of vendoring for large upstreams |
| Message-ID | <CTdGh-3p4-3@gated-at.bofh.it> |
| In reply to | #101504 |
On Fri, 03 Sep 2021 at 02:46:20 +0200, Jonas Smedegaard wrote:
> I suspect that it helps if separating reasons for _encouraging_
> embedding (tiny upstream projects and deeply integrated sets of
> upstreams, I guess) from reasons for _discouraging_ embdding (all other
> cases, I guess).
If the embedded project is specifically not maintained as API- or
ABI-stable, then I think that's also a common and valid reason to embed
it, perhaps even more so than the ones you mentioned here. This can mean
library APIs, but also command-line options, output behaviour or some
other machine interface that the embedding project might require.
(Or perhaps you intended "deeply integrated" to cover this, but I think
it's worth being explicit about the API stability factor.)
I'm mainly thinking here of "copylibs" like gnulib, libiberty, libglnx,
stb. If an upstream tells us their code will break API/ABI without
notice, then I think we should believe them - which means the only
long-term-maintainable way to depend on it is for the projects that need
it to vendor a known-compatible copy, and be responsible for updating
that copy and the code that calls it at the same time if a serious bug
is found. This is not new - gnulib and libiberty have been like this
for decades, and vendoring those is not really the same thing as vendoring
the likes of zlib or libjpeg.
I know we do have a libstb package that builds a shared library, but the
existence of a stb build system (let alone a shared library build system)
is a Debian-specific patch, which does not fill me with confidence
that it has the necessary API- and ABI-stability to be correct as a
shared library.
Outside the scope of libraries, a few packages in GNOME are currently
using gi-docgen (a gtk-doc-like API documentation generator) as a vendored
project, because it is intended to have a stable interface in future but
has not got there yet. There's a gi-docgen package in NEW, but it will
stay in experimental and will not be used in build-dependencies until
it has stabilized enough to be valid to use that way.
Of course, if an upstream decides that their project *is* ready to be a
sufficiently-stable separate project, then we should usually prefer to
package it separately (for example, bubblewrap and xdg-dbus-proxy are
vendored into flatpak, and the vendored copies were used in Debian early
in its lifetime, but both are now stable enough and independent enough
that we have moved to system copies).
smcv
[toc] | [prev] | [next] | [standalone]
| From | Paul Wise <pabs@debian.org> |
|---|---|
| Date | 2021-09-03 05:30 +0200 |
| Message-ID | <CT83T-7Qs-1@gated-at.bofh.it> |
| In reply to | #101500 |
On Thu, Sep 2, 2021 at 10:39 PM Phil Morrell wrote: > Over this last year there seems to have been a noticeable divergence of > maintainer opinion, on what has become known as vendoring Embedded copies of code/etc have downsides ... https://wiki.debian.org/EmbeddedCopies > It is my reading of the situation that not only has this practice become > more prevalent across multiple ecosystems since 2008 ... but there are many many copies in Debian and they are not going away upstream. > [security-tracker]: https://salsa.debian.org/security-tracker-team/security-tracker/-/blob/master/data/embedded-code-copies Side note: This file is very much outdated, new copies are introduced all the time and old copies get removed. This has always been the case and it always will be. So we need to cope with the consequences of this change toward embedding in the upstream FLOSS ecosystems. Personally, my recommendations are that: Debian package maintainers could investigate upstream tarballs for embedded copies before each upload containing a new/changed upstream tarball. Debian package maintainers could talk to upstream about removing embedded copies and replacing them with dependencies. Debian package maintainers could talk to upstream about upstreaming changes in modified embedded copies, removing the embedded copies and replacing them with dependencies. Debian package maintainers could use Files-Excluded or `rm -r` in debian/rules to ensure that embedded copies are not used by the build. Debian package maintainers could add hints to the source package about which embedded copies are definitely used. Debian security tracker could remove the perpetually outdated list of embedded copies. Debian security issue investigators could search the archive for similar or duplicate code (using the tools listed on the above wiki page), investigate the build logs for each package found and determine which packages are affected. This is a lot of work, but given the level of embedding we already have, it is already necessary. Also, the issue of static linking is similar; it is here, it isn't going away and so now we have to cope with it and the problems it causes are similar to embedded copies. https://wiki.debian.org/StaticLinking -- bye, pabs https://wiki.debian.org/PaulWise
[toc] | [prev] | [next] | [standalone]
| From | "Bastien Roucariès" <roucaries.bastien@gmail.com> |
|---|---|
| Date | 2021-09-03 10:20 +0200 |
| Message-ID | <CTcAy-2Kc-3@gated-at.bofh.it> |
| In reply to | #101500 |
[Multipart message — attachments visible in raw view] — view raw
Le jeudi 2 septembre 2021, 22:38:35 UTC Phil Morrell a écrit : > Over this last year there seems to have been a noticeable divergence of > maintainer opinion, on what has become known as vendoring, from a strict > reading of [policy 4.13]. I think it's notable that the heading is > [Embedded] copies and was [Convenience] copies since its inception, > thankfully I found a request to expand this section using [vendoring]. > > [policy 4.13]: > https://www.debian.org/doc/debian-policy/ch-source.html#embedded-code-copie > s [Embedded]: https://bugs.debian.org/955036 > [Convenience]: https://bugs.debian.org/392362 > [vendoring]: https://bugs.debian.org/907051 > > It is my reading of the situation that not only has this practice become > more prevalent across multiple ecosystems since 2008, but that it can be > a good thing when upstreams use it to better modularise their code. As a > consequence, and in particular for large upstream projects, it is not a > good use of maintainer time to package every single vendored library as > a separate source package. See e.g. [kubernetes], [python BoF @25mins], > [android-platform-tools], and even [uscan] grouping used by nodejs. > > [kubernetes]: https://bugs.debian.org/971515#172 > [python BoF @25mins]: > https://meetings-archive.debian.net/pub/debian-meetings/2021/DebConf21/debc > onf21-97-python-team-bof.webm [android-platform-tools]: > https://salsa.debian.org/android-tools-team/admin/-/issues/40 [uscan]: > https://manpages.debian.org/uscan#grouped_package For uscan you misread, the goal of the uscan grouped package is to avoid embeded copy. It decouple the concept of upstream package from debian package by grouping it, but (see for instance node-resolve unstable package), but decreasing the number of embeded copy in the distrib. - src:node-resolve embded a few related npm package: * node-resolve * node-types-resolve (the typescript API for this package but that is another packages upstream. It is crazy but welcome to nodejs/npm world) * is-core-module really small depends upstream of this package * path-parse really small depends upstream of this package The node-resolve arch:all debian package provides node-type-resolve (=version), node-is-core-module (=version) and so on. So apt-get install node- types-resolve works for user as expected. Moreover other package could depends on node-types-resolve (>> version) thus using the normal depends mechanism of debian. Moreover (it is for now contructed by manual rules but me and mabye yadd are planning to automate this rules), node-resolve create a /usr/share/doc/node- types-resolve directory with copyright and changelog.Debian.gz linked back to /usr/share/doc/node-resolve/copyright and /usr/share/doc/node-resolve/ changelog.Debian.gz BUT with documentation of node-types-resolve in order to allow switching easilly to non grouped package if needed (package being non leaf packages or bigger to pass ftpmaster constraint), and in order to let your user thinks node-types-resolve is a normal package, with its own documentation. Thus with my pkg-javascript team hat, I use the group feature of uscan, in order to deacrease embeded copy in javascript packages and to workarround npm one line one package culture. It is not perfect but it is I believe a good engineering trade of. The only problem is I am not english native speaker, and thus I have not documented why and how we use this feature. Patch to policy are welcome Moreover I believe that trackers should display the versionned provides and https://packages.debian.org/unstable/node-resolve sould also document versionned provides and https://packages.debian.org/unstable/node-types-resolve should redirect to d https://packages.debian.org/unstable/node-resolve in order to avoid to confuse our users. Bastien > d https://packages.debian.org/unstable/node-types-resolve > Is there any objection to the following summary? > > 1. If the reused code is small and intended to be embedded into a > package, then this MUST be documented in the [security-tracker]. > 2. If the included project has already been packaged, then the Debian > version SHOULD be used instead. > 3. If modifications have been made, then those changes SHOULD be > forwarded and/or the package ported to the official version. > 4. When 2 or 3 are too onerous to maintain, the package MAY use the > convenience copy but MUST document why in README.source and SHOULD be > included in the [security-tracker]. > 5. Where only a small number of unrelated projects are bundled, they > SHOULD be uploaded as separate source packages. > 6. If the upstream authors are largely the same, then vendored > sub-projects MAY simply be built together as the same source. > 7. A large number of vendored dependencies used only together for a > single Debian package MAY be grouped into a single source upload. > 8. If 6 or 7 are used initially but a new package has some overlap, then > the new package MUST NOT duplicate the vendoring. The duplication > SHOULD be packaged separately, then the original package SHOULD be > updated to use the Debian version instead. > 9. When upstream has a proven track record of promptly handling security > vulnerabilities inside their vendored dependencies, then maintainers > SHOULD follow the same practice, updating versions in lockstep. > > I might be misusing the MUST/SHOULD/MAY, so those can be dropped as > needed, but I tried to capture the accepted practice and deliberately > used all the different historical terms. For comparison, there's also > [Fedora] policy, but apart from not having an in-band "bundled", I also > think that the line has ended up being drawn marginally differently. > > [security-tracker]: > https://salsa.debian.org/security-tracker-team/security-tracker/-/blob/mast > er/data/embedded-code-copies [Fedora] > https://docs.fedoraproject.org/en-US/packaging-guidelines/#bundling
[toc] | [prev] | [next] | [standalone]
| From | Adrian Bunk <bunk@debian.org> |
|---|---|
| Date | 2021-09-12 19:20 +0200 |
| Message-ID | <CWB9o-37V-1@gated-at.bofh.it> |
| In reply to | #101500 |
On Thu, Sep 02, 2021 at 11:38:35PM +0100, Phil Morrell wrote: >... > 4. When 2 or 3 are too onerous to maintain, the package MAY use the > convenience copy but MUST document why in README.source and SHOULD be > included in the [security-tracker]. >... The package MUST be listed as being without security support in the debian-security-support package and the Release Notes, and it MUST NOT be installed as part of a default installation, unless the security team has explicitly agreed to support it. If we are shipping software where Debian cannot provide security support, then we shouldn't hide the problem from our users. cu Adrian
[toc] | [prev] | [next] | [standalone]
| From | Phil Morrell <debian@emorrp1.name> |
|---|---|
| Date | 2021-09-15 17:40 +0200 |
| Message-ID | <CXFaV-2OG-1@gated-at.bofh.it> |
| In reply to | #101676 |
[Multipart message — attachments visible in raw view] — view raw
Thanks to Adrian and pabs for their corrections on documenting security support, and there wasn't too much objection to the summary, more to the sad state of affairs that leads to it and a bit of clarification. I believe all the major points have cc'd 907051, so would like to encourage someone more familiar with policy process than I am to draft an amendment. There should be enough written there now to expand the section accordingly with more recommendations, and possibly file wishlist bugs for maintainers to document their reasons in the source. https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=907051
[toc] | [prev] | [standalone]
Back to top | Article view | linux.debian.devel
csiph-web