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

parpd: new package #33295

Merged
merged 1 commit into from Oct 11, 2021
Merged

parpd: new package #33295

merged 1 commit into from Oct 11, 2021

Conversation

ArsenArsen
Copy link
Contributor

General

Have the results of the proposed changes been tested?

  • I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
  • I generally don't use the affected packages but briefly tested this PR

Does it build and run successfully?

(Please choose at least one native build and, if supported, at least one cross build. More are better.)

  • I built this PR locally for my native architecture, (x86_64-glibc)
  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • aarch64
    • aarch64-musl - does not appear to build on musl? I can't do further testing right now, but it's probably due to some glibc-specific transient header include.

srcpkgs/parpd/template Outdated Show resolved Hide resolved
srcpkgs/parpd/template Show resolved Hide resolved
@paper42
Copy link
Member

paper42 commented Oct 3, 2021

The commit message should be "New package: $pkgname-$version"

@paper42 paper42 added the new-package This PR adds a new package label Oct 3, 2021
@ArsenArsen ArsenArsen force-pushed the new-package-parpd branch 2 times, most recently from 23307e7 to f4b60bd Compare October 6, 2021 09:49
@ArsenArsen
Copy link
Contributor Author

With that patch added, this builds on {aarch64,x86_64}{,-musl}. Tested via for x in {aarch64,x86_64}{,-musl}; do ./xbps-src -a "$x" pkg parpd || break; done.

@paper42
Copy link
Member

paper42 commented Oct 8, 2021

It seems like they forgot to #include <sys/uio.h> in this release, but fixed it in https://roy.marples.name/git/parpd/commit/f126cdfe98d99d083b497fefc2a2cf0eef4ef0b6

=> parpd-1.7_1: running do_build ...
cc -fstack-clash-protection -D_FORTIFY_SOURCE=2 -mtune=generic -O2 -pipe        -DSYSCONFDIR=\"/etc\" -c parpd.c -o parpd.o
cc -fstack-clash-protection -D_FORTIFY_SOURCE=2 -mtune=generic -O2 -pipe        -DSYSCONFDIR=\"/etc\" -c eloop.c -o eloop.o
cc -fstack-clash-protection -D_FORTIFY_SOURCE=2 -mtune=generic -O2 -pipe        -DSYSCONFDIR=\"/etc\" -c lpf.c -o lpf.o
sed -e 's:@SYSCONFDIR@:/etc:g' parpd.8.in >parpd.8
lpf.c: In function 'bpf_write':
lpf.c:126:9: warning: implicit declaration of function 'writev'; did you mean 'write'? [-Wimplicit-function-declaration]
  126 |  return writev(ifp->fd, iov, 2);
      |         ^~~~~~
      |         write
cc -Wl,-z,relro -Wl,-z,now -Wl,--as-needed     -o parpd parpd.o eloop.o lpf.

@ArsenArsen
Copy link
Contributor Author

right, that fixes that warning

@paper42 paper42 merged commit b385a97 into void-linux:master Oct 11, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
new-package This PR adds a new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants