Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update copyright dates for generated Cargo manifests #85

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nspin
Copy link
Member

@nspin nspin commented Feb 12, 2024

2023 -> 2024

@lsf37
Copy link
Member

lsf37 commented Feb 12, 2024

Copyright years should usually not be updated in bulk, at least not if the idea is to figure out the year of the most recent original copyright on the file. It doesn't renew automatically, only if there have been significant changes to the file.

Lots of people do this, but they are wrong ;-)

@nspin
Copy link
Member Author

nspin commented Feb 13, 2024

I'm aware of the general rules, but I thought this case might be different.

All Cargo.toml files in this repo are generated from adjacent Cargo.nix files. The Cargo.nix files have their own copyright lines. I have not changed any of those. However, their corresponding Cargo.toml files get their copyright lines from this snippet:

copyrightLines = {
coliasgroup = "Copyright 2023, Colias Group, LLC";
};

For this PR, I changed the year in that snippet and then ran the typical Cargo.toml re-generation script with make update-generated-sources.

What do you think about this particular case? The Cargo.toml files are new in the sense that they're freshly generated, but old in the sense that they are identical in content to older outputs of make update-generated-sources.

An alternative would be to include the year for the copyright line at the source-level in Cargo.nix files. That would feel a bit clunky, but would yield more control if such control were necessary.

@lsf37
Copy link
Member

lsf37 commented Feb 13, 2024

What do you think about this particular case? The Cargo.toml files are new in the sense that they're freshly generated, but old in the sense that they are identical in content to older outputs of make update-generated-sources.

An alternative would be to include the year for the copyright line at the source-level in Cargo.nix files. That would feel a bit clunky, but would yield more control if such control were necessary.

The copyright of generated code would be a mix of the copyright of the input to the generator and the generator itself, but the year/time it is generated does not really enter into it. So I guess the correct thing would be to take the year of the input file on this one.

It's not really that big a deal, I confess I just saw the large number of files with the bulk update and reacted to that. If it is too much of a hassle, I'd actually just leave it. Generally you don't really gain anything from updating the copyright year anyway. I try to keep it current on the files with significant changes, but we probably should just really not bother, because by the time copyright runs out I hope we have shifted languages and maybe computing paradigms a few times :-)

@nspin
Copy link
Member Author

nspin commented Feb 13, 2024

If it is too much of a hassle, I'd actually just leave it. Generally you don't really gain anything from updating the copyright year anyway.

What about Cargo.toml files generated from Cargo.nix files that were created in 2024?

@lsf37
Copy link
Member

lsf37 commented Feb 13, 2024

If it is too much of a hassle, I'd actually just leave it. Generally you don't really gain anything from updating the copyright year anyway.

What about Cargo.toml files generated from Cargo.nix files that were created in 2024?

Those would be fine

@nspin
Copy link
Member Author

nspin commented Feb 13, 2024

Those would be fine

By this, do you mean that it would be fine to label those with the year 2023?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants