Skip to content

Change target folder to /target and move on to version 0.13#124

Merged
webmaster128 merged 1 commit into
mainfrom
outside-target
Jun 19, 2023
Merged

Change target folder to /target and move on to version 0.13#124
webmaster128 merged 1 commit into
mainfrom
outside-target

Conversation

@webmaster128

@webmaster128 webmaster128 commented Jun 12, 2023

Copy link
Copy Markdown
Member

Closes #89

@maurolacy maurolacy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@webmaster128 webmaster128 marked this pull request as ready for review June 13, 2023 09:34

@chipshort chipshort left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@webmaster128 webmaster128 merged commit 5540c43 into main Jun 19, 2023
@webmaster128 webmaster128 deleted the outside-target branch June 19, 2023 11:28
@webmaster128

Copy link
Copy Markdown
Member Author

I tested this with the expected results on a local workspace with 9 contracts. In all cases, no local ./target folder was created. I.e. the optimizer does not mess with the local build directory.

No target cache set (not recommended but still solid)

  • Run 1: 5:58,49 total (the first one created the registry_cache cache. The following ones are a minute faster)
  • Run 2: 4:55,18 total
  • Run 3: 5:00,33 total
  • Run 4: 5:19,70 total

Target mounted (--mount type=volume,source="$(basename "$(pwd)")_cache",target=/target)

  • Run 1: 5:29,11 total
  • Run 2: 27,760 total (huge win here, as expected)
  • Run 3: 28,324 total
  • Run 4: 29,328 total

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.

Processing custom cargo environment

3 participants