-
-
Notifications
You must be signed in to change notification settings - Fork 11.4k
cowpatty: some small patches which makes correct use of stdin #42336
Conversation
@@ -11,7 +11,12 @@ class Cowpatty < Formula | |||
sha256 "35fba0f92c5e8fb0710453d0c2c5fe5e9c64857dd53b219977871340b22c4942" | |||
end | |||
|
|||
def patches | |||
[ "http://proton.cygnusx-1.org/~edgan/cowpatty/cowpatty-4.6-fixup16.patch" ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Has this patch been accepted upstream?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please submit this patch to the upstream developers of this project and add a link to the upstream patch submission and explanation of why the patch is needed in a comment in the formula file. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This patch probably never will be in upstream because this project is very old and without a maintainer. Latest version is from July 03 2009 and homepages are not working. However this patch is absolutely neccessary if you want to safe half time of wpa cracking with the use of stdout from pyrit or other similar software and it is recommended way how to get cowpatty on Debian, Ubuntu and other distros. Unfortunatelly only references to that are on several blogs..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would still like it submitted anyway, thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be:
patch do
url ""
sha256 ""
end
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, fixed..
mikemcquaid: I am sorry I don't understand to which upstream I have to submit a patch ? As I wrote, this project has no upstream, no developer, no homepages. So I cannot submit it there. May I ask you for an advice ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The formula has a homepage and therefore a developer. I'd rather we didn't accept this if upstream is now unmaintained.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, sorry for that last time web pages doesn't work. I wrote a message to the address on the end of README file, so we will see. Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ever get a response on this? If not, I think we'd prefer to pass on it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No response at all :(
It sounds like we should boneyard cowpatty. |
@tdsmith Agreed 👍 |
Not actively maintained. Oses-clay Homebrew/legacy-homebrew#42336.
Not actively maintained. Oses-clay Homebrew/legacy-homebrew#42336. Closes #75. Signed-off-by: Andrew Janke <andrew@apjanke.net>
Cowpatty has been boneyarded, in ba44936. |
Now you can be able to pass output from pyrit over stdout/stdin directly to cowpatty.