Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/626
---
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 user pdion891 commented on the pull request:
https://github.com/apache/cloudstack/pull/626#issuecomment-125581299
I will go ahead and apply this PR to fix builds, @rsafonseca can you create
a PR with your line that does not look at line 22 ?
@wilderrodrigues, I know! it
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/626#issuecomment-125581160
I'm merging it as the line#22 will be fixed in another PR:
https://github.com/apache/cloudstack/commit/e6f81dbbbc8b597755007d9e6d247d2d6ec6c74c#commi
Github user rsafonseca commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/626#discussion_r35639533
--- Diff: debian/rules ---
@@ -1,6 +1,6 @@
#!/usr/bin/make -f
# -*- makefile -*-
-VERSION := $(shell mvn
org.apache.maven.plugins:maven-h
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/626#issuecomment-125573670
@pdion891
Thanks for the PR. I'm still a bit puzzled because the latest build (from
27th July) also failed by did not mention the dependency.
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/626#issuecomment-125522926
LGTM
The thing with the line number could be done in a different PR.
Cheers,
Wilder
Sent from my iPhone
On 28 Jul 2015,
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/626#discussion_r35628826
--- Diff: debian/rules ---
@@ -1,6 +1,6 @@
#!/usr/bin/make -f
# -*- makefile -*-
-VERSION := $(shell mvn
org.apache.maven.plugins:maven-
Github user rsafonseca commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/626#discussion_r35627314
--- Diff: debian/rules ---
@@ -1,6 +1,6 @@
#!/usr/bin/make -f
# -*- makefile -*-
-VERSION := $(shell mvn
org.apache.maven.plugins:maven-h
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/626#discussion_r35620265
--- Diff: debian/rules ---
@@ -1,6 +1,6 @@
#!/usr/bin/make -f
# -*- makefile -*-
-VERSION := $(shell mvn
org.apache.maven.plugins:maven-
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/626#issuecomment-125471653
:+1:
---
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
ena
GitHub user pdion891 opened a pull request:
https://github.com/apache/cloudstack/pull/626
fix systemvm64template build, fix .deb package creation.
- build .deb package job must be change for ``dpkg-buildpackage -uc -us``
ubuntu 14.04 enforce signature of package. CLOUDSTACK-8638
11 matches
Mail list logo
