Copyright line should append after the SPDX identifier

[Dentrax]

I just tried SPDX flag first time and noticed addlicense adds copyright line before the SPDX identifier.

$ addlicense -s=only -c Furkan .

Actual:

Copyright ...
SPDX-License-Identifier:

My expectation according to other open source projects, solidity, etc.:

SPDX-License-Identifier:
Copyright ...

Any ways to reorder of those? Shouldn't the right way is the bottom one?

[willnorris]

I don't think there is a "right way". I've always listed copyright first, since that is how non-SPDX license declarations nearly always appear. I've never paid much attention to how others do it. I think it should be trivial to reverse using a custom license template?

[willnorris]

also regarding any "right way", unfortunately the SPDX docs only talk about the SPDX line itself, nothing about where it falls in relation to other information: https://spdx.dev/ids/

Though they do have a few additional links:

• the linux guidelines to explicitly specify, but a code search shows they seem to do SPDX first
• Zephyr appears to put copyright first, at least in one linked example
• hypledger seems to put copyright first
• this post from Intel shows copyright first
• etc, etc

So, there does not seem to be consensus, but most seem to list copyright first. I wonder if linux has SPDX first because they added it to most files after the fact, and it's much easier to automate simply adding it as the first line of files. This looks to be the main commit where that happened.

[Dentrax]

Thanks for the prompt response and the references! I think passing a custom license template would better in this case, as you said.