solidity-flattener icon indicating copy to clipboard operation
solidity-flattener copied to clipboard

allow multiple solc-paths

Open zetherz opened this issue 6 years ago • 5 comments

Space-delimited, i.e. --solc-paths="my_solidity_path=my_actual_path my_solidity_path2=my_actual_path2"

Using slice notation.

zetherz avatar Jul 26 '18 04:07 zetherz

Why don't you use the format solc already has for this? I don't use aliases like that but does path1=realPath1,path2=realPath2, work?

XertroV avatar Aug 13 '18 00:08 XertroV

Why don't you use the format solc already has for this? I don't sue aliases like that but does path1=realPath1,path2=realPath2, work?

Also, not sure your patch would work b/c currently you either need to manually patch core.py or use --solc-paths "--allow-paths ./p1,./p2,

XertroV avatar Aug 13 '18 00:08 XertroV

@XertroV no, that didn't work for me, hence this PR. See also #5.

Not sure what your second point means. This patch works perfectly for me as-is without either of the two kludges you suggested. I've been using it for a while. Please feel free to actually try it and let me know if you still have issues with it.

zetherz avatar Aug 13 '18 01:08 zetherz

@zetherz replied to that issue

The second point was talking about using a command like: solidity_flattener --solc-paths "--allow-paths a=b,c=d,"

Note: I patched core.py manually on my machine so I don't add in --allow-paths at the beginning, but saw it referenced in another issue/PR today, so hopefully works?

XertroV avatar Aug 13 '18 03:08 XertroV