-
Notifications
You must be signed in to change notification settings - Fork 78
[GITHUB] Update build-toolchain to use public VC6 from itsmattkc repo #931
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
base: main
Are you sure you want to change the base?
Conversation
shell: pwsh | ||
run: | | ||
Write-Host "Downloading VC6 Portable Installation" -ForegroundColor Cyan | ||
aws s3 cp s3://github-ci/VS6_VisualStudio6.7z VS6_VisualStudio6.7z --endpoint-url $env:AWS_ENDPOINT_URL | ||
Invoke-WebRequest -Uri https://github.com/itsmattkc/MSVC600/archive/refs/heads/master.zip -OutFile VS6_VisualStudio6.zip |
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.
If itsmattkc takes the repo private, then this breaks. Are we allowed to fork this repository or is this illegal?
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.
There are already several forks of this repository. If itsmattkc decides to make the repository private or remove it, we can simply switch to another existing fork. Alternatively, we could revert to using the AWS backup if necessary.
But realistically, this repository has been public for almost five years, and we are talking about end-of-life software. I find it unlikely that it will be taken down at this point.
Regarding the legality of keeping a fork, I am not a lawyer, but as far as I know, simply maintaining a fork of freely distributed executables—without modifying or hacking the binaries—should not be illegal. However, if there are any concerns, we should seek legal advice to be certain.
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.
If allowed, Superhackers should make their own fork
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.
I guess allowed would be based on whatever Microsoft says no? As it is their software, don't know if you still need a license for VS6. If you don't then we could just create our own repo and not even fork this one.
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.
it would probably be better to have our own repo, that ways it's easier for people to get hold of a copy for local use as well and it keeps dependencies within the project as much as possible.
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.
We can do that if someone can find out if that is ok to do.
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.
In legal terms what’s the difference between get vs6 from an s3 bucket and get it from any public repo?
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.
What i can read me to, it looks like we would need a license and run the script locally. I believe the actions run on @tintinhamans's home server? So that should have the license?
@OmniBlade what are your thoughts on this? |
It does make it easier to do builds in our own forks, I'd be inclined to go with it and we either revert/fix it if the repo goes down and hope we don't need VC6 by the time that happens. Github is owned by Microsoft so while someone has directed their infrastructure to create the repo, its being served from Microsoft infrastructure to be run on Microsoft infrastructure. At worst they would just stop it if they took issue with it and its not really any different from what we are doing now? Probably legally more in the clear that having Amazons infrastructure provide copies, but still clear as mud. |
This pull request updates the build workflow to download the VC6 toolchain from a public repository.
Key changes:
This change makes the build process more secure and easier to reproduce for everyone.