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

Hints for CN coins #34

Open
zawy12 opened this issue Oct 29, 2018 · 3 comments
Open

Hints for CN coins #34

zawy12 opened this issue Oct 29, 2018 · 3 comments

Comments

@zawy12
Copy link
Owner

zawy12 commented Oct 29, 2018

Post a comment to this issue if we say anything important CN / Monero clones need to know. Edit your previous comment instead of adding more than 1 comment per person. I'll start.

There were some discussions yesterday about merged-mining and how to stop the capability. We are running a fork of TurtleCoin and wondering if you guys ever came up with a good solution we could start testing...?

Just do a hard fork to a new block template with no TX_Extra

@RocksteadyTC
Copy link

Might be good to have some sort of quote attribution so we know who to ping @zawy12

@2acoin
Copy link

2acoin commented Oct 30, 2018

Posted in the thread relating to the above comment...

Test BotToday at 2:42 PM
@2ACoin in case of your coin, recommendation could be the following:
- in case you don't wish to be merge-mined as a child coin (as it happens now), you should return to similar to version 1 block template as Karbo did (also @ran (BBSCoin) proposed a different trick that could work which just checks vin/vout of parent block coinbase TX)
- in case you don't wish to be merge-mined as a parent coin (in your case AEON is the parent), you should block merge mining tag in your extra, for example, as proposed by @Aiwe (Karbo), somewhere in Currency::checkProofOfWorkV1
But please note that in case of BBS coin, the merged mining is just side effect. The attack could be the same even without any merged mining.

@aivve
Copy link

aivve commented Oct 30, 2018

If you don't want to be mined as parent coin, this should prevent miners include mm_tag in coinbase tx using modified daemon seredat/karbowanec@79da48c

Removing merged mining as slave coin is in these two commits:
seredat/karbowanec@5d4c87d
seredat/karbowanec@0ba448f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants