[GitHub] cloudstack pull request: Add improved support for packaging CloudS...

2014-11-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/46 --- 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

[GitHub] cloudstack pull request: Add improved support for packaging CloudS...

2014-11-26 Thread spark404
Github user spark404 commented on the pull request: https://github.com/apache/cloudstack/pull/46#issuecomment-64660575 @damodarreddy That commit is good, but incomplete. I used that as the basis for this commit, but there are so many small changes between centos 6 and centos 7, that

[GitHub] cloudstack pull request: Add improved support for packaging CloudS...

2014-11-26 Thread spark404
Github user spark404 commented on the pull request: https://github.com/apache/cloudstack/pull/46#issuecomment-64660807 @bhaisaab @damodarreddy @wilderrodrigues To be clear, any objections to merging this? The jenkins build is here, as travis does not do packaging:

[GitHub] cloudstack pull request: Add improved support for packaging CloudS...

2014-11-26 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/46#issuecomment-64665068 on no word the coming 24h I'll do the honours. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack pull request: Add improved support for packaging CloudS...

2014-11-26 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/46#issuecomment-64671315 No objections... --- 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

[GitHub] cloudstack pull request: Add improved support for packaging CloudS...

2014-11-26 Thread damodarreddy
Github user damodarreddy commented on the pull request: https://github.com/apache/cloudstack/pull/46#issuecomment-64747208 No Objections --- 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

[GitHub] cloudstack pull request: Add improved support for packaging CloudS...

2014-11-25 Thread spark404
GitHub user spark404 opened a pull request: https://github.com/apache/cloudstack/pull/46 Add improved support for packaging CloudStack for CentOS 7 This feature splits the packaging for CentOS 6 and CentOS 7 again. With the introduction of tomcat7 and systemd to much changes to

[GitHub] cloudstack pull request: Add improved support for packaging CloudS...

2014-11-25 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/46#issuecomment-64387309 Looks good, but I see most of the scripts in the new centos7 packaging is the same including systemd init scripts, why not use ifs and elses? I'm interested because

[GitHub] cloudstack pull request: Add improved support for packaging CloudS...

2014-11-25 Thread spark404
Github user spark404 commented on the pull request: https://github.com/apache/cloudstack/pull/46#issuecomment-64430200 @bhaisaab some of the files still need to be replaced, like the init scripts for the other services. They are copied for now and will be replaced by cents 7 counter

[GitHub] cloudstack pull request: Add improved support for packaging CloudS...

2014-11-25 Thread damodarreddy
Github user damodarreddy commented on the pull request: https://github.com/apache/cloudstack/pull/46#issuecomment-64516356 Already a commit is pushed to support RHEL7 build/install at 7ea7deded031b43042c68db0f7c5c6c8b21c18c2. Any problems with that commit? --- If your