Skip to content

[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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fbraz3
Copy link

@fbraz3 fbraz3 commented May 24, 2025

This pull request updates the build workflow to download the VC6 toolchain from a public repository.

Key changes:

  • Uses the public itsmattkc/MSVC600 repository to fetch the VC6 portable toolchain.
  • Removes the need to maintain or use any secrets for toolchain downloads in CI builds.
  • Simplifies maintenance and improves transparency for all contributors.

This change makes the build process more secure and easier to reproduce for everyone.

@xezon xezon requested a review from tintinhamans May 24, 2025 19:23
@xezon xezon added Minor Severity: Minor < Major < Critical < Blocker Build Anything related to building, compiling labels May 24, 2025
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
Copy link

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?

Copy link
Author

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.

https://github.com/itsmattkc/MSVC600/forks?include=active%2Cinactive%2Cnetwork&page=1&period=&sort_by=stargazer_counts

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.

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

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.

Copy link

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.

Copy link

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.

Copy link
Author

@fbraz3 fbraz3 May 31, 2025

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?

Copy link

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?

@xezon xezon requested a review from OmniBlade June 23, 2025 20:37
@xezon
Copy link

xezon commented Jun 23, 2025

@OmniBlade what are your thoughts on this?

@OmniBlade
Copy link

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Anything related to building, compiling Minor Severity: Minor < Major < Critical < Blocker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants