-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Create Windows build scripts and update instructions #1721
Create Windows build scripts and update instructions #1721
Conversation
lokitoth
commented
Jan 22, 2019
•
edited
Loading
edited
- Creates restore and build scripts with support for custom toolchain integration
- Updates instructions for building on Windows (includes instructions for VS2017)
- Changes AppVeyor to use new build scripts
@@ -0,0 +1,41 @@ | |||
## Windows specific info | |||
|
|||
This document details differences in pre-requisites and building instructions under Windows. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this in be added in the wiki rather than here? https://github.com/VowpalWabbit/vowpal_wabbit/wiki/Dependencies
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure; I was just trying to be parallel to README.md. Keep in both, like for Linux?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keeping it in both seems good. More ideally, we'd sync the wiki's version to the source code version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So the readme is a bit out of date at the moment, I've done work to try and make sure the wiki is up to date. I've utilized several pages in the wiki to make it a bit simpler. I was planning on making the front page readme a bit simpler and just using it to explain VW's mission and why use it, then link into the wiki for instructions
Closes #1691 |
Having the information in a single location is conceptually simple, but
maybe the other direction is better? That way it's available when you
download the code and need to operate offline?
…-John
On Wed, Jan 23, 2019 at 8:57 AM Jack Gerrits ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In README.Windows.md
<#1721 (comment)>
:
> @@ -0,0 +1,41 @@
+## Windows specific info
+
+This document details differences in pre-requisites and building instructions under Windows.
So the readme is a bit out of date at the moment, I've done work to try
and make sure the wiki is up to date. I've utilized several pages in the
wiki to make it a bit simpler. I was planning on making the front page
readme a bit simpler and just using it to explain VW's mission and why use
it, then link into the wiki for instructions
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1721 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAE25gL73AYks4KS1BaQBQawEKbh-TMcks5vGGo-gaJpZM4aNTQ_>
.
|
Totally agree, plus there's the added benefit of pull requests for changes when they are in files. So we can either migrate the existing wiki docs to a folder in the vowpalwabbit repo, or we can utilize the docs repo I recently created to host the rl_client docs. (See here https://vowpalwabbit.github.io/docs/, but this removes the benefit of automatically having it available locally for offline use) Or we can wait until the website is ready and think about how the pages fit in there? The wiki is useful for the sidebar aswell |
* Create windows build scripts and clean up readme * Update AppVeyor to use new scripts * Fix bad set command in appveyor.yml