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 headers to 2015 #7205

Merged
merged 3 commits into from
Jan 5, 2016

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Dec 13, 2015

This can be merged into 0.12 and master without conflicts or rebase.

The first commit will fix the python helper script introduced in #3646 to no longer remove previous copyright years.

@maflcko maflcko mentioned this pull request Dec 13, 2015
@@ -1,4 +1,4 @@
// Copyright (c) 2013 The Bitcoin Core developers
// Copyright (c) 2015 The Bitcoin Core developers
Copy link
Member Author

Choose a reason for hiding this comment

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

This should update to 2013-2015. (A copyright year should never be removed)

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated OP

@maflcko maflcko force-pushed the MarcoFalke-2015-BumpHeaders-0.12 branch 2 times, most recently from fa180ce to fa8580d Compare December 13, 2015 16:57
@maflcko maflcko force-pushed the MarcoFalke-2015-BumpHeaders-0.12 branch from fa8580d to fa24439 Compare December 13, 2015 17:09
@jonasschnelli
Copy link
Contributor

Does it only change files that where changed during that year? Or does the script just changes every copyright information?

@fanquake
Copy link
Member

It should only change files that have been modified.

@maflcko
Copy link
Member Author

maflcko commented Dec 14, 2015

Right, it only updates files where GitAuthorDate of the last commit to this file contains 2015.

@jonasschnelli
Copy link
Contributor

Concept ACK.

@maflcko maflcko force-pushed the MarcoFalke-2015-BumpHeaders-0.12 branch from fa9a575 to fa71669 Compare December 26, 2015 16:54
@laanwj laanwj merged commit fa71669 into bitcoin:master Jan 5, 2016
laanwj added a commit that referenced this pull request Jan 5, 2016
fa71669 [devtools] Use git pretty-format for year parsing (MarcoFalke)
fa24439 Bump copyright headers to 2015 (MarcoFalke)
fa6ad85 [devtools] Rewrite fix-copyright-headers.py (MarcoFalke)
laanwj pushed a commit that referenced this pull request Jan 5, 2016
- Bump copyright headers to 2015
- [devtools] Rewrite fix-copyright-headers.py
- [devtools] Use git pretty-format for year parsing

Github-Pull: #7205
Rebased-From: fa6ad85 fa24439 fa71669
@maflcko maflcko deleted the MarcoFalke-2015-BumpHeaders-0.12 branch January 5, 2016 13:24
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants