search

WiseEarthTechnology / CrisisCommunicator

http://WiseEarthTechnology.com
5 stars 7 forks source link

Removed LICENCE.txt header from *.py source files #18

Closed varrunr closed 11 years ago

varrunr commented 11 years ago

targetfiles=$(grep -lr --include *.py "Redistribution and use of the software" webapp) for f in $targetfiles do if [ $f != "./LICENSE.txt" ]; then sed '1,29d' $f > temp mv temp $f fi done

varrunr commented 11 years ago

FYI, the script has not been added to the repo. Create a new issue to add metadata.

swiftarrow commented 11 years ago

Cool! If I read it properly, this script removes the license by chopping the first few lines of each python file. Does this need to be tested or can we directly merge the pull request? It says that the pull request can be automatically merged.

varrunr commented 11 years ago

I just reversed the script which inserted the license which was $ cat LICENSE.txt Since the license is always at the top, it is safe to remove just the first 29 lines, the license. Testing wont be required because actual code was not added/removed.