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

New package: python3-loguru-0.6.0 #37434

Closed
wants to merge 1 commit into from
Closed

New package: python3-loguru-0.6.0 #37434

wants to merge 1 commit into from

Conversation

oliver-cfc
Copy link
Contributor

Testing the changes

  • I tested the changes in this PR: YES

}

do_check() {
python3 setup.py test
Copy link
Member

Choose a reason for hiding this comment

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

this is already the default behaviour:

: ${make_check_target:=test}
${make_check_pre} python3 setup.py ${make_check_target} ${make_check_args}

Copy link
Member

Choose a reason for hiding this comment

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

but in this case you'd want to remove the do_check function add checkdepends="python3-pytest" because the tests in this package are run with pytest

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 - I really appreciate your comments and insight.
Is there an appropriate way to test this before amending the PR? The original template compiled perfectly in a freshly created xbps-src environment but then failed the build checks.

Does it even need a do_check{} section?

Copy link
Member

Choose a reason for hiding this comment

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

running xbps-src pkg and xbps-src check would be how I'd test building the template

@classabbyamp
Copy link
Member

the CI fail is probably caused by needing python3-setuptools in hostmakedepends

there's also a changelog that can be added: https://raw.githubusercontent.com/Delgan/loguru/master/CHANGELOG.rst

@paper42 paper42 added the new-package This PR adds a new package label Jun 7, 2022
@oliver-cfc
Copy link
Contributor Author

I think this is the good one. I added python3-colorama to checkdepends (along with python3-pytest) because xbps-src check fails without it but it's not needed for python3-loguru to run. Also added hostmakedepends and checkdepends.

# Template file for 'python3-loguru'
pkgname=python3-loguru
version=0.6.0
revision=1
wrksrc="loguru-${version}"
build_style=python3-module
hostmakedepends="python3-setuptools"
checkdepends="python3-pytest python3-colorama"
short_desc="Library which aims to bring enjoyable logging in Python"
maintainer="oliver <oliver@tuta.io>"
license="MIT"
homepage="https://github.com/Delgan/loguru"
changelog="https://raw.githubusercontent.com/Delgan/loguru/master/CHANGELOG.rst"
distfiles="${PYPI_SITE}/l/loguru/loguru-$version.tar.gz"
checksum=066bd06758d0a513e9836fd9c6b5a75bfb3fd36841f4b996bc60b547a309d41c

post_install() {
        vlicense LICENSE
}

Are these the correct next steps (while in my branch)?

  1. git add ./srcpkgs/python3-loguru/template
  2. git commit --amend
  3. git rebase -i HEAD~2
  4. git push origin

@classabbyamp
Copy link
Member

Are these the correct next steps (while in my branch)?

git add srcpkgs/python3-loguru/template
git commit --amend
git push --force

should be enough

@oliver-cfc
Copy link
Contributor Author

I assume I'm hitting this: Delgan/loguru#658
Do you want this PR closed until a new release of loguru?

@classabbyamp
Copy link
Member

i think it would be better to just disable that test (see pytest docs and use make_check_args) if it's a known and fixed issue in this version

@github-actions
Copy link

Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it.

@github-actions github-actions bot added the Stale label Sep 10, 2022
@github-actions github-actions bot closed this Sep 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-package This PR adds a new package Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants