You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ra...@apache.org on 2015/04/29 05:45:49 UTC

git commit: updated refs/heads/master to 07c3089

Repository: cloudstack
Updated Branches:
  refs/heads/master 29b880066 -> 07c30895b


fix CLOUDSTACK-8402

Signed-off-by: Rajani Karuturi <ra...@gmail.com>

This closes #205


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/07c30895
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/07c30895
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/07c30895

Branch: refs/heads/master
Commit: 07c30895bb3d7f48b90f859751a659c9ea004288
Parents: 29b8800
Author: Satoru Nakaya <gi...@gmail.com>
Authored: Wed Apr 29 00:43:03 2015 +0900
Committer: Rajani Karuturi <ra...@gmail.com>
Committed: Wed Apr 29 09:15:28 2015 +0530

----------------------------------------------------------------------
 packaging/centos63/cloud.spec | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/07c30895/packaging/centos63/cloud.spec
----------------------------------------------------------------------
diff --git a/packaging/centos63/cloud.spec b/packaging/centos63/cloud.spec
index 8064594..7161398 100644
--- a/packaging/centos63/cloud.spec
+++ b/packaging/centos63/cloud.spec
@@ -113,7 +113,7 @@ The Apache CloudStack files shared between agent and management server
 %package agent
 Summary: CloudStack Agent for KVM hypervisors
 Requires: openssh-clients
-Requires: java => 1.7.0
+Requires: java-1.7.0-openjdk
 Requires: %{name}-common = %{_ver}
 Requires: libvirt
 Requires: bridge-utils
@@ -151,7 +151,7 @@ The CloudStack baremetal agent
 
 %package usage
 Summary: CloudStack Usage calculation server
-Requires: java => 1.7.0
+Requires: java-1.7.0-openjdk
 Requires: jsvc
 Requires: jakarta-commons-daemon
 Requires: jakarta-commons-daemon-jsvc


RE: git commit: updated refs/heads/master to 07c3089

Posted by Koushik Das <ko...@citrix.com>.
Agree. The commit message should provide a summary of the changes.

-----Original Message-----
From: Daan Hoogland [mailto:daan.hoogland@gmail.com] 
Sent: Wednesday, 29 April 2015 17:07
To: dev
Subject: Re: git commit: updated refs/heads/master to 07c3089

On Wed, Apr 29, 2015 at 5:45 AM,  <ra...@apache.org> wrote:
> fix CLOUDSTACK-8402

I don't think we should accept commit messages like this. thoughts?


-- 
Daan

Re: git commit: updated refs/heads/master to 07c3089

Posted by Wido den Hollander <wi...@widodh.nl>.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1



On 04/29/2015 01:36 PM, Daan Hoogland wrote:
> On Wed, Apr 29, 2015 at 5:45 AM,  <ra...@apache.org> wrote:
>> fix CLOUDSTACK-8402
> 
> I don't think we should accept commit messages like this.
> thoughts?
> 

I agree. They should be more descriptive.

Wido

> 
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIcBAEBAgAGBQJVQMiSAAoJEAGbWC3bPspCKlEP/1bzhNitvPkcBR0gqCXr2G/A
t8jTzRaBHc2erdpAnTiYBsLgzAEE1PdS4g1E5D94Dei6Od8mjoCnwiRt1O5Ty+26
VNEBVqBGs7GgLNDye63DKYqK4IhCRzuk8zvllDVNeuhLulyPx9Sg6vkG4Qeq6ybN
6ekWcvA/+TTLAZ+zTzpWPy6zBesZ+0IwmIlLyEpHVmm+dkazlwGVLNoa5DWx5oW0
cvj0ryEuxJjCNlKnkURQHmR+UgiknS11V6Vg1Um/tiusrb+6qyUiukjOxBAOl3Xo
9uyNrmog11gkq+bGbw0MOCMUPlyLBW0a8R93C0GAe//ds2qRRgu6uBKqTRFCx1Vv
pwYGK639RpZW/P5y/hliNNttbKAKOgFetJ/5rXIkYYMNjVwA8n9ErL6NnpNY8iMx
mjiZkyvHJ7DlL00AAoao0Qxcq2HvjRiIuKs4uY0ttHcyHkygQNHfa47qMO7UZIoU
N33zmnVyoKU8Yqs++gd0aCZo+ARI688TEcJ8DMLBYY7gXPobCG5lU2E8Dy0MPGDG
jmcpXQupGCogpkDNqqmjeEtM/EYi2Ut8y3sAWbARqw0bozX3955BoBFqa0+rP9e0
oWL0Ct48ryfn8CXphq1DQWIlPGCEm2xK4iLOxYiayW2j2xM+jib6W90zkIkYFNVz
GSQWdqxib8nsAfQHFnRY
=eSYl
-----END PGP SIGNATURE-----

Re: git commit: updated refs/heads/master to 07c3089

Posted by Daan Hoogland <da...@gmail.com>.
On Wed, Apr 29, 2015 at 5:45 AM,  <ra...@apache.org> wrote:
> fix CLOUDSTACK-8402

I don't think we should accept commit messages like this. thoughts?


-- 
Daan