Skip to content

Commit

Permalink
fix: fix zkforge zkbuild on ubuntu (#137)
Browse files Browse the repository at this point in the history
Co-authored-by: Dustin Brickwood <[email protected]>
  • Loading branch information
nbaztec and dutterbutter authored Nov 2, 2023
1 parent f08160b commit 3aa13ee
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions crates/zkforge/bin/cmd/zk_solc.rs
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,7 @@ impl ZkSolc {
let sources = self.get_versioned_sources().wrap_err("Cannot get source files")?;

// Step 2: Compile Contracts for Each Source
for (solc, version) in sources {
for (_solc, version) in sources {
//configure project solc for each solc version
for (contract_path, _) in version.1 {
// Check if the contract_path is in 'sources' directory or its subdirectories
Expand All @@ -250,7 +250,7 @@ impl ZkSolc {
))?;

// Step 4: Build Compiler Arguments
let comp_args = self.build_compiler_args(&contract_path, &solc);
let comp_args = self.build_compiler_args(&contract_path, &self.project.solc);

// Step 5: Run Compiler and Handle Output
let mut cmd = Command::new(&self.compiler_path);
Expand Down
2 changes: 1 addition & 1 deletion crates/zkforge/bin/cmd/zksolc_manager.rs
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ impl ZkSolcOS {
/// This function is used to construct the URI for downloading the zkSync compiler binary.
fn get_download_uri(&self) -> &str {
match self {
ZkSolcOS::Linux => "linux-amd64",
ZkSolcOS::Linux => "linux-amd64-musl",
ZkSolcOS::MacAMD => "macosx-amd64",
ZkSolcOS::MacARM => "macosx-arm64",
}
Expand Down

0 comments on commit 3aa13ee

Please sign in to comment.