Page MenuHomeWildfire Games

Small LICENES.txt cleanup
AcceptedPublic

Authored by elexis on Nov 5 2018, 3:03 PM.

Details

Reviewers
asterix
Summary

The sentence from rP6911 should be updated to reflect that we ought to have most licensing information by now.
The pathnames look like unix format, but they aren't, since paths starting with / are considered absolute.

Test Plan

Check whether the first sentence of the document shouldn't be more conditional, as one can't generalize GPL v2+ features for the entire package.

Diff Detail

Repository
rP 0 A.D. Public Repository
Branch
/ps/trunk
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 6425
Build 10644: Vulcan Build
Build 10643: arc lint + arc unit

Event Timeline

elexis created this revision.Nov 5 2018, 3:03 PM
asterix accepted this revision.Nov 6 2018, 2:31 PM
asterix added a subscriber: asterix.

The text is correct. Thank you for the corrections.

This revision is now accepted and ready to land.Nov 6 2018, 2:31 PM
smiley added a subscriber: smiley.Nov 6 2018, 5:38 PM
smiley added inline comments.
LICENSE.txt
26

Doesn't this imply that the code is also under the LICENSE.txt, or is that covered by L17?

elexis marked an inline comment as done.Nov 6 2018, 6:02 PM
elexis added inline comments.
LICENSE.txt
26

Doesn't LICENSE.txt of that directoy explain the licensing within that directory?

smiley added inline comments.Nov 6 2018, 6:30 PM
LICENSE.txt
26

Apparently so. The other directories are still kinda grey though, aren't they?
(there are height maps in there too.)