<p dir="ltr">On Fri, Nov 8, 2013 at 10:32 AM, Greg Hendershott <<a href="mailto:greghendershott@gmail.com">greghendershott@gmail.com</a>> wrote:<br>
> I gave a bad example of a GitHub package URL there. It needs a branch<br>
> on the end. Instead of:<br>
><br>
> pkg install github://<a href="http://github.com/greghendershott/markdown">github.com/greghendershott/markdown</a><br>
><br>
> should be:<br>
><br>
> pkg install github://<a href="http://github.com/greghendershott/markdown/master">github.com/greghendershott/markdown/master</a><br>
><br>
> Although that's N/A to my question, I don't want to mislead anyone<br>
> with a bad example.</p>
<p dir="ltr">Sigh. This is also misleading. This would be fine if Travis CI were always invoked to test the master branch. But not necessarily. It could be invoked to test another, "topic" branch. It could be invoked on a branch for a pull request -- precisely one of the coolest ways Travis CI can help, by testing other people's PRs even if they forgot to, and showing you the result right there in the PR UI on GitHub.</p>
<p dir="ltr">So the correct thing to use here is raco pkg install --link. i.e. Install using whatever source Travis CI `git clone`d earlier in the process.</p>
<p dir="ltr">The only other trick is that --link by itself won't install dependencies. So we need to add `--deps search-auto`. Which we'd want anyway, for the `search-auto`, so that this automated thing will proceed without asking about installing deps.</p>
<p dir="ltr">TL;DR: Adding in Jay's tip re raco test -p means the whole thing would be:</p>
<p dir="ltr">script:<br>
- cd .. # because Travis did a `cd` into our repo dir, and next...<br>
- raco pkg install --deps search-auto --link <pkg><br>
- raco test -x -p <pkg></p>
<p dir="ltr">This is the simplest way to handle a package that depends on other packages. (If you're testing a non-package, you have to use the original way. Of course that might be another reason to make it a package, especially if it has deps.)</p>
<p dir="ltr">I have a blog post about this that I'll update accordingly, as well.</p>