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 packages: klfc-1.5.7 & ascii-image-converter-1.11.0 #36922

Closed
wants to merge 2 commits into from
Closed

New packages: klfc-1.5.7 & ascii-image-converter-1.11.0 #36922

wants to merge 2 commits into from

Conversation

nadevko
Copy link

@nadevko nadevko commented Apr 30, 2022

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for my native architecture, x86_64-glibc

Copy link
Member

@classabbyamp classabbyamp left a comment

Choose a reason for hiding this comment

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

templates should build from source, not just download the binaries from upstream. This will not work for cross, or anything other than x86_64 glibc, as-is.

Please also run xlint (from xtools) on the templates

Comment on lines +15 to +17
do_extract() {
bsdtar --strip-components 1 -xf $XBPS_SRCDISTDIR/${wrksrc##*/}/v$version.tar.gz
}
Copy link
Member

Choose a reason for hiding this comment

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

should not be necessary, xbps-src knows how to unpack tar.gz and zip files itself

version='1.11.0'
checksum='17f87445971fdb491c6ca9d4beed67044a330339b47775adffb8e3cb898ec15d'

distfiles="https://github.com/TheZoraiz/ascii-image-converter/releases/download/v$version/ascii-image-converter_Linux_amd64_64bit.tar.gz>v$version.tar.gz"
Copy link
Member

Choose a reason for hiding this comment

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

should not be necessary to change the filename

@classabbyamp
Copy link
Member

also, unless one of these packages depends on the other, they should be in separate PRs

@nadevko
Copy link
Author

nadevko commented Apr 30, 2022

Thanks, I'll fix it tomorrow,

@nadevko nadevko closed this Apr 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants