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

feat: include vc in installer #897

Open
wants to merge 72 commits into
base: main
Choose a base branch
from

Conversation

Misieq01
Copy link
Collaborator

@Misieq01 Misieq01 commented Oct 21, 2024

[ Improvment ] => Microsoft Visual C++ Redistributable [ WINDOWS ONLY ]

Extended installer with setup for mentioned package

[ Features ]

  • Created new bundler in => tari-win-bundler directory which combines Tari universe installer.msi and vc_redist.exe
  • Extended CI workflow release with setup required for building mentioned bundle and attaching it to release

[ What is missing ]

  • Signing bundle in workflow release. Without this part when user start installer its being picked up by Microsoft Defender SmartScreen
  • Styling and theming

Before its merged !!!

Extend - platform: 'windows-2019' args: '--bundles msi' with updater argument, see in relesae workflow on main branch

@Misieq01 Misieq01 requested a review from a team as a code owner October 21, 2024 12:35
@leet4tari
Copy link
Contributor

This is a pretty radical change for a very small number of users, thou it should be addressed some how. Other users might see this as a download tax, that they don't need.

I was thinking we could possible do this build option as a full offline installer, but would make this package option pretty big.

Could also include the tor package, as this has been requested a few times to be bundled. Windows Bundle options for Tauri can be found - https://v1.tauri.app/v1/guides/building/windows/#webview2-installation-options.

Not sure why anybody would use an offline installer as internet is needed for any useful usage of the package, but it's an idea to put all this in one place.

@Misieq01
Copy link
Collaborator Author

Misieq01 commented Oct 23, 2024

This is a pretty radical change for a very small number of users, thou it should be addressed some how. Other users might see this as a download tax, that they don't need.

I was thinking we could possible do this build option as a full offline installer, but would make this package option pretty big.

Could also include the tor package, as this has been requested a few times to be bundled. Windows Bundle options for Tauri can be found - https://v1.tauri.app/v1/guides/building/windows/#webview2-installation-options.

Not sure why anybody would use an offline installer as internet is needed for any useful usage of the package, but it's an idea to put all this in one place.

I don't think its affect very small number of users. It's affects all future windows installation when users does not have vc_redist already on the machine so this number can get big when tari will be released to wider audience.

Also the documentation that you provided does not refer to this problem as its only WebView2 installation configuration ( at least section from the link ), its not like we can use it to include more packages

As for including others packages as tor, gpu miner I think we could do it with tauri options for configuring wix termplate as long as there is no extra installer in place.

If we want to include vc_redist then all work done in this PR is required

@brianp
Copy link
Collaborator

brianp commented Oct 25, 2024

@leet4tari I can look at the changes here but most of it won't mean anything to me. So I'm going to lean on your review if you could officially make one.

And testing... Any chance you can test it?

Copy link
Collaborator

@brianp brianp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMy untrained windows knowledge 👍🏻 🙃

@Misieq01
Copy link
Collaborator Author

@leet4tari I can look at the changes here but most of it won't mean anything to me. So I'm going to lean on your review if you could officially make one.

And testing... Any chance you can test it?

@brianp if you would like to check the installer, the only way now is to download it from releases on my fork https://github.com/Misieq01/universe/releases => Tari.Universe_0.5.22_x64_en-US.exe

@leet4tari
Copy link
Contributor

I think this can be paused for now, but should be in place by release as this will be something that comes up alot. My view on combining two problems into one solution would mean we get to solve many problems with one download. ie: include vc_redist, solve this problem. Include tor, solve that download problem. Include other tools, solve offline single-shot download and install. Down side is huge download, but I only seeing that happen for initial install, else download updates as needed.

@brianp
Copy link
Collaborator

brianp commented Oct 25, 2024

Related to #712

Both will wait until next week for updates, and firewall logic may be moved into the installer logic. In any case we probably need to wait for p2pool to be complete before this will get updated/merger.

@brianp brianp added the on hold Issue is being held for progress later label Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on hold Issue is being held for progress later p-controversial
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants