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

script: Setup Improvements #1895

Merged
merged 4 commits into from
Sep 29, 2020

Conversation

nathanielcwm
Copy link
Contributor

@nathanielcwm nathanielcwm commented Sep 23, 2020

Main purpose of this was to make the setup read the existing install location from the registry.

I also resized some of the bitmaps which makes them look better for example:

5.0.1 This PR
image image1

@nathanielcwm
Copy link
Contributor Author

Not entirely sure how to make the installer. With makensis it doesn't seem to create it properly.

@jamescowens
Copy link
Member

jamescowens commented Sep 23, 2020 via email

@nathanielcwm
Copy link
Contributor Author

I believe it doesn't work for the 32 bit setup.

@jamescowens
Copy link
Member

I will check today.

also read registry for 32 bit installer
@nathanielcwm
Copy link
Contributor Author

Looks like b852566 broke some stuff that didn't break on my machine lol

@nathanielcwm nathanielcwm changed the title script: attempt to fix #1893 script: Setup Improvements Sep 26, 2020
@nathanielcwm nathanielcwm marked this pull request as ready for review September 26, 2020 15:48
@jamescowens
Copy link
Member

I am testing this now.

Copy link
Member

@jamescowens jamescowens left a comment

Choose a reason for hiding this comment

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

Tested on both 64-bit and 32-bit Windows and works well.

@jamescowens jamescowens merged commit 93cb040 into gridcoin-community:development Sep 29, 2020
@nathanielcwm nathanielcwm deleted the patch-1 branch October 5, 2020 15:51
jamescowens added a commit that referenced this pull request Oct 8, 2020
…cyrossignol)

 - refactor: port chainparams #1878 (@div72)
 - gui: Update default font to Inter-Regular and console font to Inconsolata (@opsinphark, @jamescowens)
 - gui: Add "review beacon verification" button to wizard summary page #1912 (@cyrossignol)
 - rpc, wallet: Implement liststakes #1909 (@jamescowens)
 - rpc: Add "getlaststake" RPC function #1913 (@cyrossignol)
 - gui: Install bold variant of Inter font #1914 (@cyrossignol)

 - refactor: Consolidate Gridcoin-specific code #1894 (@cyrossignol)
 - script: Setup improvements #1895 (@nathanielcwm)
 - gui: Diagnostics refresh #1899 (@jamescowens)
 - superblock: Optimize superblock size calculation #1906 (@cyrossignol)
 - gui: Adjust stylesheets and scale icons to improve HiDPI side toolbar display #1911 (@jamescowens)
 - doc: Tell user to disable win32 application support in WSL (for building) #1917 (@nathanielcwm)
 - rpc: Revise and expand help for beaconconvergence rpc call #1918 (@jamescowens)
 - scheduler: Increase default update check interval to 5 days #1920 (@cyrossignol)
 - gui: Prevent multiple dialogs from being open at the same time #1922 (@scribblemaniac)

 - refactor: Clean up remaining legacy timer code #1892 (@cyrossignol)

 - build: Add --without-brotli option to curl.mk #1902 (@G_UK)
 - test: Remove fs_tests... file after the fs test #1903 (@div72)
 - util, gui: Fix shutdown segfault and repair broken overview page staking status #1901 (@jamescowens)
 - scraper: Fix order of destruction for global scraper objects #1904 (@cyrossignol)
 - scraper: Fix global object destruction order for MacOS #1905 (@cyrossignol)
 - util: Decouple out-of-sync state from block acceptance #1921 (@cyrossignol)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Windows] Automatically set the install path in setup to where the wallet is currently installed.
2 participants