Feathercoin daemon and wallet production version 0.19.1
Old daemon and wallet version 0.18.3

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


  • Moderators | Tip 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.


  • Moderators | Tip 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


  • Moderators | Tip 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


  • Moderators | Tip 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)   
    

  • Moderators | Tip 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


  • Moderators | Tip 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. 🙂


  • Moderators | Tip 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


  • Moderators | Tip 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


  • Moderators | Tip 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


  • Moderators | Tip 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.


  • Regular Member | Tip 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.


  • Moderators | Tip 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)


  • Moderators | Tip 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


  • Moderators | Tip 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


  • Moderators | Tip 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?)


  • Moderators | Tip 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


  • Moderators | Tip wrapper


  • Moderators | Tip 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


  • Moderators | Tip 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


  • Moderators | Tip wrapper

    Missing / changed files from Feathercoin 0.11 version compared to 0.9.6.2:

    Results of synopsis of missing files / difference show :

    Missing functionality.
    Nameview
    Stealth coins
    Coinector
    Shapeshift

    Missing Icon Upgrade
    Updated Translation files
    Updated auto unit tests

    Detail of the differences - using difference2.txt

    https://github.com/wrapperband/FTC-Wallet-Layout-Examples/blob/master/Develpment/difference2.txt

    Two directories were compared using diff command :

    diff -r ~/projects/Feathercoin ~/Feathercoin  | grep ~/projects/Feathercoin   > difference2.txt
    
    diff -r ~/projects/Feathercoin ~/Feathercoin  | grep ~/projects/Feathercoin | awk '{print $4}'  > difference3.txt
    

    A number of files in these directories :
    Only in : ~/projects/Feathercoin/contrib/
    Only in ~/projects/Feathercoin/doc/
    Only in ~/projects/Feathercoin/qa/
    Only in ~/projects/Feathercoin/share/

    These files missing:
    Only in ~/projects/Feathercoin/src: allocators.cpp
    Only in ~/projects/Feathercoin/src: allocators.h
    Only in ~/projects/Feathercoin/src: auxpow.cpp
    Only in ~/projects/Feathercoin/src: auxpow.h
    Only in ~/projects/Feathercoin/src: base40.h
    Only in ~/projects/Feathercoin/src: bitcoin-config.h

    Only in ~/projects/Feathercoin/src: core.cpp
    Only in ~/projects/Feathercoin/src: core.h
    Only in ~/projects/Feathercoin/src: crypter.cpp
    Only in ~/projects/Feathercoin/src: crypter.h
    Only in ~/projects/Feathercoin/src: db.cpp
    Only in ~/projects/Feathercoin/src: db.h
    Only in ~/projects/Feathercoin/src: feathercoin-cli.cpp
    Only in ~/projects/Feathercoin/src: feathercoin-cli-res.rc
    Only in ~/projects/Feathercoin/src: feathercoind.cpp
    Only in ~/projects/Feathercoin/src: feathercoind-res.rc

    Only in ~/projects/Feathercoin/src/json: json_spirit_reader.cpp
    Only in ~/projects/Feathercoin/src/json: json_spirit_value.cpp
    Only in ~/projects/Feathercoin/src/json: json_spirit_writer.cpp
    Only in ~/projects/Feathercoin/src/json: LICENSE.txt

    Only in ~/projects/Feathercoin/src/leveldb: AUTHORS
    Only in ~/projects/Feathercoin/src/leveldb: NEWS
    Only in ~/projects/Feathercoin/src/leveldb/port: README
    Only in ~/projects/Feathercoin/src/leveldb: README

    Only in ~/projects/Feathercoin/src/obj: .gitignore
    Only in ~/projects/Feathercoin/src: obj-test

    Only in ~/projects/Feathercoin/src/qt: bitcoinstrings.cpp
    Only in ~/projects/Feathercoin/src/qt: coinnectordialog.cpp
    Only in ~/projects/Feathercoin/src/qt: coinnectordialog.h

    Only in ~/projects/Feathercoin/src/qt/forms: aboutdialog.ui
    Only in ~/projects/Feathercoin/src/qt/forms: coinnectordialog.ui
    Only in ~/projects/Feathercoin/src/qt/forms: opennamedialog.ui
    Only in ~/projects/Feathercoin/src/qt/forms: shiftdialog.ui
    Only in ~/projects/Feathercoin/src/qt/locale: bitcoin_de_AT.ts
    Only in ~/projects/Feathercoin/src/qt: Makefile.am
    Only in ~/projects/Feathercoin/src/qt: monitoreddatamapper.cpp
    Only in ~/projects/Feathercoin/src/qt: monitoreddatamapper.h
    Only in ~/projects/Feathercoin/src/qt: shiftdialog.cpp
    Only in ~/projects/Feathercoin/src/qt: shiftdialog.h

    Only in ~/projects/Feathercoin/src/qt/res: feathercoin-qt-res.rc
    Only in ~/projects/Feathercoin/src/qt/res: fonts
    Moved?
    Only in ~/projects/Feathercoin/src: rpcdump.cpp
    Only in ~/projects/Feathercoin/src: rpcwallet.cpp
    Only in ~/projects/Feathercoin/src: script.cpp
    Only in ~/projects/Feathercoin/src: script.h
    Only in ~/projects/Feathercoin/src: stealth.cpp
    Only in ~/projects/Feathercoin/src: stealth.h
    Only in ~/projects/Feathercoin/src: version.cpp
    Only in ~/projects/Feathercoin/src: wallet.cpp
    Only in ~/projects/Feathercoin/src: walletdb.cpp
    Only in ~/projects/Feathercoin/src: walletdb.h
    Only in ~/projects/Feathercoin/src: wallet.h

    Images and icons
    Only in ~/projects/Feathercoin/src/qt/res: feathercoin-qt-res.rc
    Only in ~/projects/Feathercoin/src/qt/res: fonts

    Binary files ~/projects/Feathercoin/src/qt/res/icons/bitcoin.icns and ~/Feathercoin/src/qt/res/icons/bitcoin.icns differ
    Binary files ~/projects/Feathercoin/src/qt/res/icons/bitcoin.png and ~/Feathercoin/src/qt/res/icons/bitcoin.png differ

    Only in ~/projects/Feathercoin/src/qt/res/icons: bitcoin_testnet.ico
    Only in ~/projects/Feathercoin/src/qt/res/icons: bitcoin_testnet.png
    Only in ~/projects/Feathercoin/src/qt/res/icons: coinnector.png
    Only in ~/projects/Feathercoin/src/qt/res/icons: shapeshift.png
    Only in ~/projects/Feathercoin/src/qt/res/icons: toolbar_testnet.png
    Only in ~/projects/Feathercoin/src/qt/res/images: about.1.png
    Only in ~/projects/Feathercoin/src/qt/res/images: about_bitcoin.png

    Binary files ~/projects/Feathercoin/src/qt/res/images/about.png and ~/Feathercoin/src/qt/res/images/about.png differ

    Only in ~/projects/Feathercoin/src/qt/res/images: coinnector.png
    Only in ~/projects/Feathercoin/src/qt/res/images: LOGO_b.png
    Only in ~/projects/Feathercoin/src/qt/res/images: LOGO_b_sm.png
    Only in ~/projects/Feathercoin/src/qt/res/images: LOGO_dg.png
    Only in ~/projects/Feathercoin/src/qt/res/images: LOGO_g.png
    Only in ~/projects/Feathercoin/src/qt/res/images: LOGO_g_sm.png

    Only in ~/projects/Feathercoin/src/qt/res/images: LOGO.png
    Only in ~/projects/Feathercoin/src/qt/res/images: mainbg.grey.1.png
    Only in ~/projects/Feathercoin/src/qt/res/images: mainbg.png
    Only in ~/projects/Feathercoin/src/qt/res/images: mainbg.shade.2.png -
    Only in ~/projects/Feathercoin/src/qt/res/images: mainbg.white.3.png

    Binary files ~/projects/Feathercoin/src/qt/res/images/paper_wallet.png and ~/Feathercoin/src/qt/res/images/paper_wallet.png differ

    Only in ~/projects/Feathercoin/src/qt/res/images: splash.png
    Only in ~/projects/Feathercoin/src/qt/res/images: splash_testnet.png

    Only in ~/projects/Feathercoin/src/qt/res: src


Log in to reply