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

Merged
merged 8 commits into from Aug 30, 2012

Conversation

Projects
None yet
4 participants
@caridy
Collaborator

caridy commented Aug 29, 2012

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

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.
@travisbot

This comment has been minimized.

Show comment
Hide comment
@travisbot

travisbot Aug 29, 2012

This pull request fails (merged cc8fc22 into f5818a5).

This pull request fails (merged cc8fc22 into f5818a5).

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

This comment has been minimized.

Show comment
Hide comment
@travisbot

travisbot Aug 29, 2012

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

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

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

This comment has been minimized.

Show comment
Hide comment
@travisbot

travisbot Aug 29, 2012

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

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

@travisbot

This comment has been minimized.

Show comment
Hide comment
@travisbot

travisbot Aug 29, 2012

This pull request fails (merged 0231358 into f5818a5).

This pull request fails (merged 0231358 into f5818a5).

@travisbot

This comment has been minimized.

Show comment
Hide comment
@travisbot

travisbot Aug 29, 2012

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

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

@drewfish

This comment has been minimized.

Show comment
Hide comment
@drewfish

drewfish Aug 29, 2012

Contributor

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

Contributor

drewfish commented Aug 29, 2012

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

@drewfish

This comment has been minimized.

Show comment
Hide comment
@drewfish

drewfish Aug 29, 2012

Contributor

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

Contributor

drewfish commented Aug 29, 2012

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

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.
@caridy

This comment has been minimized.

Show comment
Hide comment
@caridy

caridy Aug 29, 2012

Collaborator

@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.

Collaborator

caridy commented Aug 29, 2012

@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 comment has been minimized.

Show comment
Hide comment
@travisbot

travisbot Aug 29, 2012

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

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

@travisbot

This comment has been minimized.

Show comment
Hide comment
@travisbot

travisbot Aug 29, 2012

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

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

@travisbot

This comment has been minimized.

Show comment
Hide comment
@travisbot

travisbot Aug 29, 2012

This pull request fails (merged b37483c into f5818a5).

This pull request fails (merged b37483c into f5818a5).

@travisbot

This comment has been minimized.

Show comment
Hide comment
@travisbot

travisbot Aug 29, 2012

This pull request fails (merged b37483c into f5818a5).

This pull request fails (merged b37483c into f5818a5).

@caridy

This comment has been minimized.

Show comment
Hide comment
@caridy

caridy Aug 29, 2012

Collaborator

@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?
Collaborator

caridy commented Aug 29, 2012

@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

This comment has been minimized.

Show comment
Hide comment
@mridgway

mridgway Aug 30, 2012

Collaborator

+1, looks good.

Collaborator

mridgway commented Aug 30, 2012

+1, looks good.

caridy added a commit that referenced this pull request Aug 30, 2012

Merge pull request #451 from caridy/yaf-develop
mojito yaf-develop to use the bleeding version of yui to keep them in sync

@caridy caridy merged commit a87298c into YahooArchive:yaf-develop Aug 30, 2012

@drewfish

This comment has been minimized.

Show comment
Hide comment
@drewfish

drewfish Sep 4, 2012

Contributor

I got this error:

shifter --config build-npm.json
make: shifter: No such file or directory
make: *** [npm] Error 1
Contributor

drewfish commented Sep 4, 2012

I got this error:

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

This comment has been minimized.

Show comment
Hide comment
@drewfish

drewfish Sep 4, 2012

Contributor

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

Contributor

drewfish commented Sep 4, 2012

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

@caridy

This comment has been minimized.

Show comment
Hide comment
@caridy

caridy Sep 4, 2012

Collaborator

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

Collaborator

caridy commented Sep 4, 2012

@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