GitHub user ahgittin opened a pull request:
https://github.com/apache/incubator-brooklyn/pull/874
Update for 0.8.0 release
This tidies the release process instructions (which were excellent,
@rdowner !).
It also improves a problem with the download links at Apache (on top of
#873).
And it adds `r.js` which I think completes addressing the comments on
licenses from the 0.7.0-incubating release, /cc @hzbarcea .
For reference the comments there were:
> LICENCE is missing standard appendix
addressed with LICENSE file overhaul, now includes standard appendices and
clearer format
> probably no need for the full text of MIT and BSD as the short version is
preferred and already used
we still use the full text but this appears to be standard practice in
other projects, and safer
> LICENSE is missing:
> riak (see
./software/nosql/src/main/resources/brooklyn/entity/nosql/riak/vm.args + others)
these files are *our* work based on config files supplied by riak. both
theirs and ours are examples which people are intended to modify, i understand
in such cases -- same as cassandra and mysql config files -- these should not
have license headers nor be claimed as used under license. people need to be
able to modify these without license restrictions!
> normalize.css (see
./sandbox/mobile-app/src/main/webapp/assets/mobile/libs/mcasimir-mobile-angular-ui-be39011/dist/css/mobile-angular-ui-base.css)
> mobileangular-ui (see
./sandbox/mobile-app/src/main/webapp/assets/mobile/libs/mcasimir-mobile-angular-ui-be39011/dist/js/mobile-angular-ui.js)
> angular (see
./sandbox/mobile-app/src/main/webapp/assets/mobile/libs/angular-1.2.19)
these are in a sandbox project not included in the releases AFAIK, so i
think it is correct not to list them? (if this is incorrect i think we should
simply remove sandbox items; they can be maintained outwith apache or in a
sub-git-repo.)
> r.js (see ./usage/jsgui/src/build/requirejs-maven-plugin/r.js)
good spot, added this
---
we still need to:
* run the license update to incorporate r.js
* back port this to 0.8.0 prior to releasing it and run the license update
there also
i suggest that be done by the person merging this.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ahgittin/incubator-brooklyn release-update
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-brooklyn/pull/874.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #874
----
commit 48d06240189db8ff13107c57e4a37ec935703b65
Author: Alex Heneveld <[email protected]>
Date: 2015-09-01T19:24:25Z
minor mods to the release instructions
commit bcb2d52fc3f229647cebada32f50477467d20e67
Author: Alex Heneveld <[email protected]>
Date: 2015-09-02T07:46:45Z
use closer.lua not closer.cgi for apache mirror links
this is what cassandra for instance does, it means the mirror site can
follow links.
commit f2ace507a5404cac70d7c8c9fdb7b615b7c667be
Author: Alex Heneveld <[email protected]>
Date: 2015-09-02T08:54:06Z
add r.js to list of dependencies
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
