Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Include formula for asciinema (http://asciinema.org/) #26011

Closed

Conversation

rogeriopradoj
Copy link
Contributor

No description provided.

Signed-off-by: Rogerio Prado de Jesus <rogeriopradoj@gmail.com>
url 'https://github.com/sickill/asciinema/archive/v0.9.7.tar.gz'
sha1 '1f93942163d36ea5ca94f9ee9c68d45c3a213148'

depends_on :python => "2.7"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the => "2.7"; this syntax is now deprecated.

@MikeMcQuaid
Copy link
Member

Also, please test this: install doesn't work as-is.

@rogeriopradoj
Copy link
Contributor Author

Sorry, @MikeMcQuaid , I'll work on it.

One question: I didn't find any formula in homebrew's repo using pip install XXXXX. Is it a restriction?

Signed-off-by: Rogerio Prado de Jesus <rogeriopradoj@gmail.com>
@MikeMcQuaid
Copy link
Member

Yes. If you need stuff from pip you should do it like ansible.rb.

Signed-off-by: Rogerio Prado de Jesus <rogeriopradoj@gmail.com>
Signed-off-by: Rogerio Prado de Jesus <rogeriopradoj@gmail.com>
@rogeriopradoj
Copy link
Contributor Author

@MikeMcQuaid I just include more commits in the PR, the tests are just the minimal for know (I'll work on this).

Instead of using pip I tried to follow the instructions of https://github.com/Homebrew/homebrew/wiki/Homebrew-and-Python the best I could, using the setup.py way.

The build will run again automatically in Jenkins, or do you prefer I send a new PR?

Thanks!!!

Signed-off-by: Rogerio Prado de Jesus <rogeriopradoj@gmail.com>
Signed-off-by: Rogerio Prado de Jesus <rogeriopradoj@gmail.com>
Signed-off-by: Rogerio Prado de Jesus <rogeriopradoj@gmail.com>
system "python", "setup.py", "install"
end

def test
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test do

@MikeMcQuaid
Copy link
Member

brew install and brew audit are failing. Please run brew install on your local machine and verify that it is working as expected before submitting changes to the pull request (or, in future, new pull requests).

@rogeriopradoj
Copy link
Contributor Author

Hi, @MikeMcQuaid, thanks for the tips!

1 similar comment
@rogeriopradoj
Copy link
Contributor Author

Hi, @MikeMcQuaid, thanks for the tips!

@kud
Copy link
Contributor

kud commented Feb 20, 2014

I want it I want it I want it.

@adamv
Copy link
Contributor

adamv commented Feb 20, 2014

@kud convince @rogeriopradoj to implement @MikeMcQuaid 's suggestions, or open a new clean pull request

@kud
Copy link
Contributor

kud commented Feb 20, 2014

Yeah I think I will do a PR. Cheers.

@adamv
Copy link
Contributor

adamv commented Feb 20, 2014

Note there is a new 0.9.8 version as well.

@rogeriopradoj
Copy link
Contributor Author

Please do it, :-)

I've used my pull request as a lab (my first contribution of a Python based
project) and, as expected for a lab project, it has blown!

I'm waiting yours to learn how to do it the right way!!!

See ya!

@adamv
Copy link
Contributor

adamv commented Mar 18, 2014

Will review an updated pull request for this.

@adamv adamv closed this Mar 18, 2014
@ku1ik
Copy link
Contributor

ku1ik commented Dec 18, 2014

Hey, asciinema 0.9.9 has been released. It has been rewritten in Go, so Python packaging problems go away. Now, what's the best way to package Go command line app? There are compiled binaries here https://github.com/asciinema/asciinema-cli/releases/tag/v0.9.9 , there's also easy way to build from source with make deps build && PREFIX=... make install. Thoughts?

@MikeMcQuaid
Copy link
Member

Build it from source, thanks.

@ku1ik ku1ik mentioned this pull request Dec 18, 2014
@Homebrew Homebrew locked and limited conversation to collaborators Feb 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants