You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by wido <gi...@git.apache.org> on 2015/09/07 16:58:57 UTC
[GitHub] cloudstack pull request: debian: Updated dependencies for Packages
GitHub user wido opened a pull request:
https://github.com/apache/cloudstack/pull/783
debian: Updated dependencies for Packages
A few dependencies have been updated to their latest version and some
have been removed.
The ordering for some dependencies has been changed so that we will
depend on Java 8 over Java 7 when doing a new install.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/wido/cloudstack debian-depends
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/783.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 #783
----
commit ae5025aca8ce7c0a29e2dcf0db86dbb862546796
Author: Wido den Hollander <wi...@widodh.nl>
Date: 2015-09-07T14:45:34Z
debian: Updated dependencies for Packages
A few dependencies have been updated to their latest version and some
have been removed.
The ordering for some dependencies has been changed so that we will
depend on Java 8 over Java 7 when doing a new install.
----
---
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.
---
[GitHub] cloudstack pull request: debian: Updated dependencies for Packages
Posted by wido <gi...@git.apache.org>.
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/783#issuecomment-138465225
@resmo Fair enough. These changes are just some small fixes. A new PR with those fixes is fine with me, but I think we can merge this one?
---
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.
---
[GitHub] cloudstack pull request: debian: Updated dependencies for Packages
Posted by resmo <gi...@git.apache.org>.
Github user resmo commented on the pull request:
https://github.com/apache/cloudstack/pull/783#issuecomment-138466873
It depends, openjdk-8 would not work with current init script in my opinion.
---
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.
---
[GitHub] cloudstack pull request: debian: Updated dependencies for Packages
Posted by wilderrodrigues <gi...@git.apache.org>.
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/783#issuecomment-154341186
#1040
---
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.
---
[GitHub] cloudstack pull request: debian: Updated dependencies for Packages
Posted by wido <gi...@git.apache.org>.
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/783#issuecomment-138475391
@resmo Yes, we need to fix that further. It works now on Ubuntu 12.04 and 14.04, but for the LTS coming in 16.04 we need to have this fixed.
---
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.
---
[GitHub] cloudstack pull request: debian: Updated dependencies for Packages
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/783
---
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.
---
[GitHub] cloudstack pull request: debian: Updated dependencies for Packages
Posted by milamberspace <gi...@git.apache.org>.
Github user milamberspace commented on the pull request:
https://github.com/apache/cloudstack/pull/783#issuecomment-138449931
Check on packages.ubuntu.com for the requirements of version/packages. All is good for 12.04/14.04/14.10/15.04.
LGTM
(Perhaps remove the tomcat6: exists only on 12.04, but tomcat7 already exists for the ubuntu version)
---
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.
---
[GitHub] cloudstack pull request: debian: Updated dependencies for Packages
Posted by resmo <gi...@git.apache.org>.
Github user resmo commented on the pull request:
https://github.com/apache/cloudstack/pull/783#issuecomment-138463931
IMHO we need some more changes here, and it might be more clear and easier to handle if we would make a package per ubuntu release and debian release, beginning from ubuntu 12.04.x and debian 7.x
Some problems I have identified.
* tomcat6.conf in https://github.com/apache/cloudstack/blob/master/debian/cloudstack-management.install#L28
* jvm hardcoded https://github.com/apache/cloudstack/blob/master/packaging/debian/init/cloud-management#L68
* tomcat6.conf in https://github.com/apache/cloudstack/blob/master/packaging/debian/init/cloud-management#L38
* TOMCAT6_USER=tomcat6 in https://github.com/apache/cloudstack/blob/master/packaging/debian/init/cloud-management#L64
---
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.
---