search

tari-project / universe

Other
11 stars 24 forks source link

Minotari node requires VS redist #132

Closed ghost closed 2 days ago

ghost commented 3 months ago

Describe the bug After using the workaround to the bug here: https://github.com/tari-project/universe/issues/131 , I got the Tari Universe app to install but running it was blocked by an error. It was never able to start.

To Reproduce Steps to reproduce the behavior: See the workaround for issue https://github.com/tari-project/universe/issues/131 . I had to use the workaround to get to the point where Tari Universe was installed and trying to Complete is own download prior to mining.

I got this error after the app opened and tried to start its initialization: Screenshot 2024-08-23 at 5 22 40 PM

Desktop (please complete the following information):

leet4tari commented 3 months ago

@Cultmojo if you download and install the Microsoft Visual C++ Redistributable from https://aka.ms/vs/17/release/vc_redist.x64.exe and restart the app, does it solve the problem?

ghost commented 3 months ago

@leet4tari That does solve the install issue.

leet4tari commented 3 months ago

After trying to bundle the Microsoft Visual C++ Redistributable and running into a number of install problems, I think it might be easier and possible better, to do an register check if the VC redistributable is installed, and if not, open a dialog/popup that will have the link to download and install, instead of failing.

Key="SOFTWARE\Microsoft\VisualStudio\14.0\VC\Runtimes\X64"
Name="Installed"
Value="1"
Screenshot 2024-08-29 at 10 57 28 Screenshot 2024-08-29 at 10 58 40
uforiaio commented 2 months ago

If the installer can query wmic to see if vc++ redist is installed or not, it would make it easier to validate the dependencies. If the vc++ versions are added to experimental "versions" it may be helpful. If it is not detected, have an info dialogue to download them via a link within Tari.

image

wmic product where "Name like '%%Visual C++%%'" get Name, Version

Microsoft Visual C++ 2022 X86 Minimum Runtime - 14.40.33810 14.40.33810 Microsoft Visual C++ 2022 X86 Additional Runtime - 14.40.33810 14.40.33810 Microsoft Visual C++ 2022 X64 Minimum Runtime - 14.40.33810 14.40.33810 Microsoft Visual C++ 2022 X64 Additional Runtime - 14.40.33810 14.40.33810

leet4tari commented 1 month ago

Some corner cases covered #750 , still a little bit of work to complete.

brianp commented 2 days ago

Closing as I believe this issue is mostly solved by the required dependencies modal for windows. Feel free to comment or re-open if issues remain.

uforiaio commented 2 days ago

It was early October, it's been fixed from what I experienced and what the common questions/complaints were on TG. Thanks!

On Tue, Nov 26, 2024, 7:52 AM Brian Pearce @.***> wrote:

Closing as I believe this issue is mostly solved by the required dependencies modal for windows. Feel free to comment or re-open if issues remain.

— Reply to this email directly, view it on GitHub https://github.com/tari-project/universe/issues/132#issuecomment-2500861769, or unsubscribe https://github.com/notifications/unsubscribe-auth/BLQZFLVQSZOJGADBSXETZDL2CR4LRAVCNFSM6AAAAABNA4WDP2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBQHA3DCNZWHE . You are receiving this because you commented.Message ID: @.***>