autopsy 4.21.0 #190813
Draft
autopsy 4.21.0 #190813
+59
−94
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
github-actions
bot
added
perl
xakep8
force-pushed
the
autopsy_upgrade
branch
3 times, most recently
from
fce2d3c
to
8f19b37
Compare
carlocab
reviewed
Sep 16, 2024
xakep8
force-pushed
the
autopsy_upgrade
branch
from
8f19b37
to
7833b7a
Compare
github-actions
bot
added
java
xakep8
force-pushed
the
autopsy_upgrade
branch
2 times, most recently
from
65efbe7
to
44ee7e3
Compare
github-actions
bot
removed
the
missing license
carlocab
reviewed
Sep 17, 2024
Comment on lines
-23
to
-24
| # Installs prebuilt binaries, broken on arm: https://github.com/Homebrew/homebrew-core/issues/175053 | ||
| deprecate! date: "2024-06-22", because: :does_not_build |
There was a problem hiding this comment.
Thanks for working on this, but it looks like the reason we deprecated it still applies:
autopsy
* Binaries built for a non-native architecture were installed into autopsy's prefix.
The offending files are:
/opt/homebrew/Cellar/autopsy/4.21.0/libexec/autopsy/markmckinnon/Export_srudb_macos (x86_64)
/opt/homebrew/Cellar/autopsy/4.21.0/libexec/autopsy/markmckinnon/parse_prefetch_macos (x86_64)
/opt/homebrew/Cellar/autopsy/4.21.0/libexec/platform/modules/lib/x86_64/libjnidispatch-nb.jnilib (x86_64)
There was a problem hiding this comment.
@carlocab the formula is still being worked on I was just testing what was the issue with the old formula
There was a problem hiding this comment.
Sure, no problem! Please feel free to continue working on this.
When you feel that this PR is close to ready, please edit your commit message to read
autopsy 4.21.0
Thanks!
xakep8
force-pushed
the
autopsy_upgrade
branch
from
44ee7e3
to
b1b306d
Compare
This commit is part of issue Homebrew#139929. It upgrades the autopsy formula. style fix style fix autopsy: removed the deprecated label. autopsy: testing the old code and checking where it fails. fixed style issues added license. trying to fix the issue on arm64 added zlib
xakep8
force-pushed
the
autopsy_upgrade
branch
from
b1b306d
to
cca9c18
Compare
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.
This commit is part of issue #139929. It upgrades the autopsy formula.
HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where<formula>is the name of the formula you're submitting?brew test <formula>, where<formula>is the name of the formula you're submitting?brew audit --strict <formula>(after doingHOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it passbrew audit --new <formula>?