Skip to content

Loading…

mojito yaf-develop to use the bleeding version of yui to keep them in sync #451

Merged
merged 8 commits into from

4 participants

@caridy

trying to get travis to build yui npm module using after_install.

@caridy caridy mojito develep to use the bleeding version of yui to keep them in syn…
…c. trying to get travis to build yui npm module using after_install.
cc8fc22
@travisbot

This pull request fails (merged cc8fc22 into f5818a5).

@travisbot

This pull request fails (merged 4161ba0 into f5818a5).

@caridy caridy hmm, travis is not picking up the scripts, trying to use before inste…
…ad of after, adding more debug statements as well. take 2
1bf02d4
@travisbot

This pull request fails (merged 1bf02d4 into f5818a5).

@travisbot

This pull request fails (merged 0231358 into f5818a5).

@travisbot

This pull request fails (merged 904f9be into f5818a5).

@drewfish
Yahoo Inc. member

Is this the approach that YUI uses for their travis builds?

@drewfish
Yahoo Inc. member

From the travis logs: make: *** No rule to make target 'npm'. Stop.

@caridy caridy alright, few attempts to get this ready. It builds locally, missing t…
…ravis.sh so people can do it locally as well, but this one should pass.
66e0f07
@caridy

@drewfish yes, this is the approach they use. I had some issues with the path, and the workflow, should be solved with the last commit.

@travisbot

This pull request fails (merged 66e0f07 into f5818a5).

@travisbot

This pull request fails (merged 66e0f07 into f5818a5).

@travisbot

This pull request fails (merged b37483c into f5818a5).

@travisbot

This pull request fails (merged b37483c into f5818a5).

@caridy

@drewfish it seems to be working now. Can you try it locally, by running ./travis/travis.sh?

Things that are pending:

  • how to facilitate the use of this travis config for bleeding branches only but not on develop and master?
  • how to speed up the process since it takes a long time now due the YUI builder?
@mridgway
Yahoo Inc. member

+1, looks good.

@caridy caridy merged commit a87298c into yahoo:yaf-develop
@drewfish
Yahoo Inc. member

I got this error:

shifter --config build-npm.json
make: shifter: No such file or directory
make: *** [npm] Error 1
@drewfish
Yahoo Inc. member

Also, after cloning the YUI repo into yui-src, it doesn't appear to checkout the 3.x branch.

@caridy

@drewfish I added the -b 3.x option on the clone command as part of #459

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Aug 29, 2012
  1. @caridy

    mojito develep to use the bleeding version of yui to keep them in syn…

    caridy committed
    …c. trying to get travis to build yui npm module using after_install.
  2. @caridy

    hmm, travis is not picking up the scripts, trying to use before inste…

    caridy committed
    …ad of after, adding more debug statements as well.
  3. @caridy

    hmm, travis is not picking up the scripts, trying to use before inste…

    caridy committed
    …ad of after, adding more debug statements as well. take 2
  4. @caridy
  5. @caridy
  6. @caridy

    alright, few attempts to get this ready. It builds locally, missing t…

    caridy committed
    …ravis.sh so people can do it locally as well, but this one should pass.
  7. @caridy

    fixing the symlink reference so travis can use the bleeding yui modul…

    caridy committed
    …e from yui-src/build-npm folder
  8. @caridy
Showing with 94 additions and 1 deletion.
  1. +4 −0 .travis.yml
  2. +1 −1 package.json
  3. +21 −0 travis/README.md
  4. +22 −0 travis/before.sh
  5. +33 −0 travis/install.sh
  6. +13 −0 travis/travis.sh
View
4 .travis.yml
@@ -3,6 +3,10 @@ node_js:
- 0.8
- 0.6
- 0.4
+before_install:
+ - ./travis/before.sh
+install:
+ - ./travis/install.sh
notifications:
email:
recipients:
View
2 package.json
@@ -29,7 +29,7 @@
"semver": "1.0.14",
"wrench": "~1.3.9",
"ycb": "~1.0.0",
- "yui": "git://github.com/yui/yui3.git#3.x",
+ "yui": "3.x",
"yuidocjs": "~0.3.14",
"yuitest": "~0.7.4"
},
View
21 travis/README.md
@@ -0,0 +1,21 @@
+Mojito testing with TravisCI
+====================================
+
+This directory contains the scripts used to automate Mojito testing with TravisCI for our bleeding
+branches, specifically those using the latest YUI version.
+
+Scripts
+-------
+
+ * `before.sh` - Runs in the `before_install` build step to clone and create the latest YUI npm package
+ * `install.sh` - Runs in the `install` build step to do an `npm install` for mojito dependencies
+ and `npm install` on YUI's `build-npm` (created from the above step)
+ * `travis.sh` - Runs the travis tests locally for testing, not used in the Travis build.
+
+Running the test locally
+-------------------------
+
+Clone the repo, then:
+
+ cd mojito;
+ ./travis/travis.sh
View
22 travis/before.sh
@@ -0,0 +1,22 @@
+#!/bin/bash
+
+echo "Before Script: `pwd`"
+
+cd "$(dirname "$0")"
+
+echo "cd: `pwd`"
+
+if [ -n "$TRAVIS" ]; then
+ echo "Installing Shifter.."
+ npm -g install shifter -loglevel silent
+fi
+
+# YUI bleeding
+cd ../
+echo "Cloning YUI Repository"
+git clone git://github.com/yui/yui3.git yui-src
+wait
+cd ./yui-src/src/yui
+echo "Making YUI NPM Module"
+make npm
+wait
View
33 travis/install.sh
@@ -0,0 +1,33 @@
+#!/bin/bash
+
+echo "Starting YUI install: `pwd`"
+
+cd "$(dirname "$0")"
+
+echo "cd: `pwd`"
+
+# Building NPM module
+cd ../
+echo "NPM Build Dir: `pwd`"
+wait
+echo "Installing NPM Modules"
+npm install -loglevel silent
+wait
+
+# Building YUI module
+cd ./yui-src/build-npm;
+echo "YUI NPM Build Dir: `pwd`"
+wait
+echo "Installing YUI NPM Module"
+npm install -loglevel silent
+wait
+cd ../../
+
+# Linking YUI module
+echo "Linking YUI NPM..."
+wait
+rm -rf ./node_modules/yui
+ln -s ../yui-src/build-npm ./node_modules/yui
+
+echo "YUI NPM Install Complete"
+echo ""
View
13 travis/travis.sh
@@ -0,0 +1,13 @@
+#!/bin/bash
+
+echo "Starting a Travis Build"
+
+cd "$(dirname "$0")"
+
+cd ../
+
+./travis/before.sh
+wait
+./travis/install.sh
+wait
+npm test
Something went wrong with that request. Please try again.