brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ahgittin <...@git.apache.org>
Subject [GitHub] incubator-brooklyn pull request: Update for 0.8.0 release
Date Wed, 02 Sep 2015 09:06:28 GMT
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 <alex.heneveld@cloudsoftcorp.com>
Date:   2015-09-01T19:24:25Z

    minor mods to the release instructions

commit bcb2d52fc3f229647cebada32f50477467d20e67
Author: Alex Heneveld <alex.heneveld@cloudsoftcorp.com>
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 <alex.heneveld@cloudsoftcorp.com>
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message