erc-1155 icon indicating copy to clipboard operation
erc-1155 copied to clipboard

don't play well with openzepplin

Open cedricwalter opened this issue 4 years ago • 5 comments

Thanks for this great library! but updating openzeppelin-solidity to latest 4.0.0 and I can not compile my project

running but outdated combination taken from https://github.com/ProjectOpenSea/opensea-erc1155

    "multi-token-standard": "github:ProjectOpenSea/multi-token-standard",
    "openzeppelin-solidity": "^2.1.3",

How to reproduce, upgrading 3rd parties

openzeppelin-solidity 4.0.0    // use pragma solidity ^0.8.0;
@0xsequence/erc-1155 3.0.4    // use use pragma solidity ^0.7.4;

Tried both

    solc: {
      version: "^0.8.0",
    },

,ParserError: Source file requires different compiler version (current compiler is 0.8.3+commit.8d00100c.Emscripten.clang) - note that nightly builds are considered to be strictly less than
the released version
 --> @0xsequence/erc-1155/contracts/tokens/ERC1155/ERC1155MintBurn.sol:2:1:
  |
2 | pragma solidity 0.7.4;

or

    solc: {
      version: "^0.7.4",
    },

,ParserError: Source file requires different compiler version (current compiler is 0.8.3+commit.8d00100c.Emscripten.clang) - note that nightly builds are considered to be strictly less than
the released version
 --> @0xsequence/erc-1155/contracts/tokens/ERC1155/ERC1155MintBurn.sol:2:1:
  |
2 | pragma solidity 0.7.4;

what do you recommend? rollbacking openzeppelin-solidity to a previous version? hoping that I find one which stilluse Solc 0.7 branch?

cedricwalter avatar Apr 05 '21 08:04 cedricwalter

@cedricwalter thanks for the report -- I've noticed you closed this ticket -- did you find a solution? any updates on your experience?

thanks

pkieltyka avatar Apr 05 '21 16:04 pkieltyka

Yes and no, I switched to a pure Open Zeppelin ERC1155 implementation. I closed this report since it is not really an error, mixing 2 library (yours and open zeppelin), and adding truffle is something you may not want to urgently fix.

cedricwalter avatar Apr 05 '21 16:04 cedricwalter

Cool thanks. I'll reopen for us to look into some time in the future

pkieltyka avatar Apr 05 '21 17:04 pkieltyka

I actually prefer this implementation but want to use some other contracts from Open Zeppelin. I run into this same issue. Are you planning to upgrade to 0.8.*?

aalmada avatar May 04 '22 18:05 aalmada

@aalmada yes we will – it will be an easy upgrade too.

curious which parts you prefer in this implementation?

pkieltyka avatar May 04 '22 22:05 pkieltyka