don't check error code of diskutil eject #4978
Merged
+12
−1
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
nstead, determine success by directly testing the intended effect: does the mount path still exist? Also: - retry once on failure. - silent success if the given mount point did not exist. These changes are intended to help with unpredictable problems with the test suite that manifest frequently on Travis. References: Homebrew#4975, Homebrew#4900, Homebrew#4857
rolandwalker
added a commit
that referenced
this pull request
Jun 19, 2014
don't check error code of diskutil eject
rolandwalker
added a commit
to rolandwalker/homebrew-cask
that referenced
this pull request
Jun 25, 2014
- Raise `CaskError` instead of `Plist::ParseError` from module - Improve error message when parse result is empty - remove leading garbage text and emit it to stderr (seen in Homebrew#5060) - remove trailing garbage text and emit it to stderr (seen in Homebrew#4819) This has the incidental effect of emitting DMG licenses during installation, which seems desirable as permanent functionality. If not permanent, the warnings to STDERR should still be kept temporarily to help get better bug reports on `hdiutil`. A bug wrt DMG licenses must have been introduced in one of Homebrew#4892, Homebrew#4887, Homebrew#4889, Homebrew#4900, Homebrew#4975, Homebrew#4978, or Homebrew#4857. Presumably, the cause is that STDERR was previously silenced when running `hdiutil`. It would be cleaner (and more reliable) to redirect STDERR and examine it separately, rather than clean up the merged outputs. closes Homebrew#4819 closes Homebrew#5060
This was referenced Jun 25, 2014
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Instead, check the result: does the mount path still exist?
This is intended to help with unpredictable problems with the
test suite that manifest frequently on Travis.
References: #4975, #4900, #4857