Skip to content
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

testing/gist: new aport #4272

Closed
wants to merge 1 commit into from
Closed

testing/gist: new aport #4272

wants to merge 1 commit into from

Conversation

refacto
Copy link
Contributor

@refacto refacto commented May 14, 2018

https://github.com/jdowner/gist
gist command line interface

python3 setup.py install --prefix=/usr --root="$pkgdir"

# remove unneeded files
rm $pkgdir/usr/share/$pkgname/LICENSE
Copy link
Collaborator

Choose a reason for hiding this comment

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

You should quote $pkgdir otherwise this rm breaks when the absolute path contains a space.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed.

# Contributor: Sascha Paunovic <azarus@posteo.net>
# Maintainer: Sascha Paunovic <azarus@posteo.net>
pkgname="gist"
_pkgname="python-gist"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Consider: _pkgname="python-$pkgname"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks!

arch="noarch"
license="MIT"
depends="python3"
makedepends="python3-dev py-setuptools"
Copy link
Member

Choose a reason for hiding this comment

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

py-setuptools is for python2, it should not be needed for python3.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, you're right. Will adjust the commit.

@andypost andypost added the A-new Adds a new aport package label May 16, 2018
@stale
Copy link

stale bot commented Mar 3, 2019

This pull request has been automatically marked as stale because it has not had activity in the last 60 days. It will be automatically closed if no further activity occurs in the next 14 days. If you are still waiting for a review or response from one of our developers you can leave a comment to remove the stale label. If you know the username of the maintainer of the aport you could try to mention the username in the comment. You can also join our IRC channels on freenode and see if you can find an available developer. Thank you for your contributions.

@stale stale bot added the S-stale label Mar 3, 2019
@stale
Copy link

stale bot commented Mar 17, 2019

This pull request has been automatically closed.

@stale stale bot closed this Mar 17, 2019
@maxice8
Copy link
Contributor

maxice8 commented Jun 14, 2019

Can't be merged, depends on unpackaged python-gnupg

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-new Adds a new aport package S-stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants