[Dev] Feathercoin 0.9.6.2 * Maintenance fix, build & upgrade issues notes.

Wellenreiter

@lizhi said in [Dev] Feathercoin 0.9.6.1 * Bug fix & issues notes for 0.11:

src\clientversion.h

That definition is used only, if there is no bitcoin-config.h, which in turn is created/updated by the autoconfig scripr, so the definitions in src\clientversion.h are never used.

AcidD

@Wellenreiter said in [Dev] Feathercoin 0.9.6.1 * Bug fix & issues notes for 0.11:

@lizhi said in [Dev] Feathercoin 0.9.6.1 * Bug fix & issues notes for 0.11:

src\clientversion.h

That definition is used only, if there is no bitcoin-config.h, which in turn is created/updated by the autoconfig scripr, so the definitions in src\clientversion.h are never used.

I think Lizhi might be right. Also we have a slight problem.
https://github.com/FeatherCoin/Feathercoin/blob/0.9.6/src/feathercoind-res.rc

#include <windows.h>             // needed for VERSIONINFO
#include "clientversion.h"       // holds the needed client version information

https://github.com/FeatherCoin/Feathercoin/blob/0.9.6/src/clientversion.h

// These need to be macros, as version.cpp's and bitcoin-qt.rc's voodoo requires it
#define CLIENT_VERSION_MAJOR       0
#define CLIENT_VERSION_MINOR       9
#define CLIENT_VERSION_REVISION    3
#define CLIENT_VERSION_BUILD       99

// Set to true for release, false for prerelease or test build
#define CLIENT_VERSION_IS_RELEASE  false

// Copyright year (2009-this)
// Todo: update this when changing our copyright comments in the source
#define COPYRIGHT_YEAR 2016

and my release Feathercoin-qt.exe
Wrong Version

Look at the version, it’s wrong. and matches what Lizhi was saying.

AcidD

confirmed, changing the details in
https://github.com/FeatherCoin/Feathercoin/blob/0.9.6/src/clientversion.h

Now the right version and copyright date appear. Good find @lizhi !!

alt text

Wellenreiter

@aciddude

also read lines 1 -5 of that file:

#ifndef CLIENTVERSION_H
#define CLIENTVERSION_H

#if defined(HAVE_CONFIG_H)
//

// These need to be macros, as version.cpp's and bitcoin-qt.rc's voodoo requires it
#define CLIENT_VERSION_MAJOR       0
#define CLIENT_VERSION_MINOR       9
#define CLIENT_VERSION_REVISION    3
#define CLIENT_VERSION_BUILD       99

There are some lines missing:

#if defined(HAVE_CONFIG_H)
#include "bitcoin-config.h" 
#else

.....

is the right syntax

AcidD

@Wellenreiter said in [Dev] Feathercoin 0.9.6.1 * Bug fix & issues notes for 0.11:

There are some lines missing:
#if defined(HAVE_CONFIG_H)
#include "bitcoin-config.h" 
#else

.....
is the right syntax

will it need to be changed in the .rc files as well?

i think there’s three for feathercoind, cli, qt.exe

https://github.com/FeatherCoin/Feathercoin/blob/0.9.6/src/feathercoind-res.rc

#include <windows.h>             // needed for VERSIONINFO
#include "clientversion.h"       // holds the needed client version information

#define VER_PRODUCTVERSION     CLIENT_VERSION_MAJOR,CLIENT_VERSION_MINOR,CLIENT_VERSION_REVISION,CLIENT_VERSION_BUILD
#define VER_PRODUCTVERSION_STR STRINGIZE(CLIENT_VERSION_MAJOR) "." STRINGIZE(CLIENT_VERSION_MINOR) "." STRINGIZE(CLIENT_VERSION_REVISION) "." STRINGIZE(CLIENT_VERSION_BUILD)
#define VER_FILEVERSION        VER_PRODUCTVERSION
#define VER_FILEVERSION_STR    VER_PRODUCTVERSION_STR
#define COPYRIGHT_STR          "Feathercoin developers 2013-" STRINGIZE(COPYRIGHT_YEAR) ", The Bitcoin developers 2009-" STRINGIZE(COPYRIGHT_YEAR)

Wellenreiter

according to the piece of code you copied it should be ok.
If clientversion.h is patched in the qt-directory, it uses the values defined by the confiugure script and we only have one place to change the version numbers

AcidD

@Wellenreiter and I have been working on the sendAlert call which is used to send alerts across the Feathercoin Network.

I Believe we’ve solved this. Wellenreiter was able to send an alert over the TestNet and he received it on his node.

There was an issue with one of the blocks on the testnet so that’s just being fixed…one that’s done I should hopefully be able to connect to the TestNet Node Wellenreiter set up and receive his test message.

Then we can apply the same patch to prod. :)

wrapper

Just a note to double check FTC code for this issue LTC #293 - List of FTC custom settings?

https://github.com/litecoin-project/litecoin/blob/master/src/main.cpp#L4161

main.cpp nCheckDepth #293

In the code it says
nCheckDepth = 1000000000; // suffices until the year 19000

But Litecoin is 4x faster then Bitcoin.
The note should say “until the year 4750”

or should the nCheckDepth be changed into 4000000000 s

https://github.com/litecoin-project/litecoin/issues/293#issuecomment-285490529

wrapper

Release of New Feathercoin Wallet version 0.9.6.1

@Wellenreiter has started to lock down the code for the release of version 0.9.6.1.

Reviewed outstanding document updates and policy.

The FTC software settings, naming conventions and software development fork procedure showed that a lot of work is need to up keep Feathercoin name interface changes. Some changes such as comments, or where Feathercoin operates the same as Bitcoin core are unnecessary. How to handle files in the future for cherry pick or updates from head were discussed.

 /doc
 /doc/release-notes

ToDo :

Include Feathercoin 0.9.6.1 release notes
Create Beta test builds / Window / MAC etc
~~Review - update Wallet guide, MultiSig is an important, but complex update, needs better screen shots / help text.~~
https://github.com/wrapperband/FeathercoinWalletGuide#multisig-button
* Review / consider - Include 5, 7 name Multisignature addresses i.e do further update

* Multisig: A beginner’s guide
https://99bitcoins.com/multisig-a-beginners-guide/

Note : 0.9.6.1 will be a long term version, compatible as a wallet with 0.11. The most usefull cases for Multisig, once you have got over the learning curve, is for more names, and say 3 out of 5, so there is redundancy for fund handling or other open source management / DAO criteria.
~~Review wither to including Feathercoin Wallet Guide in release~~

Wellenreiter

@wrapper said in [Dev] Feathercoin 0.9.6.1 * Bug fix & issues notes for 0.11:

Review wither to including Feathercoin Wallet Guide in release

I’d prefer to just include a link to the online guilde in the wallet-GUI. it gives more flexibility and enables us to update/enhance the guide without releasing new versions of the Wallet

wrapper

##- Please type your reply above this line -##
Email to shapeshift

Jun 28, 2:54 AM MDT

Feathercoin has spent considerable bounties to include shapeshift, for over one year. We understood FTC was to be reinstated after you had some problems.
The next release is to be finalised within 5 days and it will then be about 6 months before it can be reinstated.

Wrapper Admin FTC forum/dev

Hello from the ShapeShift.io Support Team!

Urgent Note: https://www.facebook.com/notes/shapeshiftio/note-on-support-requests-for-the-weekend-of-june-16-2017/1321495071261253/

Your request (64102) has been received and is being reviewed by our team. Please, only send in one ticket per issue - this is important to speed up your resolution time.

To ensure you get the best support possible please be sure you have included the ShapeShift deposit address and the TXID (or Hash) of your deposit(s).
ShapeShift Support is available 7 days a week from 11am-7pm EST. Support is available, but limited outside of those hours. Your request will be resolved within 3-4 days.

AmDD

@wrapper said in [Dev] Feathercoin 0.9.6.1 * Bug fix & issues notes for 0.11:

##- Please type your reply above this line -##
Email to shapeshift

Jun 28, 2:54 AM MDT

Feathercoin has spent considerable bounties to include shapeshift, for over one year. We understood FTC was to be reinstated after you had some problems.
The next release is to be finalised within 5 days and it will then be about 6 months before it can be reinstated.

Wrapper Admin FTC forum/dev

Hello from the ShapeShift.io Support Team!

Urgent Note: https://www.facebook.com/notes/shapeshiftio/note-on-support-requests-for-the-weekend-of-june-16-2017/1321495071261253/

Your request (64102) has been received and is being reviewed by our team. Please, only send in one ticket per issue - this is important to speed up your resolution time.

To ensure you get the best support possible please be sure you have included the ShapeShift deposit address and the TXID (or Hash) of your deposit(s).
ShapeShift Support is available 7 days a week from 11am-7pm EST. Support is available, but limited outside of those hours. Your request will be resolved within 3-4 days.

I also sent SHapeShift an email a few weeks ago about adding Feathercoin back. I didnt get a direct response yet.

wrapper

Incorrect setting of minimum fee #181
https://github.com/FeatherCoin/Feathercoin/issues/181

In at least one place (in main.cpp) - FTC custom settings are required because of block time and reward scaling of Bitcoin parameters.

Further, whilst reviewing an emergency change, comparison should be made with the Litecoin setting, to ensure the further scaling of coin reward is reflected in the minimum fees.

This is very important as it is currently mode of failure of Bitcoin and we will be stuck with high fees till the next release.

v0.8 https://github.com/FeatherCoin/Feathercoin/blob/master-0.8/src/main.cpp
/** Fees smaller than this (in satoshi) are considered zero fee (for transaction creation) /
int64 CTransaction::nMinTxFee = 2000000;
/* Fees smaller than this (in satoshi) are considered zero fee (for relaying) */
int64 CTransaction::nMinRelayTxFee = 2000000;

v0.9 https://github.com/FeatherCoin/Feathercoin/blob/0.9.6/src/main.cpp
/** Fees smaller than this (in satoshi) are considered zero fee (for transaction creation) /
int64_t CTransaction::nMinTxFee = 1000000; // Override with -mintxfee
/* Fees smaller than this (in satoshi) are considered zero fee (for relaying and mining) */
int64_t CTransaction::nMinRelayTxFee = 1000000;

In 0.8 but not 0.9 main.cpp

// Feathercoin
// To limit dust spam, add nBaseFee for each output less than DUST_SOFT_LIMIT
BOOST_FOREACH(const CTxOut& txout, vout)
if (txout.nValue < DUST_SOFT_LIMIT)
nMinFee += nBaseFee;

// Raise the price as the block approaches full

if (nBlockSize != 1 && nNewBlockSize >= MAX_BLOCK_SIZE_GEN/2)

wrapper

Notes : Minimum fee issue / review

Review src/util.h 0.9.x

 static const int64_t COIN = 100000000;

https://github.com/FeatherCoin/Feathercoin/blob/0.9.6/src/main.h

nBaseFee is the value you configure. In 0.9 default is 1000000

int64_t nMinFee = (1 + (int64_t)nBytes / 1000) * nBaseFee

Review main.cpp main.h 0.8.x

https://github.com/FeatherCoin/Feathercoin/blob/master-0.8/src/main.h

  /** Dust Hard Limit, ignored as wallet inputs (mininput default) */
 static const int64 DUST_HARD_LIMIT = 1000; // 0.00001 FTC mininput

 /** Dust Soft Limit, allowed with additional fee per output */
static const int64 DUST_SOFT_LIMIT = 100000; // 0.001 FTC

Litcoin related issues and updates (over Bitcoin)

Minimum Transaction Fee review

https://github.com/litecoin-project/litecoin/commit/3ce57329263cd347ada29172947b23dfc787e898
https://github.com/litecoin-project/litecoin/commit/378ff026d0fc5205b2e15d2c2b05534d013ef5c5
https://github.com/litecoin-project/litecoin/commits/0.8?after=9cffb23c2d48d47bb67df78ea1164738b41e8c9d+69
https://github.com/litecoin-project/litecoin/commit/f80351c3215014935c5836f7d90cc21af5fc0cdf
https://github.com/litecoin-project/litecoin/commit/72b4e39c092b03abc5495b908f671b2ca8fe03cd
https://github.com/litecoin-project/litecoin/commit/def3c9e53b20815c8eef0aef7c9315cdd4a889c6

https://github.com/litecoin-project/litecoin/commit/70313dd1e7518d410dac30db4906aac4846520a7

wrapper

https://litecoin.info/Transaction_fees

limitfreerelay 15,000 (Bytes)

Limit of free transactions the client will relay to other nodes per minute

mintxfee 0.001 (LTC)

Minimum fee per KB for non free transactions (creating)

minrelaytxfee 0.001 (LTC)

Minimum fee per KB for non free transactions (relaying)

mininput 0.00001 (LTC)

Wallet ignores smaller inputs

DUST_THRESHOLD 0.001 (LTC)

Minimum output to avoid an additional mintxfee charge per smaller output

blockmaxsize 1,000,000 (Bytes)

Maximum block size

blockminsize 0 (Bytes)

Minimum block size

blockprioritysize 17,000 (Bytes)

Size of the block reserved for highest-priority transactions (regardless the fee)

maxhighprioritytxsize 12,000 (Bytes)

Maximum size of a transaction to be eligible for highest-priority

wrapper

Re : How should the FTC fees been set?

LTC was 4 times Faster than BTC, so fee level need to be 1/4 BTC to be proportional.

FTC had 4 times the amount of coins than LTC, so FTC fee need to be 1/16 of BTC value, or 1/4 LTC value.

It looks like LTC settled on 1/10 for ease of calculation and make fees lower than BTC.

In 0.6.x and 0.7.x FTC settled on 1/20 which make our fees slightly less than LTC, and is easier to code.

Our dust level needs to be reduced by same level (which has been done?)

wrapper

Re : neoscrypt version. I was checking neoscrypt version as @Ghostlander had mentioned somewhere it may be out of date. / where to cherry pick from list

add Implemented ACP and neoscrypt

https://github.com/FeatherCoin/Feathercoin/commit/c8ca4c906144910c62acd34e23a4ed624c0fbf7d

https://github.com/ghostlander/nsgminer

ToDo :

add neoscrypt issue to 0.11 done
Create the where to cherry pick from list

wrapper

This issue can be closed : https://github.com/FeatherCoin/Feathercoin/issues/102

wrapper

Testing the 0.9.6.2 updated GNU/ Linux Ubuntu example instructions, ready to pull into 0.11.2

Failed to build with error :

/usr/include/x86_64-linux-gnu/qt5/QtCore/qglobal.h:1087:4:
error: #error "You must build your code with position independent code if Qt was built with -reduce-relocations. " “Compile your code with -fPIC (-fPIE is not enough).”

# error "You must build your code with position independent code if Qt was built with -reduce-relocations. "\

Tried : Adding -fPIC to CFAGS in makefile

git checkout 0.11.2

export CXXFLAGS="$CXXFLAGS -fPIC"
make clean
./autogen.sh
autoupdate

./configure --with-gui=qt5 --enable-tests=no  --with-incompatible-bdb --enable-upnp-default --with-qrcode=yes  --disable-hardening --enable-shared
    
make

Gives error :
CXX qt/qt_libbitcoinqt_a-addressbookpage.o
qt/addressbookpage.cpp:32:26: fatal error: qrcodedialog.h: No such file or directory
#include “qrcodedialog.h”

Someone else fixing it :
https://github.com/ctrlaltca/Cockatrice/commit/9cfb09cd5b25a01fb142d425ba70b77b89f65a09
https://github.com/Civil/cantata/commit/9a6d5464376d48cc4c4e37c37459bc8b8aa74ba1

wrapper

After adding missing files / updating configure.ac :

src/qt/qrcodedialog.h and qrcodedialog.cpp

Compile fails at :

In file included from qt/addressbookpage.cpp:32:0:
qt/qrcodedialog.h:22:9: error: expected ‘:’ before ‘slots’
private slots:

https://stackoverflow.com/questions/20185423/qt-cmake-missing-before-identifier-slots

Tried adding missing file : feathercoin.qrc

Created list of files Not in version 0.11.2 but in version 0.9.6.2 in ~/projects/Feathercoin :

diff -r ~/projects/Feathercoin ~/Feathercoin | grep ~/projects/Feathercoin > difference2.txt