Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes a conflict with the Windows file system. #2286

Merged
merged 2 commits into from
Sep 22, 2016
Merged

Fixes a conflict with the Windows file system. #2286

merged 2 commits into from
Sep 22, 2016

Conversation

KevinBrogan
Copy link
Contributor

Fixing a bug that makes it impossible to checkout source code on a windows file system.

aux is a reserved keyword

@knewmanTE
Copy link
Contributor

Were you able to test this locally with the changes and get it to run successfully? When running locally on a fork of your branch, I am getting the following error:

Setup fasthttp: can't load package: package templates: no buildable Go source files in 
/home/techempower/FrameworkBenchmarks/frameworks/Go/fasthttp/src/templates

Is it possible that the aux file is a requirement for fasthttp to run properly?

@KevinBrogan
Copy link
Contributor Author

damn it, I typoed the renamed file location. missed the templates directory.

@KevinBrogan
Copy link
Contributor Author

I don't have a working copy on my machine yet. Had to get the source on it first, which is impossible until this rename is completed.

@knewmanTE
Copy link
Contributor

@KevinBrogan no worries! It looks like it's doing okay on Travis, so as long as it passes on Travis we should be okay to merge this in.

@knewmanTE
Copy link
Contributor

Looks like that fix did the trick. Merging in!

@knewmanTE knewmanTE merged commit 0b2e505 into TechEmpower:master Sep 22, 2016
@KevinBrogan
Copy link
Contributor Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants