1 |
On Wed, Nov 14, 2018 at 4:00 PM Patrick McLean <chutzpah@g.o> wrote: |
2 |
> |
3 |
> I don't object to adding the copyright notice to the commit message |
4 |
> rather than the ebuild, the only caveat is that the copyright should be |
5 |
> propigated to the rsync tree as well (since most users use rsync rather |
6 |
> than git). Perhaps the "fattening" step could do it, even just a |
7 |
> top-level file listing packages and copyright owners, should not be |
8 |
> overly hard to generate, maybe with a post-push hook that updates it |
9 |
> when a copyright tag is detected in the push. |
10 |
|
11 |
IMO documenting copyright data in git would be a value-add, and could |
12 |
potentially be assisted by repoman just as signed-off-by tags are. |
13 |
|
14 |
Whether it gets propagated to rsync I'll leave to others who care |
15 |
more. Personally I don't get why we propagate anything other than the |
16 |
ebuilds to rsync, but if rsync users want to punish themselves who am |
17 |
I to argue with them? :) |
18 |
|
19 |
> |
20 |
> IANAL nor do I pretend to be one. A lawyer did instruct me to add the |
21 |
> copyright notices to ebuilds that I work on during work hours. |
22 |
|
23 |
In my experience corporate lawyers have a lot of incentive to do this |
24 |
sort of thing. |
25 |
|
26 |
First, it costs them nothing to direct people to do this. It doesn't |
27 |
really cost the company much either, so there won't be push-back |
28 |
internally. |
29 |
|
30 |
It is also easy to explain a policy like this to a pointy-haired boss |
31 |
type. It gets seen as "doing something" to protect the company as |
32 |
well. Whether it actually is making a difference is another matter, |
33 |
but it isn't like anybody is going to ever look into that. It isn't |
34 |
adding any risk. |
35 |
|
36 |
At work I've seen far more costly policies than this established by |
37 |
lawyers that are of dubious value to the company, but since the lawyer |
38 |
doesn't get paid less if the policy is expensive, but they might lose |
39 |
their job if anything bad happens, they have a lot of incentive to |
40 |
enact every possible rule so that they can be seen as being innocent |
41 |
if anything goes wrong. |
42 |
|
43 |
I think a lot of it just happens without any malicious intent. Lawyer |
44 |
is asked for opinion, lawyer gives opinion. Everybody is happy. :) |
45 |
|
46 |
> |
47 |
> AFAIK no one on this list/thread is a lawyer, so much if this is not |
48 |
> particularly valuable. |
49 |
|
50 |
Well, if somebody who is a lawyer wants to add perspective I'm all |
51 |
ears, but this is a community-based FOSS projects and decisions are |
52 |
generally made based on arguments debated in public, and not "a friend |
53 |
of a friend talked to a lawyer and got some free advice so we better |
54 |
follow it." A lot of pros and cons have been presented for this |
55 |
change, and those in charge can weigh them, but I've never really been |
56 |
a fan of arguments that deference must be made to arguments that |
57 |
aren't even presented simply due to somebody holding a credential. |
58 |
|
59 |
> Personally, I don't see why there is a strong objection to a practice that is quite common in open source code. |
60 |
|
61 |
I suspect that some of this is due to the nature of ebuilds themselves |
62 |
and our workflow. |
63 |
|
64 |
If you have some file full of 2000 lines of source code, you're going |
65 |
to spend all your time buried in whatever function you're refactoring |
66 |
and you're not really looking at the boilerplate at the start of the |
67 |
file. You might spend days looking at the file and your editor |
68 |
remembers where you left off. You never even look at the copyright |
69 |
notices and a few more lines isn't a big deal. You don't even look at |
70 |
the other functions in the file you're editing but trust them to honor |
71 |
their APIs. |
72 |
|
73 |
On the other hand Gentoo ebuilds might only be 10-20 lines long, and a |
74 |
lot of the stuff that people are often looking at (keywords, iuse, |
75 |
my_foo, etc) are right at the top. Often this stuff is "above the |
76 |
fold" as they say. Sticking another half a dozen lines of cruft at |
77 |
the top means that you're hunting more to find the stuff you care |
78 |
about, and the copyright boilerplate could become half the file |
79 |
contents in a file that is mostly boilerplate already. |
80 |
|
81 |
Maybe a compromise would be to stick all the copyright cruft at the |
82 |
end of the file, but IMO it is better stored in git, preferably in |
83 |
some standardized format that could be parsed by tools. |
84 |
|
85 |
-- |
86 |
Rich |