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

Upgrade create-dmg to latest version 1.1.0 #2664

Merged
merged 1 commit into from
Jun 23, 2022

Conversation

ann0see
Copy link
Member

@ann0see ann0see commented Jun 22, 2022

I'd like to have this in 3.9.0

Short description of changes

Upgrades create-dmg to the latest version. Nothing changed (especially this PR does not change the compression format)

CHANGELOG: Mac: Upgraded create-dmg to the latest version

Context: Fixes an issue?

No. Related to: #2663

Does this change need documentation? What needs to be documented and how?

No.

Status of this Pull Request
Ready for second review.

What is missing until this pull request can be merged?

CI and testing

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I tested my code and it does what I want
  • My code follows the style guide
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • I've filled all the content above

@ann0see ann0see requested a review from hoffie June 22, 2022 18:26
@hoffie hoffie added this to the Release 3.9.0 milestone Jun 22, 2022
@hoffie hoffie changed the title Upgrade create-dmg to latest version Upgrade create-dmg to latest version 1.1.0 Jun 22, 2022
Copy link
Member

@hoffie hoffie left a comment

Choose a reason for hiding this comment

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

Looks good. Proper create-dmg version has been installed, build is green. Can't do any other Mac tests though.

@ann0see ann0see force-pushed the upgradeCreate-DMG branch from 065c740 to c88680c Compare June 22, 2022 18:40
@ann0see
Copy link
Member Author

ann0see commented Jun 22, 2022

I'll test it once the CI has compiled again.

@ann0see
Copy link
Member Author

ann0see commented Jun 22, 2022

Ok. Tested the old build (with the wrong commit message) and it installs as before on macOS

@ann0see ann0see requested a review from pljones June 23, 2022 10:50
@pljones
Copy link
Collaborator

pljones commented Jun 23, 2022

Do we know why this is happening (was on 1.0.9, too) - happens just after the create-dmg run line:

relocator: cannot create slice file: /Users/runner/work/_temp/codeql_databases/working/copy-root/000001F5/usr/bin/nice.semmle.00001D1A.slice.x86_64: Operation not permitted

I take it there's no reason why this would affect the supported legacy MacOS version as it's just the DMG packaging?

@ann0see
Copy link
Member Author

ann0see commented Jun 23, 2022

Since nothing big changes (only disabled support for more modern compression and signing), nothing changes for now.

@ann0see
Copy link
Member Author

ann0see commented Jun 23, 2022

No, I don't know about this error

@ann0see ann0see merged commit 3f406aa into jamulussoftware:master Jun 23, 2022
@ann0see ann0see deleted the upgradeCreate-DMG branch June 23, 2022 20:40
@ann0see
Copy link
Member Author

ann0see commented Jul 25, 2022

@pljones Another minor fix here.

@pljones
Copy link
Collaborator

pljones commented Jul 26, 2022

Where?

@ann0see
Copy link
Member Author

ann0see commented Jul 26, 2022

Upgrade was changed to upgraded

@pljones
Copy link
Collaborator

pljones commented Jul 26, 2022

Just to note: as I'm editing these by hand (I'd have to delete them all and start again otherwise, losing all the edits I've already done), it's clearer to state the change you're suggesting than making it.

@ann0see
Copy link
Member Author

ann0see commented Jul 26, 2022

Ok. Will do that in future.

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