nikitakit / ucpp

[UNMAINTAINED] Universal FRC C++ Development Tools -- for Linux and more
24 stars 15 forks source link

2nd Mid Season Update #15

Closed anidev closed 11 years ago

anidev commented 11 years ago

The install_wpilib function sorts the list of WPILib links in the wrong order (sort -r instead of just sort); this changes that and thus it can correctly find and download the 2nd mid season update.

Alternatively, instead of changing sort -r to sort, the tail -n 1 could have been changed to head -n 1.

nikitakit commented 11 years ago

Did you intend to submit all of the intermediate commits? (Fixed issue..Up to date with upstream)? Otherwise I can just cherry-pick 0e257 and mark this as closed.

anidev commented 11 years ago

Sorry about that, you can cherry pick the last one, it's just a small change. All the other commits were for making it work on Windows, but by the time I finished those your version already had similar changes for working on Windows. On Feb 10, 2013 7:18 PM, "Nikita Kitaev" notifications@github.com wrote:

Did you intend to submit all of the intermediate commits? (Fixed issue..Up to date with upstream)? Otherwise I can just cherry-pick 0e257 and mark this as closed.

— Reply to this email directly or view it on GitHubhttps://github.com/nikitakit/ucpp/pull/15#issuecomment-13364703..

nikitakit commented 11 years ago

Cherry-picked. Thanks!