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

Updated year on hkdf.c #24239

Closed
wants to merge 1 commit into from
Closed

Updated year on hkdf.c #24239

wants to merge 1 commit into from

Conversation

cm3173
Copy link

@cm3173 cm3173 commented Apr 23, 2024

Updated the year on the license disclaimer. @bbbrumley

CLA: trivial

Checklist
  • documentation is added or updated
  • tests are added or updated

Updated the year on the license disclaimer. 

CLA: trivial
@@ -1,5 +1,5 @@
/*
* Copyright 2021-2023 The OpenSSL Project Authors. All Rights Reserved.
* Copyright 2021-2024 The OpenSSL Project Authors. All Rights Reserved.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, it doesn't look to me that this file was updated in 2024 - the last change I see in here is

commit 86db958835d1f8ba9ce49a9f93b5309c3d13b91c
Author: James Muir <james@openssl.org>
Date:   Fri Nov 10 14:02:00 2023 -0500

    demos: tidy up makefiles, fix warnings

@mattcaswell
Copy link
Member

Generally speaking updating the copyright year in files like this is not normally necessary. We have a script that does this when we create a release from a branch (i.e. if a year is wrong it will be corrected when the contents of the master branch become the 3.4 release at some point in the future).

https://github.com/openssl/tools/blob/master/release-tools/do-copyright-year

@bbbrumley
Copy link
Contributor

Thanks both for the feedback

@cm3173 In light of the comments, probably best to close this one out

Nevertheless, thank you for submitting your PR

@t8m t8m added triaged: cleanup The issue/pr deals with cleanup of comments/docs not altering code significantly resolved: not a bug The issue is not considered a bug resolved: wont fix The issue has been confirmed but won't be fixed and removed resolved: not a bug The issue is not considered a bug labels Apr 23, 2024
@cm3173 cm3173 closed this Apr 23, 2024
@cm3173
Copy link
Author

cm3173 commented Apr 23, 2024

I will be closing it now. Thank you for working on this great project :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
resolved: wont fix The issue has been confirmed but won't be fixed triaged: cleanup The issue/pr deals with cleanup of comments/docs not altering code significantly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants