Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1641)

Issue 9836045: Add AGPL license and headers.

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 9 months ago by teknico
Modified:
10 years, 9 months ago
Reviewers:
frankban, mp+168711, matthew.scott
Visibility:
Public.

Description

Add AGPL license and headers. Add headers pointing to the AGPL license to almost all files, and add the license text itself. Sorry for the size, it's just comments, no behavior change. https://code.launchpad.net/~teknico/charms/precise/juju-gui/add-agpl/+merge/168711 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 2

Patch Set 2 : Add AGPL license and headers. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1028 lines, -9 lines) Patch
A COPYING View 1 chunk +661 lines, -0 lines 0 comments Download
M Dependencies.md View 2 chunks +9 lines, -2 lines 0 comments Download
M HACKING.md View 1 chunk +9 lines, -0 lines 0 comments Download
M Makefile View 1 chunk +15 lines, -1 line 0 comments Download
M README.md View 1 chunk +9 lines, -0 lines 0 comments Download
A [revision details] View 1 1 chunk +2 lines, -0 lines 0 comments Download
M config.yaml View 1 1 chunk +16 lines, -0 lines 0 comments Download
M copyright View 1 chunk +3 lines, -3 lines 0 comments Download
M hooks/backend.py View 1 chunk +16 lines, -0 lines 0 comments Download
M hooks/bootstrap_utils.py View 1 chunk +16 lines, -0 lines 0 comments Download
M hooks/config-changed View 1 chunk +15 lines, -2 lines 0 comments Download
M hooks/install View 1 chunk +16 lines, -0 lines 0 comments Download
M hooks/start View 1 chunk +16 lines, -0 lines 0 comments Download
M hooks/stop View 1 chunk +16 lines, -0 lines 0 comments Download
M hooks/utils.py View 1 1 chunk +16 lines, -0 lines 0 comments Download
M hooks/web-relation-joined View 1 chunk +16 lines, -0 lines 0 comments Download
M metadata.yaml View 1 chunk +16 lines, -0 lines 0 comments Download
M revision View 1 1 chunk +1 line, -1 line 0 comments Download
M tests/00-setup View 1 chunk +16 lines, -0 lines 0 comments Download
M tests/10-unit.test View 1 chunk +16 lines, -0 lines 0 comments Download
M tests/20-functional.test View 1 chunk +16 lines, -0 lines 0 comments Download
M tests/deploy.py View 1 chunk +16 lines, -0 lines 0 comments Download
M tests/helpers.py View 1 chunk +16 lines, -0 lines 0 comments Download
M tests/requirements.pip View 1 chunk +16 lines, -0 lines 0 comments Download
M tests/test_backends.py View 1 chunk +16 lines, -0 lines 0 comments Download
M tests/test_deploy.py View 1 chunk +16 lines, -0 lines 0 comments Download
M tests/test_helpers.py View 1 chunk +16 lines, -0 lines 0 comments Download
M tests/test_utils.py View 1 chunk +16 lines, -0 lines 0 comments Download

Messages

Total messages: 5
teknico
Please take a look.
10 years, 9 months ago (2013-06-11 15:38:48 UTC) #1
matthew.scott
LGTM - thanks for all the headers!
10 years, 9 months ago (2013-06-11 17:10:10 UTC) #2
frankban
LGTM, I successfully run the tests. Even if this branch does not include code changes ...
10 years, 9 months ago (2013-06-12 08:22:08 UTC) #3
teknico
Revision incremented as suggested. Thanks, frankban. https://codereview.appspot.com/9836045/diff/1/hooks/utils.py File hooks/utils.py (right): https://codereview.appspot.com/9836045/diff/1/hooks/utils.py#newcode3 hooks/utils.py:3: # This file ...
10 years, 9 months ago (2013-06-12 09:02:03 UTC) #4
teknico
10 years, 9 months ago (2013-06-12 09:04:54 UTC) #5
*** Submitted:

Add AGPL license and headers.

Add headers pointing to the AGPL license to almost all files,
and add the license text itself.

Sorry for the size, it's just comments, no behavior change.

R=matthew.scott, frankban
CC=
https://codereview.appspot.com/9836045
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b