Spec URL: http://people.redhat.com/steved/rpcbind/rpcbind.spec SRPM URL: http://people.redhat.com/steved/rpcbind/rpcbind-0.1.4-1.fc7.src.rpm Description: The rpcbind utility is a server that converts RPC program numbers into universal addresses. It must be running on the host to be able to make RPC calls on a server on that machine.
I don't think this should be fedora-review+ until someone actually reviews and approves it.
Sorry about that... I ment to set the '?' flag, not the '+' flag assuming that the way things should work...
Couple issues thus far: Mixed tabs/spaces in the spec (URL line is tabbed, rest are spaces) Mixed use of $RPM_BUILD_ROOT and %{buildroot}, please pick one. Do you need to --prefix %{buildroot} ? doesn't %configure set a proper prefix that will be used when you do DESTDIR= in %install? Init files aren't supposed to be config files are they? Why is there an 'exit 0' in your %post ? Also the build fails in mock: security.c:24:27: error: rpcsvc/rquota.h: No such file or directory
> Mixed tabs/spaces in the spec (URL line is tabbed, rest are spaces) changes the tabs to spaces > Mixed use of $RPM_BUILD_ROOT and %{buildroot}, please pick one. %{buildroot} won... > Do you need to --prefix %{buildroot} ? doesn't %configure set a proper prefix > that will be used when you do DESTDIR= in %install? True, --prefix %{buildroot} is not needed... > Init files aren't supposed to be config files are they? Per our IRC converstion, I would like to leave this in since thats the way a number of other RPC/NFS related packages work. > Why is there an 'exit 0' in your %post ? because someone complained about the %post not returning a zero exit code... but it really not needed in this case so I remove it. > security.c:24:27: error: rpcsvc/rquota.h: No such file or directory added a quota requirement so in the end here is the diff... diff -r1.1 rpcbind.spec 9c9 < URL: http://nfsv4.bullopensource.org --- > URL: http://nfsv4.bullopensource.org 19c19 < Requires: libtirpc --- > Requires: libtirpc quota 49,50c49 < --enable-debug \ < --prefix=%{buildroot} --- > --enable-debug 65c64 < rm -rf $RPM_BUILD_ROOT --- > rm -rf %{buildroot} 79d77 < exit 0 The source rpm and spec on my people page have been updated.
You're supposed to add in the spec comments about the changes you've applied. There are still mixed spaces/tabs, (spaces: line 3, tab: line 48) Requires: libtirpc is redundant. RPM automatically picks up and adds a requirement on libtirpc.so.1()(64bit) (for 64bit builds). You don't need to list the libtirpc Requirement. rpcbind is enabled by default, is there good reason for this? Must be asked of any service that is enabled by default.
> There are still mixed spaces/tabs, (spaces: line 3, tab: line 48) There were a couple other places as well... I think I got them all.. > Requires: libtirpc is redundant. RPM automatically picks up and adds a > requirement on libtirpc.so.1()(64bit) (for 64bit builds). You don't need to > list the libtirpc Requirement. Didn't know rpm will do an ldd to figure out which libs are needed... > rpcbind is enabled by default, is there good reason for this? Well this is a portmapper replacement so not only should rpcbind be enable by default, the portmapper needed to be turn off or even removed One final note, the rpcinfo and its man page that are installed from glibc-common need to be removed as well... How does one do that?
Spec URL: http://people.redhat.com/steved/rpcbind/rpcbind.spec SRPM URL: http://people.redhat.com/steved/rpcbind/rpcbind-0.1.4-1.fc7.src.rpm have been updated!
Looks good now. The last thing would be coordinating the Obsoletes/Provides with portmap for the removal of portmap. I'm approving this package (although I'm still not happy about the init script being marked as a config file)
What is the status of coordinating this with the removal of portmap?
Well I'm ready when you are.... Also, we should synchronize the removal of the rpcinfo that lives in glibc-common. Note: the rpcinfo command that lives in the rpcbind package is currently being installed in /usr/bin/ and probably should be moved over to /usr/sbin (once the glibc-common is removed).
rpcbind was built for rawhide, closing.