incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rohit Yadav <bhais...@apache.org>
Subject Re: console-proxy dir move; issues building on master
Date Wed, 13 Feb 2013 06:28:48 GMT
Yes Dave I confirm the issue, these commits broke the build:

commit 4869f0cacfc1319ac725e35ac4ebd1b59da8e5e1
Author: Alex Huang <alex.huang@citrix.com>
Date:   Tue Feb 12 15:59:02 2013 -0800

    Moved console-proxy into services

commit 2293caa32e4387d7c71c1d4f677e3015856eff0a
Author: Alex Huang <alex.huang@citrix.com>
Date:   Tue Feb 12 15:58:38 2013 -0800

    moved console proxy into services directory.  no code change

I'll try to fix them on my end unless someone beats me to it.
Note to developers: please do a clean build if you refactor something
and better run cloudstack to confirm that your refactoring actually
works.

Regards.

On Wed, Feb 13, 2013 at 11:05 AM, Dave Cahill <dcahill@midokura.com> wrote:
> This appears to have broken the build according to jenkins too:
> https://builds.apache.org/job/cloudstack-master-maven/707/
>
> From jenkins' console output:
>
> message : Failed to execute goal
> org.apache.maven.plugins:maven-antrun-plugin:1.7:run
> (generate-resource) on project cloud-client-ui: An Ant BuildException
> has occured: /home/jenkins/jenkins-slave/workspace/cloudstack-master-maven/services/console-proxy/server/dist
> does not exist.
> around Ant part ...<copy
> todir="/home/jenkins/jenkins-slave/workspace/cloudstack-master-maven/client/target/generated-webapp/WEB-INF/classes/vms">...
> @ 17:130 in /home/jenkins/jenkins-slave/workspace/cloudstack-master-maven/client/target/antrun/build-main.xml
> cause : An Ant BuildException has occured:
> /home/jenkins/jenkins-slave/workspace/cloudstack-master-maven/services/console-proxy/server/dist
> does not exist.
> around Ant part ...<copy
> todir="/home/jenkins/jenkins-slave/workspace/cloudstack-master-maven/client/target/generated-webapp/WEB-INF/classes/vms">...
> @ 17:130 in /home/jenkins/jenkins-slave/workspace/cloudstack-master-maven/client/target/antrun/build-main.xml
> Stack trace :
> org.apache.maven.lifecycle.LifecycleExecutionException: Failed to
> execute goal org.apache.maven.plugins:maven-antrun-plugin:1.7:run
> (generate-resource) on project cloud-client-ui: An Ant BuildException
> has occured: /home/jenkins/jenkins-slave/workspace/cloudstack-master-maven/services/console-proxy/server/dist
> does not exist.
> around Ant part ...<copy
> todir="/home/jenkins/jenkins-slave/workspace/cloudstack-master-maven/client/target/generated-webapp/WEB-INF/classes/vms">...
> @ 17:130 in /home/jenkins/jenkins-slave/workspace/cloudstack-master-maven/client/target/antrun/build-main.xml
>
>
>
> On Wed, Feb 13, 2013 at 1:56 PM, Dave Cahill <dcahill@midokura.com> wrote:
>
>> Hi,
>>
>> With a fresh checkout of master, I tried:
>>
>> mvn -P developer,systemvm clean install
>>
>>
>> And got:
>>
>>
>> [ERROR] Failed to execute goal org.apache.maven.plugins:maven-antrun-plugin:1.7:run
(generate-resource) on project cloud-client-ui: An Ant BuildException has occured: /root/incubator-cloudstack/services/console-proxy/server/dist
does not exist.
>>
>> [ERROR] around Ant part ...<copy todir="/root/incubator-cloudstack/client/target/generated-webapp/WEB-INF/classes/vms">...
@ 17:95 in /root/incubator-cloudstack/client/target/antrun/build-main.xml
>>
>> [ERROR] -> [Help 1]
>>
>>
>> I can see that the console-proxy folder was moved in a checkin this morning, but
I can't see what's wrong with that checkin.
>>
>> https://git-wip-us.apache.org/repos/asf?p=incubator-cloudstack.git;a=commit;h=4869f0cacfc1319ac725e35ac4ebd1b59da8e5e1
>>
>>
>> https://git-wip-us.apache.org/repos/asf?p=incubator-cloudstack.git;a=commit;h=2293caa32e4387d7c71c1d4f677e3015856eff0a
>>
>>
>> Is anyone else seeing the same issue? Proposed fixes?
>>
>>
>> I also noticed while debugging that the CentOS spec file will likely need to change
to deal with the move of console-proxy:
>>
>> [root@devcloud-kvm incubator-cloudstack]# grep -R "console-proxy/dist" *
>>
>> packaging/centos63/cloud.spec:install -D console-proxy/dist/systemvm.iso ${RPM_BUILD_ROOT}%{_datadir}/%{name}-common/vms/systemvm.iso
>>
>> packaging/centos63/cloud.spec:install -D console-proxy/dist/systemvm.zip ${RPM_BUILD_ROOT}%{_datadir}/%{name}-common/vms/systemvm.zip
>>
>>
>> Thanks,
>>
>> Dave.
>>
>>
>>

Mime
View raw message