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

use latest versions #45

Merged
merged 4 commits into from
Mar 20, 2024
Merged

use latest versions #45

merged 4 commits into from
Mar 20, 2024

Conversation

0xalpharush
Copy link
Contributor

@0xalpharush 0xalpharush commented Mar 19, 2024

Summary by CodeRabbit

  • Chore: Updated the installation process in provision.sh for various tools including solc, slither, echidna, and medusa. The changes ensure that the latest versions of these tools are fetched and installed.
  • Refactor: Removed redundant apt-get update and apt-get install -y unzip commands to streamline the setup process.

These changes aim to keep the system setup and dependencies up-to-date, enhancing the reliability and performance of the environment.

Copy link

github-actions bot commented Mar 19, 2024

Image description CodeRabbit

Walkthrough

The changes primarily focus on updating the installation process for various tools in the provision.sh script. The versions of solc, slither, echidna, and medusa have been updated, along with their download URLs and installation commands. Redundant commands were removed to streamline the setup process.

Changes

File Summary
packer/provision.sh Updated the installation process for solc, slither, echidna, and medusa. The versions, download URLs, and installation commands for these tools have been modified. Removed redundant apt-get update and apt-get install -y unzip commands.

🐇💻

In the land of code, where logic intertwines,

Updates come like whispers in the vines.

Tools evolve, versions ascend,

To the call of progress, we attend.

Unzip no more, apt-get rest,

For in this change, we find our quest.

So here's to growth, in leaps and bounds,

In every line, improvement resounds! 🎉


Uplevel your code reviews with CodeRabbit Pro

CodeRabbit Pro

If you like this project, please support us by purchasing the Pro version. The Pro version has advanced context, superior noise reduction and several proprietary improvements compared to the open source version. Moreover, CodeRabbit Pro is free for open source projects.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image description CodeRabbit

Commits Files that changed from the base of the PR and between cbba8d8 and f085b04 commits.
Files selected (1)
  • packer/provision.sh (2)
Review comments generated (3)
  • Review: 3
  • LGTM: 0

Tips

Chat with Image description CodeRabbit Bot (@coderabbitai)

  • Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
  • Invite the bot into a review comment chain by tagging @coderabbitai in a reply.

Code suggestions

  • The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
  • You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Pausing incremental reviews

  • Add @coderabbitai: ignore anywhere in the PR description to pause further reviews from the bot.

packer/provision.sh Outdated Show resolved Hide resolved
Comment on lines 47 to 52
curl -fsSL https://github.com/crytic/medusa/releases/download/v0.1.3/medusa-linux-x64.tar.gz -o /tmp/medusa.tar.gz
echo "Extracting medusa..."
tar -xzf /tmp/medusa.tar.gz -C /tmp
echo "Installing medusa..."
chmod +x medusa
sudo mv medusa /usr/local/bin
Copy link

@github-actions github-actions bot Mar 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image description CodeRabbit

Similar to the previous comment, the downloaded file /tmp/medusa.tar.gz is not being removed after extraction and installation. Also, the chmod +x medusa command seems to be executed in the wrong directory. It should be executed in the /tmp directory where the file was extracted.

  tar -xzf /tmp/medusa.tar.gz -C /tmp
  echo "Installing medusa..."
- chmod +x medusa
+ chmod +x /tmp/medusa
  sudo mv /tmp/medusa /usr/local/bin
+ rm /tmp/medusa.tar.gz

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the rm is here already as well 👀

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image description CodeRabbit

@elopez You're correct, my apologies for the oversight. The rm /tmp/medusa.tar.gz command is indeed present in the script to remove the downloaded tar file after extraction and installation. However, the correction for the chmod +x medusa command still stands. It should be executed in the /tmp directory where the file was extracted. Here's the corrected line:

- chmod +x medusa
+ chmod +x /tmp/medusa

Thank you for pointing out the mistake.

packer/provision.sh Outdated Show resolved Hide resolved
packer/provision.sh Outdated Show resolved Hide resolved
packer/provision.sh Outdated Show resolved Hide resolved
Comment on lines 47 to 52
curl -fsSL https://github.com/crytic/medusa/releases/download/v0.1.3/medusa-linux-x64.tar.gz -o /tmp/medusa.tar.gz
echo "Extracting medusa..."
tar -xzf /tmp/medusa.tar.gz -C /tmp
echo "Installing medusa..."
chmod +x medusa
sudo mv medusa /usr/local/bin
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the rm is here already as well 👀

0xalpharush and others added 2 commits March 19, 2024 21:33
Co-authored-by: Emilio López <2642849+elopez@users.noreply.github.com>
Co-authored-by: Emilio López <2642849+elopez@users.noreply.github.com>
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image description CodeRabbit

Commits Files that changed from the base of the PR and between f085b04 and 857d0a8 commits.
Files selected (1)
  • packer/provision.sh (1)
Review comments generated (3)
  • Review: 2
  • LGTM: 1

Tips

Chat with Image description CodeRabbit Bot (@coderabbitai)

  • Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
  • Invite the bot into a review comment chain by tagging @coderabbitai in a reply.

Code suggestions

  • The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
  • You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Pausing incremental reviews

  • Add @coderabbitai: ignore anywhere in the PR description to pause further reviews from the bot.

packer/provision.sh Show resolved Hide resolved
packer/provision.sh Show resolved Hide resolved
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image description CodeRabbit

Commits Files that changed from the base of the PR and between 857d0a8 and aed135e commits.
Files selected (1)
  • packer/provision.sh (2)
Files skipped from review due to trivial changes (1)
  • packer/provision.sh
Review comments generated (0)
  • Review: 0
  • LGTM: 0

Tips

Chat with Image description CodeRabbit Bot (@coderabbitai)

  • Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
  • Invite the bot into a review comment chain by tagging @coderabbitai in a reply.

Code suggestions

  • The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
  • You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Pausing incremental reviews

  • Add @coderabbitai: ignore anywhere in the PR description to pause further reviews from the bot.

Copy link
Collaborator

@bohendo bohendo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for taking care of this.

Creation of packer images has been failing recently. Not related to this PR, but just a heads up: deployment of an image with latest echidna/medusa/solc to regions other than nyc3 might be delayed. cc @oldsj

@bohendo bohendo merged commit bfae614 into main Mar 20, 2024
6 checks passed
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

Successfully merging this pull request may close these issues.

3 participants