-
Notifications
You must be signed in to change notification settings - Fork 21
change /bin/sh to /bin/bash to fix failure on ubuntu #1
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
Open
sneak
wants to merge
109
commits into
unsound:master
Choose a base branch
from
sneak:patch-1
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or 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
Also started to modularize DMGExtractor (i.e. not using one large static method).
Also managed to slip in some changes that NetBeans made to the project files.
- Added 'struct classes' for the on-disk data representation of various UDIF structs. - Added a few new useful utilities. - Updated documentation on UDIF format. - Various minor fixes.
Hopefully it compiles...
Integrated into HFSExplorer and functioning nicely... (much work has been done)
A lot less information is now kept in memory, and parsing is faster (though other parts might be slower, but it can be optimized later). A lot of changes are related to switching from working with byte arrays to working with streams. I HAVE WORKED LIKE HELL!
- Renamed the XML parser to something more presentable (APX Parser for XML). - Updated the entire DMGExtractor program to use a more modular approach, with all the new classes. Found some bugs along the way, so some minor things should work better now.
It is better suited to be in a separate JAR, so that it won't have to be recompiled every time.
Will convert entire program to GPL3 to allow it to link with Apache bzip2 library.
Old dmg2iso reference sources are moved to a 'subprojects' directory and iHarder Bas64 sources are stored pre-compiled in a JAR file, which is extracted into the target JAR build dir when building the final JAR.
Yes I'm a fuckin' moron.
The plist parsing functionality was previously split between XMLNode, which is supposed to belong to the XML parser and not deal with specific applications of XML, and Plist.java in org.catacombae.dmg.udif, which is specific to UDIF parsing. To get a more general plist parsing facility, the org.catacombae.plist package was created which isolates the plist parsing specifics (and also introduces a possibility of adding support for non-XML plist formats in the future (binary plists for instance).
Method threw a NullPointerException instead of returning null when no key was found.
- IOException does not take a 'Throwable cause' in its constructor in JDK 5. - @OverRide annotation cannot be used for methods that are implementing an interface in JDK 5.
change /bin/sh to /bin/bash to fix failure on ubuntu (bash is no longer /bin/sh).
Owner
|
Hi, I just pushed a commit which should fix the root cause of this issue (using bash-specific comparison operators in scripts). The latest commit should make this pull request obsolete. |
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.
change /bin/sh to /bin/bash to fix failure on ubuntu (bash is no longer /bin/sh).