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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to