Update run.yml #33
Open
Update run.yml #33
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Checking for the Correct Path: The script now checks if the Applications directory exists. If it doesn’t, it looks in Library/MobileSubstrate/DynamicLibraries or other relevant paths depending on where the files are located in the .deb. Fail Gracefully: If no appropriate directory is found, the script exits with a message indicating that no application files were found. This adjustment should prevent the script from failing due to missing directories.
Changes Made: Correct Path for zip: The zip -r App.ipa Payload command is executed in the ipa directory. This ensures the IPA file is created in the correct location. File Check: A check is added after the zip command to confirm that the .ipa file was created successfully. If the file is not found, the workflow will fail with a message. This should prevent the error and ensure that the .ipa file is created and uploaded correctly.
Key Updates: Extracting the Filename: The script extracts the filename from the .deb URL using basename and removes the .deb extension. This filename is then used to name the .ipa file. Dynamic .ipa Name: The .ipa file is named using the .deb file's base name (e.g., if the .deb file is com.hoangdus.speedster_2.1.1_iphoneos-arm.deb, the .ipa will be com.hoangdus.speedster_2.1.1_iphoneos-arm.ipa). This will ensure that the .ipa file has the same name as the .deb file, maintaining a consistent naming convention across the files.
What Changed: Handling Different Directory Structures: The script now checks if the extracted files are in var/jb/Applications or var/jb/Library. This allows it to work with .deb files that use different directory structures (common in jailbroken apps or tweaks). Error Handling: The script will exit with an error if no valid application files are found in any expected directory. This update should resolve the issue you encountered by adapting to the structure of the .deb file.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Checking for the Correct Path: The script now checks if the Applications directory exists. If it doesn’t, it looks in Library/MobileSubstrate/DynamicLibraries or other relevant paths depending on where the files are located in the .deb. Fail Gracefully: If no appropriate directory is found, the script exits with a message indicating that no application files were found. This adjustment should prevent the script from failing due to missing directories.