-
Notifications
You must be signed in to change notification settings - Fork 64
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
Version 6.7.0 #314
Merged
Merged
Version 6.7.0 #314
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* update name from simplewallet to concealwallet * update class to conceal_wallet * create main.cpp strictly for main * rename DepositHelper to ClientHelper * add functions to clienthelper * move transfer command into its own file * remove calling wallet in main, just show flags not full wallet commands * create Const header file * add save_wallet function to client helper to save code duplication *
--> Removed redundant dependencies 'gcc' and 'g++' are included in the 'build-essential' package, so also listing them to be installed is redundant. --> Changed 'python-dev' to 'python3-dev' On Ubuntu and Debian Based Systems, 'python' refers to Python 2, and the package that is actually needed is named 'python3-dev' --> Changed 'Ubuntu' to 'Linux' Changed the header of 'Building/Ubuntu' to 'Building/Linux', as the commands there are pretty much identical across all distributions. --> 'apt-get' is now 'apt' In the dependencies, removed '-get' from 'apt-get install', as it's not required, 'apt' works, and most people are just going to use plain 'apt'
Included the 'pthread.h' file, this was causing the build to fail on Arch Linux.
README: Updated Dependencies & Other Stuff
Mimic shell behaviour by expanding the tilde to either $HOME (Linux, OS X, BSD) or %USERPROFILE% (Windows) - most people use "~" instead of typing the entire path to their home directory.
Platform/Linux/System/Dispatcher: 'pthread' Header
Instead of using "#if defined", we just use Boost to see if the platform is Windows.
Script that will get the amount of system memory, and then create a swap file with the size required to get a combined total of 13GB - the recommended stated in the README Works on both Raspberry Pi and standard Linux distributions
A script that automates "deployment" - only requires the script and 'git'. The script will install any dependencies, and can be used to automate it on many systems, such as multiple Raspberry Pi's at a time. Also provides basic functionality to 'concealwallet'
Quality of Life Improvements
Development
Client Wallet code reorganisation
* move balance function to `client_helper` so then we can call `m_chelper` in main.cpp to show all balances in wallet * `check_address` command to verify $CCX addresses
* @ TurdBird in Discord said: "can someone change the wallet time lock from SPENT, to WITHDRAWN? i see this all the time, and it gives ppl the wrong idea. thanks!"
`concealwallet` Improvements
Dont pass hex through logger
Add Ubuntu 22.04
Fix WalletUserTransactionsCache reset
Update currency and fee calculation
Update dependencies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.