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: gek-0.1 #37817

Closed
wants to merge 1 commit into from
Closed

New package: gek-0.1 #37817

wants to merge 1 commit into from

Conversation

johngebbie
Copy link
Contributor

Testing the changes

  • I tested the changes in this PR: YES

New package

@classabbyamp classabbyamp added the new-package This PR adds a new package label Jul 3, 2022
srcpkgs/gek/template Outdated Show resolved Hide resolved
srcpkgs/gek/template Outdated Show resolved Hide resolved
srcpkgs/gek/template Outdated Show resolved Hide resolved
Comment on lines 20 to 21
mkdir "${DESTDIR}/usr/share/doc"
cp -r doc "${DESTDIR}/usr/share/doc/${pkgname}"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
mkdir "${DESTDIR}/usr/share/doc"
cp -r doc "${DESTDIR}/usr/share/doc/${pkgname}"
vmkdir /usr/share/doc
vcopy doc "/usr/share/doc/${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.

Ok I've done it.

@johngebbie
Copy link
Contributor Author

I'm not sure what to do about the failing tests, I can't tell what's going wrong.

@classabbyamp
Copy link
Member

=> ERROR: gek-0.1_1: vinstall: cannot find '/builddir/gek-v0.1/_build-gek-xbps/bin/src'...

I believe go puts things in a different dir for cross builds.

this is what the build style does:

do_install() {
for f in ${GOPATH}/bin/* ${GOPATH}/bin/**/*; do
if [ -f "$f" ] && [ -x "$f" ]; then
vbin "$f"
fi
done
}

srcpkgs/gek/template Outdated Show resolved Hide resolved
@johngebbie
Copy link
Contributor Author

I'll close this, my program could do with some work, and I don't want to add to the backlog. Thanks for helping me learn the xbps-src.

@johngebbie johngebbie closed this Sep 29, 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants