| Age | Commit message (Collapse) | Author | Files | Lines |
|---|
|
This file was never truly necessary and has never actually been used in
the history of Tailscale's open source releases.
A Brief History of AUTHORS files
---
The AUTHORS file was a pattern developed at Google, originally for
Chromium, then adopted by Go and a bunch of other projects. The problem
was that Chromium originally had a copyright line only recognizing
Google as the copyright holder. Because Google (and most open source
projects) do not require copyright assignemnt for contributions, each
contributor maintains their copyright. Some large corporate contributors
then tried to add their own name to the copyright line in the LICENSE
file or in file headers. This quickly becomes unwieldy, and puts a
tremendous burden on anyone building on top of Chromium, since the
license requires that they keep all copyright lines intact.
The compromise was to create an AUTHORS file that would list all of the
copyright holders. The LICENSE file and source file headers would then
include that list by reference, listing the copyright holder as "The
Chromium Authors".
This also become cumbersome to simply keep the file up to date with a
high rate of new contributors. Plus it's not always obvious who the
copyright holder is. Sometimes it is the individual making the
contribution, but many times it may be their employer. There is no way
for the proejct maintainer to know.
Eventually, Google changed their policy to no longer recommend trying to
keep the AUTHORS file up to date proactively, and instead to only add to
it when requested: https://opensource.google/docs/releasing/authors.
They are also clear that:
> Adding contributors to the AUTHORS file is entirely within the
> project's discretion and has no implications for copyright ownership.
It was primarily added to appease a small number of large contributors
that insisted that they be recognized as copyright holders (which was
entirely their right to do). But it's not truly necessary, and not even
the most accurate way of identifying contributors and/or copyright
holders.
In practice, we've never added anyone to our AUTHORS file. It only lists
Tailscale, so it's not really serving any purpose. It also causes
confusion because Tailscalars put the "Tailscale Inc & AUTHORS" header
in other open source repos which don't actually have an AUTHORS file, so
it's ambiguous what that means.
Instead, we just acknowledge that the contributors to Tailscale (whoever
they are) are copyright holders for their individual contributions. We
also have the benefit of using the DCO (developercertificate.org) which
provides some additional certification of their right to make the
contribution.
The source file changes were purely mechanical with:
git ls-files | xargs sed -i -e 's/\(Tailscale Inc &\) AUTHORS/\1 contributors/g'
Updates #cleanup
Change-Id: Ia101a4a3005adb9118051b3416f5a64a4a45987d
Signed-off-by: Will Norris <will@tailscale.com>
|
|
This produces the following omitsizes output:
Starting with everything and removing a feature...
tailscaled tailscale combined (linux/amd64)
27005112 18153656 39727288
- 7696384 - 7282688 -19607552 .. remove *
- 167936 - 110592 - 245760 .. remove acme
- 1925120 - 0 - 7340032 .. remove aws
- 4096 - 0 - 8192 .. remove bird
- 20480 - 12288 - 32768 .. remove capture
- 0 - 57344 - 61440 .. remove completion
- 249856 - 696320 - 692224 .. remove debugeventbus
- 12288 - 4096 - 24576 .. remove debugportmapper
- 0 - 0 - 0 .. remove desktop_sessions
- 815104 - 8192 - 544768 .. remove drive
- 65536 - 356352 - 425984 .. remove kube
- 233472 - 286720 - 311296 .. remove portmapper (and debugportmapper)
- 90112 - 0 - 110592 .. remove relayserver
- 655360 - 712704 - 598016 .. remove serve (and webclient)
- 937984 - 0 - 950272 .. remove ssh
- 708608 - 401408 - 344064 .. remove syspolicy
- 0 - 4071424 -11132928 .. remove systray
- 159744 - 61440 - 225280 .. remove taildrop
- 618496 - 454656 - 757760 .. remove tailnetlock
- 122880 - 0 - 131072 .. remove tap
- 442368 - 0 - 483328 .. remove tpm
- 16384 - 0 - 20480 .. remove wakeonlan
- 278528 - 368640 - 286720 .. remove webclient
Starting at a minimal binary and adding one feature back...
tailscaled tailscale combined (linux/amd64)
19308728 10870968 20119736 omitting everything
+ 352256 + 454656 + 643072 .. add acme
+ 2035712 + 0 + 2035712 .. add aws
+ 8192 + 0 + 8192 .. add bird
+ 20480 + 12288 + 36864 .. add capture
+ 0 + 57344 + 61440 .. add completion
+ 262144 + 274432 + 266240 .. add debugeventbus
+ 344064 + 118784 + 360448 .. add debugportmapper (and portmapper)
+ 0 + 0 + 0 .. add desktop_sessions
+ 978944 + 8192 + 991232 .. add drive
+ 61440 + 364544 + 425984 .. add kube
+ 331776 + 110592 + 335872 .. add portmapper
+ 122880 + 0 + 102400 .. add relayserver
+ 598016 + 155648 + 737280 .. add serve
+ 1142784 + 0 + 1142784 .. add ssh
+ 708608 + 860160 + 720896 .. add syspolicy
+ 0 + 4079616 + 6221824 .. add systray
+ 180224 + 65536 + 237568 .. add taildrop
+ 647168 + 393216 + 720896 .. add tailnetlock
+ 122880 + 0 + 126976 .. add tap
+ 446464 + 0 + 454656 .. add tpm
+ 20480 + 0 + 24576 .. add wakeonlan
+ 1011712 + 1011712 + 1138688 .. add webclient (and serve)
Fixes #17139
Change-Id: Ia91be2da00de8481a893243d577d20e988a0920a
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
Starting at a minimal binary and adding one feature back...
tailscaled tailscale combined (linux/amd64)
30073135 17451704 31543692 omitting everything
+ 480302 + 10258 + 493896 .. add debugportmapper
+ 475317 + 151943 + 467660 .. add portmapper
+ 500086 + 162873 + 510511 .. add portmapper+debugportmapper
Fixes #17148
Change-Id: I90bd0e9d1bd8cbe64fa2e885e9afef8fb5ee74b1
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
When you say --features=foo,bar, that was supposed to mean
to only show features "foo" and "bar" in the table.
But it was also being used as the set of all features that are
omittable, which was wrong, leading to misleading numbers
when --features was non-empty.
Updates #12614
Change-Id: Idad2fa67fb49c39454032e84a3dede967890fdf5
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
And remove a bogus omit feature from feature/featuretags.
Updates #12614
Change-Id: I0a08183fb75c73ae75b6fd4216d134e352dcf5a0
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
Updates #12614
Change-Id: I4012c33095c6a7ccf80ad36dbab5cedbae5b3d47
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
|
Updates #12614
Change-Id: I8f85d7275bc8eecedbabe6631b50e1cf70791d2d
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
|
