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

fix: remove --deep #188

Merged
merged 1 commit into from
Nov 7, 2024
Merged

fix: remove --deep #188

merged 1 commit into from
Nov 7, 2024

Conversation

Kek5chen
Copy link
Contributor

@Kek5chen Kek5chen commented Nov 5, 2024

Resolves #182

See issue for more info.

@github-actions github-actions bot added gui Affects the Avalonia UI or translations. infra Updates to dependencies labels Nov 5, 2024
@GreemDev
Copy link
Owner

GreemDev commented Nov 5, 2024

If you could test this PR build once it finishes to see if anything is broken that'd be appreciated; I don't own a mac.
If all is well I should have no issues merging.

@Bentheminernz
Copy link

If you could test this PR build once it finishes to see if anything is broken that'd be appreciated; I don't own a mac. If all is well I should have no issues merging.

Everything is working the same as before for me. All still works

@Kek5chen
Copy link
Contributor Author

Kek5chen commented Nov 5, 2024

It also builds in nixpkgs. That was my success marker. Just previously it didn't build on MacOS 13.0+ because of the removal of --deep from codesign

@sokennethwasall
Copy link

Working well. Seems to be more stable than the current version- Been playing Echoes of Wisdom for a couple hours now without a crash

@GreemDev GreemDev merged commit 36c374c into GreemDev:master Nov 7, 2024
11 checks passed
@amurgshere
Copy link
Contributor

amurgshere commented Nov 8, 2024

Hello @GreemDev ,

Since this has been checked in I can no longer create and sign the app bundle locally.

I am using the following to build, create, and sign the app bundle:

./distribution/macos/create_macos_build_ava.sh ./ ./distribution/macos/temp ./distribution/macos/output ./distribution/macos/entitlements.xml 1.0.0 0 Release

I get this error when it attempts to sign:

/Volumes/MACBOOK/ROMS/NSW/Ryujinx/GitHub/Ryujinx/distribution/macos/temp/output_x64/Ryujinx.app: code object is not signed at all
In subcomponent: /Volumes/MACBOOK/ROMS/NSW/Ryujinx/GitHub/Ryujinx/distribution/macos/temp/output_x64/Ryujinx.app/Contents/Frameworks/libglfw.3.dylib

if I reinstate the "--deep" parameter it starts working again, I assume this is because the "/libglfw.3.dylib" file and other library files need to be re-signed and it is no longer being signed because of the missing "--deep" parameter, do I need to do something differently now to get the app bundle to be signed successfully?

Note that if I add an extra line before this line in the "create_app_bundle.sh" script:

codesign --entitlements "$ENTITLEMENTS_FILE_PATH" -f -s - "$APP_BUNDLE_DIRECTORY"

Of:

find "$APP_BUNDLE_DIRECTORY" -name "*.dylib" -exec codesign -f -s - {} ;

Forcing these libraries to be code re-signed, the issue also goes away, maybe we should add this line in?

REPRODUCTION

  1. Clone the repository to an M1 MAC
  2. Run this command from the cloned directory:

./distribution/macos/create_macos_build_ava.sh ./ ./distribution/macos/temp ./distribution/macos/output ./distribution/macos/entitlements.xml 1.0.0 0 Release

  1. x64 version is not signed properly and Universal bundle is not created at all as the x64 signing failed which stops the script.

Thank You.

@Juoelenis
Copy link

does it fix crashing on rasberry pi 5?

@Kek5chen
Copy link
Contributor Author

Kek5chen commented Nov 9, 2024

@amurgshere please make a new issue for this.

@amurgshere
Copy link
Contributor

Thanks @Kek5chen , I have raised Issue #210 and Pull Request #208 .

@GreemDev GreemDev mentioned this pull request Nov 13, 2024
GreemDev added a commit that referenced this pull request Nov 19, 2024
Significant changes include LDN functionality from @Vudjun (no more
separate build!) and an XCI trimmer from @amurgshere.

Merged PRs in this release (in the order they were merged): 
#183, #150, #105, #160, #188, #98, #158, #13, #216, #73, #217, #122,
#228, #65, #226, #236, #247, #243, #249, #242, #260, #273, #272, #262,
#259, #241

## Versioning:
There now exists "stable" (release branch) and ["canary" (master
branch)](https://github.com/GreemDev/Ryujinx-Canary/releases) versions.
Instead of everyone using the same emulator, getting updates for every
code change, you now *opt-in* to the more frequent updates by using the
Canary version. Use stable and you'll get about an update a week, but
that update will be MUCH more significant as it's the entire previous
week's changes & PR merges.

## LDN
LDN functionality is now merged! Use
[this](https://github.com/GreemDev/Ryujinx/wiki/Multiplayer%E2%80%90(LDN%E2%80%90Local%E2%80%90Wireless)%E2%80%90Guide)
to get started.
Please note that LDN is only for local wireless; **this is not a
Nintendo Switch Online emulation feature**.

## UI
  - Added an XCI trimmer (#105).
- You can use this feature to trim dead bytes & the embedded firmware
out of your dumped XCIs, to make them smaller.
- If you right-click an XCI and the trim button it is greyed out, that
means your XCI is already as small as possible.
  - Fix for fullscreen not being really fullscreen (#150)
  - Fix window sizing calculations when Show Title Bar is enabled (#247)
- The "Install/Uninstall file types" buttons will be enabled/disabled
depending on which one you contextually need; install will be clickable
when they aren't installed, and vice versa.
- Fix for showing default config screen when swapping players in
controller settings (#122)
- Command-line argument to prevent update checking `--hide-updates`
(#272)
  - # RPC: 
    - Added a LOT of game images to Discord RPC.
    - Play time will now show the time unit hours at a maximum.

## Localization
- Update outdated/incorrect & added missing translations for zh-TW
(#158)
  - Add many missing locale strings to all languages (#160)
  - Update & improve Korean translation (#226)
  - Minor fixes & add missing translations to Spanish translation (#242)

## Headless
- Added `ignore-controller-applet` as an option you can configure via
headless command-line options.

## Graphics Backend
  - ### Vulkan
    - fix divide-by-zero when recovering from missed draw (#235) 
      - fixes crash in 'Baldo: The Guardian Owls' opening cutscene
    
## Horizon
- fix crash that occurs when launching an NSP forwarder generated by
Nro2Nsp (#237)

# Nerd Zone
Slightly more technical information. If you don't understand what's
under here, no worry.

- Updater now uses the release's Tag Name instead of its Name for
version checking.
- Baked in value change logging into ReactiveObject.
- Split ConfigurationState into 3, smaller partial classes of the same
name.
- Specify if the current version is Canary in the version log line

---------

Co-authored-by: James Duarte <[email protected]>
Co-authored-by: Luke Warner <[email protected]>
Co-authored-by: TheToid <[email protected]>
Co-authored-by: GabCoolGuy <[email protected]>
Co-authored-by: Kekschen <[email protected]>
Co-authored-by: WilliamWsyHK <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jacobwasbeast <[email protected]>
Co-authored-by: Piplup <[email protected]>
Co-authored-by: Vladimir Sokolov <[email protected]>
Co-authored-by: Jonas Henriksson <[email protected]>
Co-authored-by: Vudjun <[email protected]>
Co-authored-by: extherian <[email protected]>
Co-authored-by: Hack茶ん <[email protected]>
Co-authored-by: EmulationEnjoyer <[email protected]>
Co-authored-by: Nicola <[email protected]>
Co-authored-by: jzumaran <[email protected]>
Co-authored-by: Pitchoune <[email protected]>
Co-authored-by: Narugakuruga <[email protected]>
marco-carvalho pushed a commit to marco-carvalho/Ryujinx-1 that referenced this pull request Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gui Affects the Avalonia UI or translations. infra Updates to dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] don't sign with --deep on MacOS
6 participants