solidity icon indicating copy to clipboard operation
solidity copied to clipboard

installable solc from popular package managers

Open InoMurko opened this issue 4 years ago • 4 comments

Hey, first and foremost... love your work!

One thing I'd like to see... (willing to work too, but checking if you guys agree with the direction) is support for popular tooling that allows you to install raw binaries, brew and asdf would be my two choices for now. Perhaps for brew, we could add it in the ethereums repo?

But, that does mean that if you'd like to have L1 solc next to L2 solc, you need to differentiate binaries somehow. Is a general name replacement an option? Something like solc -> osolc? Next, a very small change to the codebase:

diff --git a/solc/CommandLineInterface.cpp b/solc/CommandLineInterface.cpp
index 05b021379..a4d1735ce 100644
--- a/solc/CommandLineInterface.cpp
+++ b/solc/CommandLineInterface.cpp
@@ -231,7 +231,7 @@ static set<string> const g_yulDialectArgs
 static void version()
 {
        sout() <<
-               "solc, the solidity compiler commandline interface" <<
+               "solc, the optimism solidity compiler commandline interface" <<
                endl <<
                "Version: " <<
                dev::solidity::VersionString <<
/usr/local/bin/solc --version
solc, the optimism solidity compiler commandline interface
Version: 0.5.16

Thanks and cheers

InoMurko avatar Apr 28 '21 10:04 InoMurko

One such framework that directly calls solc is https://github.com/compound-finance/saddle

InoMurko avatar Apr 29 '21 10:04 InoMurko

I can look into doing this! @smartcontracts do you know if this requires changes to our solidity plugin? Intuitively it seems like the command changing for the native build would be fine for our stuff, which directly calls the emscripten binaries, but I'm not positive.

ben-chain avatar Apr 29 '21 19:04 ben-chain

awesome @ben-chain lemme know if I can help in any way!

InoMurko avatar Apr 30 '21 13:04 InoMurko

Intuitively it seems like the command changing for the native build would be fine for our stuff, which directly calls the emscripten binaries, but I'm not positive.

I think you're correct. Don't think this would impact our plugins.

smartcontracts avatar May 12 '21 00:05 smartcontracts