Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35239 )
Change subject: hatch: automate creating a new variant in coreboot ......................................................................
Patch Set 14:
(2 comments)
https://review.coreboot.org/c/coreboot/+/35239/13/util/mainboard/google/hatc... File util/mainboard/google/hatch/create_coreboot_variant.sh:
https://review.coreboot.org/c/coreboot/+/35239/13/util/mainboard/google/hatc... PS13, Line 54: # Start a branch. Use YMD timestamp to avoid collisions. : DATE=$(date +%Y%m%d) : repo start "create_${variant}_${DATE}"
Ok, I guess i'd let the user do all the git work, but that's just my own preference. […]
Those are all good ideas, but not for this revision.
https://review.coreboot.org/c/coreboot/+/35239/14/util/mainboard/google/hatc... File util/mainboard/google/hatch/create_coreboot_variant.sh:
https://review.coreboot.org/c/coreboot/+/35239/14/util/mainboard/google/hatc... PS14, Line 54: # Copy a template file to a new name while substituting the : # current year for __YEAR__
I've had feedback both ways. At first, it said 2019 (as a constant) in the template. […]
The initial copy of a file to another directory isn't a copyrightable work anyway, but the intent was that when the author came along to make changes to those files for the variant, the copyright would already be correct. Martin has pointed out that Google LLC is already listed in the AUTHORS file, and so the copyright notices are redundant. So, there will be no copyright line, and no __YEAR__ to update.