Bitcoin Forum
September 27, 2018, 10:39:03 PM *
News: ♦♦ New info! Bitcoin Core users absolutely must upgrade to previously-announced 0.16.3 [Torrent]. All Bitcoin users should temporarily trust confirmations slightly less. More info.
 
   Home   Help Search Donate Login Register  
Pages: [1]
  Print  
Author Topic: Bounty for CPFP Enabled Wallet  (Read 2429 times)
d4n13
Full Member
***
Offline Offline

Activity: 210
Merit: 101


“Create Your Decentralized Life”


View Profile
July 15, 2015, 08:12:27 AM
 #1

CPFP (Child Pays for Parent) Bounty
3FEUByMeaxrNmBCVYjvsnhyAjiUdat5i7M

Anyone who can put a CPFP transaction creation feature into any wallet featured on the bitcoin homepage can claim this bounty.  

I outlined the basic idea of the bounty in this BIP proposal before I was educated on CPFP.

To claim the bounty / pizza you will need to convince the following holders of the multisig

Rules:
  • Must be a wallet featured on the bitcoin homepage
  • Must be either accepted into primary build branch of (1), or available in an official release / beta / dev build of (1)

References:

1538087943
Hero Member
*
Offline Offline

Posts: 1538087943

View Profile Personal Message (Offline)

Ignore
1538087943
Reply with quote  #2

1538087943
Report to moderator
1538087943
Hero Member
*
Offline Offline

Posts: 1538087943

View Profile Personal Message (Offline)

Ignore
1538087943
Reply with quote  #2

1538087943
Report to moderator
Make a difference with your Ether.
Donate Ether for the greater good.
SPRING.WETRUST.IO
Advertised sites are not endorsed by the Bitcoin Forum. They may be unsafe, untrustworthy, or illegal in your jurisdiction. Advertise here.
Andreas Schildbach
Hero Member
*****
Offline Offline

Activity: 483
Merit: 500


View Profile
July 15, 2015, 09:41:53 AM
 #2

We're already working on getting child-pays-for-parent into Bitcoin Wallet. The branch is here: https://github.com/schildbach/bitcoin-wallet/commits/fee-cpfp

It's still a bit rough and needs work. The biggest missing part is PIN entry, so you currently need to unset your PIN before raising the fee.

Any code review is appreciated.
d4n13
Full Member
***
Offline Offline

Activity: 210
Merit: 101


“Create Your Decentralized Life”


View Profile
July 15, 2015, 03:09:42 PM
 #3

We're already working on getting child-pays-for-parent into Bitcoin Wallet. The branch is here: https://github.com/schildbach/bitcoin-wallet/commits/fee-cpfp

It's still a bit rough and needs work. The biggest missing part is PIN entry, so you currently need to unset your PIN before raising the fee.

Any code review is appreciated.
Wow... fantastic news.... I'll be happy to look at it...

Setting up an android build server will take a bit.  If no one beats me too it, I'll try to post a testnet build from the branch... expect it to take a while.

achow101
Moderator
Legendary
*
expert
Offline Offline

Activity: 1540
Merit: 1659


3F1Y9yquzvY6RWvKbw2n2zeo9V5mvBhADU


View Profile WWW
July 15, 2015, 03:40:58 PM
 #4

Does the CPFP patch have to make it into the stable build released by the developers of the wallets or can it just be a patch that you apply yourself?

d4n13
Full Member
***
Offline Offline

Activity: 210
Merit: 101


“Create Your Decentralized Life”


View Profile
July 15, 2015, 03:49:48 PM
 #5

Does the CPFP patch have to make it into the stable build released by the developers of the wallets or can it just be a patch that you apply yourself?
I added a rules section to OP
Quote
Rules:
  • Must be a wallet featured on the bitcoin homepage
  • Must be either accepted into primary build branch of (1), or available in a production / beta / dev build of (1)

Andreas Schildbach
Hero Member
*****
Offline Offline

Activity: 483
Merit: 500


View Profile
July 16, 2015, 03:06:38 PM
 #6

I just updated our branch so that it supports encrypted wallets (spending PIN).
Andreas Schildbach
Hero Member
*****
Offline Offline

Activity: 483
Merit: 500


View Profile
July 17, 2015, 03:18:50 PM
 #7

The feature is now finished, merged and starts to roll out to 10% of users.

If you feel like skipping the queue you can download directly from here:

https://github.com/schildbach/bitcoin-wallet/releases/tag/v4.37
d4n13
Full Member
***
Offline Offline

Activity: 210
Merit: 101


“Create Your Decentralized Life”


View Profile
July 17, 2015, 04:16:24 PM
 #8

The feature is now finished, merged and starts to roll out to 10% of users.

If you feel like skipping the queue you can download directly from here:

https://github.com/schildbach/bitcoin-wallet/releases/tag/v4.37
All over it... Thx... I assume the "test" is testnet.

d4n13
Full Member
***
Offline Offline

Activity: 210
Merit: 101


“Create Your Decentralized Life”


View Profile
July 21, 2015, 03:02:30 PM
 #9

The feature is now finished, merged and starts to roll out to 10% of users.

If you feel like skipping the queue you can download directly from here:

https://github.com/schildbach/bitcoin-wallet/releases/tag/v4.37
Awarded
to Andreas Schildbach

congrats!
https://blockchain.info/tx/8ec943be37e8e45ba60e0c3ae82323cade5e084b25e9154714e86a5d80a1a44b

unamis76
Legendary
*
Offline Offline

Activity: 1414
Merit: 1000


View Profile
July 21, 2015, 03:24:48 PM
 #10

Good news that this has been implemented... However I don't use the software in which it was Cheesy Hoping this at least makes it to the Core wallet.
arnuschky
Hero Member
*****
Offline Offline

Activity: 516
Merit: 500


View Profile
July 26, 2015, 10:15:18 PM
 #11


A bit OT, but which other nodes/wallets implement CPFP already out-of-the-box?
brianddk
Newbie
*
Offline Offline

Activity: 28
Merit: 0


View Profile WWW
July 26, 2015, 10:49:44 PM
 #12


A bit OT, but which other nodes/wallets implement CPFP already out-of-the-box?
Only Android Bitcoin-Wallet...

No others that I've seen, which is why it won the bounty.
voisine
Member
**
Offline Offline

Activity: 120
Merit: 10


View Profile
August 26, 2015, 03:26:16 PM
 #13

We included CPFP in breadwallet a few months back (two months before the bounty was created) for when the user spends unconfirmed non-change inputs. Happy to let the bounty go to Andreas though. Andreas and co. are doing good work.

https://github.com/voisine/breadwallet/blob/v0.5.1/BreadWallet/BRWallet.m#L382
unamis76
Legendary
*
Offline Offline

Activity: 1414
Merit: 1000


View Profile
August 26, 2015, 03:43:30 PM
 #14

We included CPFP in breadwallet a few months back (two months before the bounty was created) for when the user spends unconfirmed non-change inputs. Happy to let the bounty go to Andreas though. Andreas and co. are doing good work.

https://github.com/voisine/breadwallet/blob/v0.5.1/BreadWallet/BRWallet.m#L382

Wasn't aware that one of my favorite wallets already had it Smiley Nice job. It will probably come in handy some day Grin
Pages: [1]
  Print  
 
Jump to:  

Sponsored by , a Bitcoin-accepting VPN.
Powered by MySQL Powered by PHP Powered by SMF 1.1.19 | SMF © 2006-2009, Simple Machines Valid XHTML 1.0! Valid CSS!