Merge lp:~robru/unity-webapps-cuttherope/require-utils into lp:unity-webapps-cuttherope

[Robert Bruce Park]

This should fix the recent build error[0], I hope. I'm looking for the CI feedback here ;-)

[0] https://launchpadlibrarian.net/139243760/buildlog_ubuntu-raring-i386.unity-webapps-cuttherope_2.2daily13.05.06ubuntu.unity.next-0ubuntu1_FAILEDTOBUILD.txt.gz

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:30
http://jenkins.qa.ubuntu.com/job/unity-webapps-cuttherope-ci/1/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-webapps-cuttherope-raring-amd64-ci/1/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-webapps-cuttherope-raring-armhf-ci/1/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-webapps-cuttherope-ci/1/rebuild

review: Needs Fixing (continuous-integration)

[Robert Bruce Park (robru)]

Ugh! Not rules.mk again! This error is because jenkins is missing the build-dep from the PPA.

[Didier Roche-Tolomelli (didrocks)]

I think you need to check with vrruiz, they should include the dest ppa, which is next.

[Víctor R. Ruiz (vrruiz)]

Robert: ppa:webapps/staging has been added as jenkins hook. Please, resubmit.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:30
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~robru/unity-webapps-cuttherope/require-utils/+merge/163031/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/unity-webapps-cuttherope-ci/2/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-webapps-cuttherope-raring-amd64-ci/2/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-webapps-cuttherope-raring-armhf-ci/2/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-webapps-cuttherope-ci/2/rebuild

review: Needs Fixing (continuous-integration)

[Alexandre Abreu (abreu-alexandre)]

Mmmh why would you need the require when you don't really use any of it? ... I am not sure jslint supports const (which is not EcmaScript5), ...

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:30
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~robru/unity-webapps-cuttherope/require-utils/+merge/163171/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/unity-webapps-cuttherope-ci/3/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-webapps-cuttherope-raring-amd64-ci/3/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-webapps-cuttherope-raring-armhf-ci/3/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-webapps-cuttherope-ci/3/rebuild

review: Needs Fixing (continuous-integration)

Unmerged revisions

=== modified file 'CutTheRope.user.js'
--- CutTheRope.user.js	2013-05-03 23:10:57 +0000
+++ CutTheRope.user.js	2013-05-09 16:00:58 +0000
@@ -1,5 +1,6 @@
 // ==UserScript==
 // @include       http://*.cuttherope.ie/*
+// @require       utils.js
 // ==/UserScript==
 
 // This placeholder gets munged with real data at build time.