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

hdiutils fails on macos-13 build #475

Closed
SanderVocke opened this issue Dec 9, 2024 · 5 comments
Closed

hdiutils fails on macos-13 build #475

SanderVocke opened this issue Dec 9, 2024 · 5 comments
Labels
bug Something isn't working fix-me for OpenHands agent

Comments

@SanderVocke
Copy link
Owner

The build step on macos-13 runners, as defined in build_and_test.yml GitHub Actions script, fails on creating .dmg images with hdiutil.

hdiutil is indirectly used in our build scripts by using the create-dmg tool.

Online reports have stated that dmgbuild does seem to work stably on these runners.

Change the github actions scripts to use dmgbuild instead of create-dmg to create a dmg application image. If possible keep the user experience the same, but this is not necessary as long as a working installable dmg is created.

Note that currently, create-dmg is installed in .github/actions/prepare_build_macos/action.yml, and used in .github/actions/build_package/action.yml. These places are likely where changes will need to be made.

@SanderVocke SanderVocke added bug Something isn't working fix-me for OpenHands agent labels Dec 9, 2024
Copy link

github-actions bot commented Dec 9, 2024

OpenHands started fixing the issue! You can monitor the progress here.

Copy link

github-actions bot commented Dec 9, 2024

A potential fix has been generated and a draft PR #476 has been created. Please review the changes.

@SanderVocke
Copy link
Owner Author

@openhands-agent

Please try again but disregard the potential solution of switching to dmgbuild. Instead, wrap the create-dmg command in a retry loop for max 10 iterations, hoping it will eventually succeed.

An example of how this can be done:

          # FIXME: Undo this overengineered crap once the following issue gets figured out:
          #        https://github.com/actions/runner-images/issues/7522
          max_tries=10
          i=0
          until create-dmg \
                  --no-internet-enable \
                  --format ULFO \
                  --background ../packaging/macos/dmg-background.png \
                  --hide-extension 'Notes Better.app' \
                  --icon 'Notes Better.app' 180 170 \
                  --icon-size 160 \
                  --text-size 12 \
                  --volname Notes \
                  --volicon ../src/images/notes_icon.icns \
                  --window-size 660 400 \
                  --app-drop-link 480 170 \
                  '${{ steps.vars.outputs.file_name }}' \
                  'Notes Better.app'
          do
              if [ $i -eq $max_tries ]
              then
                  echo 'Error: create-dmg did not succeed even after 10 tries.'
                  exit 1
              fi
              i=$((i+1))
          done

Copy link

OpenHands started fixing the issue! You can monitor the progress here.

Copy link

An attempt was made to automatically fix this issue, but it was unsuccessful. A branch named 'openhands-fix-issue-475-try2' has been created with the attempted changes. You can view the branch here. Manual intervention may be required.

@SanderVocke SanderVocke closed this as not planned Won't fix, can't repro, duplicate, stale Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fix-me for OpenHands agent
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant